Skip to content

Commit 838f456

Browse files
Keep the amber fill on focused accent buttons
1 parent 57ffce1 commit 838f456

2 files changed

Lines changed: 11 additions & 0 deletions

File tree

‎CHANGELOG.md‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,11 @@
22

33
What changed in each SpaceSharp release, newest first. The 1.x releases are in [CHANGELOG-1.x.md](CHANGELOG-1.x.md). A section marked unreleased is what the next version will say; the marker comes off when it ships, since the release script and the in-app What's new read the `## version` heading.
44

5+
## 2.2.1 (unreleased)
6+
7+
### Fixes
8+
- The Map and Insights buttons went gray with a dark label after a click, because the new keyboard-focus look for toolbar buttons replaced their amber fill. The accent buttons keep the fill and show focus as a dark edge.
9+
510
## 2.2.0
611

712
### Sixteen more languages

‎SpaceSharp/Themes/Styles.xaml‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,12 @@
7575
<Trigger Property="IsMouseOver" Value="True">
7676
<Setter Property="Background" Value="{DynamicResource AccentHover}" />
7777
</Trigger>
78+
<!-- Overrides the base focus look, which would swap the amber fill for the gray hover tone and leave the
79+
dark label on it: the accent button keeps its fill and shows focus as a dark edge instead. -->
80+
<Trigger Property="IsKeyboardFocused" Value="True">
81+
<Setter Property="Background" Value="{DynamicResource AccentHover}" />
82+
<Setter Property="BorderBrush" Value="{DynamicResource AccentText}" />
83+
</Trigger>
7884
<Trigger Property="IsPressed" Value="True">
7985
<Setter Property="Background" Value="{DynamicResource AccentPressed}" />
8086
</Trigger>

0 commit comments

Comments
 (0)