[gchq#181] returns banner height to original 30 pixels

valdelaseras · Feb 3, 2024 · cf59bd3 · cf59bd3
1 parent b51a298
commit cf59bd3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/web/App.mjs b/src/web/App.mjs
@@ -842,14 +842,14 @@ class App {
     /**
      * Due to variable available heights on mobile devices ( due to the
      * address bar etc. ), we need to calculate the available space and
-     * set some heights programmatically based on the full view height,
-     * minus fixed height elements.
+     * set some heights programmatically based on the full viewport
+     * minus fixed-height elements.
      *
      * Be mindful to update these fixed numbers accordingly in the stylesheets
      * ( themes/_structure ) if you make changes to those elements' height.
      */
     assignAvailableHeight() {
-        const bannerHeight = 40;
+        const bannerHeight = 30;
         const controlsHeight = 50;
         const operationsHeight = 80;
 

diff --git a/src/web/stylesheets/themes/_structure.css b/src/web/stylesheets/themes/_structure.css
@@ -1,6 +1,6 @@
 :root {
     /* Fixed heights */
-    --banner-height: 40px;
+    --banner-height: 30px;
 
     /* Mobile height */
     --title-height: 40px;
@@ -16,7 +16,7 @@
  * A note:
  *
  * Heights of #recipe, #input and #output are set programmatically
- * in App.js
+ * in App.js assignAvailableHeight();
  */