Merge pull request #1811 from simon-leech/button-row-flex

The btnRow should be set to 'flex' not 'block'
GEOLYTIX · Jan 15, 2025 · 3a3aae7 · 3a3aae7
2 parents 07a4ff5 + 5a7e923
commit 3a3aae7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/lib/ui/Dataview.mjs b/lib/ui/Dataview.mjs
@@ -154,7 +154,7 @@ function show() {
   }
 
   //Show toolbar buttons if there are any
-  this.btnRow?.style.setProperty('display','block')
+  this.btnRow?.style.setProperty('display','flex')
 
   this.target.style.display = 'block'
 }

diff --git a/lib/ui/Tabview.mjs b/lib/ui/Tabview.mjs
@@ -153,7 +153,7 @@ function addTab(entry) {
     }
 
     //Show toolbar buttons if there are any
-    entry.btnRow?.style.setProperty('display', 'block')
+    entry.btnRow?.style.setProperty('display', 'flex')
   }
 
   /**

diff --git a/tests/lib/ui/Dataview.test.mjs b/tests/lib/ui/Dataview.test.mjs
@@ -34,7 +34,7 @@ export async function DataviewTest(mapview) {
         await codi.it('dataview btnRow should be display true when show method called', async () => {
 
             entry.show();
-            codi.assertEqual(entry.btnRow.style.display, 'block', 'The dataview entry btnRow should be display block.');
+            codi.assertEqual(entry.btnRow.style.display, 'flex', 'The dataview entry btnRow should be display flex.');
 
         });
-Original file line number
+Diff line change
@@ Expand Up / @@ -153,7 +153,7 @@ function addTab(entry) { @@
         }
         //Show toolbar buttons if there are any
-        entry.btnRow?.style.setProperty('display', 'block')
+        entry.btnRow?.style.setProperty('display', 'flex')
       }
       /**
@@ Expand Down @@