Give nav buttons a background on hover

Max Brunsfeld created

Change summary

crates/workspace/src/toolbar.rs   | 1 +
styles/src/styleTree/workspace.ts | 6 ++++--
2 files changed, 5 insertions(+), 2 deletions(-)

Detailed changes

crates/workspace/src/toolbar.rs 🔗

@@ -173,6 +173,7 @@ fn nav_button<A: Action + Clone>(
             .constrained()
             .with_width(style.button_width)
             .with_height(style.button_width)
+            .aligned()
             .boxed()
     })
     .with_cursor_style(if enabled {

styles/src/styleTree/workspace.ts 🔗

@@ -142,13 +142,15 @@ export default function workspace(theme: Theme) {
       navButton: {
         color: iconColor(theme, "primary"),
         iconWidth: 8,
-        buttonWidth: 12,
+        buttonWidth: 18,
+        cornerRadius: 6,
         hover: {
           color: iconColor(theme, "active"),
+          background: backgroundColor(theme, 300),
         },
         disabled: {
           color: iconColor(theme, "muted")
-        }
+        },
       },
       padding: { left: 16, right: 8, top: 4, bottom: 4 },
     },