Skip to content

Commit 314e7b1

Browse files
committed
Fixing that $so-navigation-toggler-size variable was not used and fixing accessibility warning about its size
"All touch targets must be 24px large, or leave sufficient space"
1 parent f2f44da commit 314e7b1

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

ShowOrchard.Theme/Assets/Styles/components/_navigation.scss

+4-3
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ $so-navigation-opened-menu-item-color: #{$so-color-saltpan};
1010
$so-navigation-active-menu-color: #{$so-color-lunar-green};
1111
$so-navigation-active-opened-menu-bg-color: #{$so-color-cape-cod};
1212
$so-navigation-hover-menu-bg-color: #{$so-color-pumice};
13-
$so-navigation-toggler-size: 22px;
13+
$so-navigation-toggler-size: 24px;
1414

1515
.navigation {
1616
background-color: $so-navigation-bg-color;
@@ -48,8 +48,9 @@ $so-navigation-toggler-size: 22px;
4848

4949
&__toggler {
5050
display: block;
51-
width: 22px;
52-
height: 22px;
51+
min-width: $so-navigation-toggler-size;
52+
width: $so-navigation-toggler-size;
53+
height: $so-navigation-toggler-size;
5354
}
5455

5556
&__togglerIcon {

0 commit comments

Comments
 (0)