Avoid flickering welcome banner
authorTom Hughes <tom@compton.nu>
Mon, 10 Sep 2018 20:21:43 +0000 (21:21 +0100)
committerTom Hughes <tom@compton.nu>
Mon, 10 Sep 2018 20:21:43 +0000 (21:21 +0100)
Make the welcome banner default to being hidden and then show
it (by adding a class) if there is no cookie.

Fixes #1951

app/assets/javascripts/index.js
app/assets/stylesheets/common.scss
app/assets/stylesheets/small.scss

index ccf4e56..e8e566f 100644 (file)
@@ -187,12 +187,12 @@ $(document).ready(function () {
     $.cookie('_osm_location', OSM.locationCookie(map), { expires: expiry, path: '/' });
   });
 
-  if ($.cookie('_osm_welcome') === 'hide') {
-    $('.welcome').hide();
+  if ($.cookie('_osm_welcome') !== 'hide') {
+    $('.welcome').addClass('visible');
   }
 
   $('.welcome .close-wrap').on('click', function() {
-    $('.welcome').hide();
+    $('.welcome').removeClass('visible');
     $.cookie('_osm_welcome', 'hide', { expires: expiry, path: '/' });
   });
 
index cf10193..0ba5e65 100644 (file)
@@ -611,11 +611,18 @@ body.compact {
     height: auto;
     overflow: hidden;
 
-    #banner,
-    .welcome {
+    #banner {
       display: block;
     }
 
+    .welcome {
+      display: none;
+
+      &.visible {
+        display: block;
+      }
+    }
+
     #sidebar_content {
       display: none;
     }
index 9843647..12e21c7 100644 (file)
@@ -126,9 +126,12 @@ body.small {
     }
   }
 
-  #sidebar .welcome,
-  #sidebar #banner {
-    display: none !important;
+  .overlay-sidebar #sidebar .welcome.visible {
+    display: none;
+  }
+
+  .overlay-sidebar #sidebar #banner {
+    display: none;
   }
 
   .leaflet-top.leaflet-right {