diff --git a/src/common/_semanticColorVariables.scss b/src/common/_semanticColorVariables.scss index 675b1281a604dd..50396aa1abc99d 100644 --- a/src/common/_semanticColorVariables.scss +++ b/src/common/_semanticColorVariables.scss @@ -1,4 +1,4 @@ -$focusedBorderColor: $ms-color-neutralSecondaryAlt; +$focusedBorderColor: $ms-color-neutralSecondary; $bodyBackgroundColor: $ms-color-white; $bodyForegroundColor: $ms-color-neutralPrimary; diff --git a/src/components/ContextualMenu/ContextualMenu.scss b/src/components/ContextualMenu/ContextualMenu.scss index 8e6ff438757941..f9bdd9193f170f 100644 --- a/src/components/ContextualMenu/ContextualMenu.scss +++ b/src/components/ContextualMenu/ContextualMenu.scss @@ -1,9 +1,9 @@ @import '../../common/common'; $ContextualMenu-background: $ms-color-white; -$ContextualMenu-itemHover: $ms-color-neutralLight; +$ContextualMenu-itemHover: $ms-color-neutralLighter; $ContextualMenu-expandedItemBackground: $ms-color-neutralQuaternaryAlt; -$ContextualMenu-itemHeight: 40px; +$ContextualMenu-itemHeight: 36px; $ContextualMenu-iconWidth: 24px; .ms-ContextualMenu { @@ -40,15 +40,22 @@ $ContextualMenu-iconWidth: 24px; height: $ContextualMenu-itemHeight; line-height: $ContextualMenu-itemHeight; display: block; + cursor: pointer; @include text-align(left); &:hover:not([disabled]) { background: $ContextualMenu-itemHover; } + .is-focusVisible &:focus { + background: $ContextualMenu-itemHover; + } + &.is-expanded, &.is-expanded:hover { background: $ContextualMenu-expandedItemBackground; + color: $ms-color-black; + font-weight: $ms-font-weight-semibold; } } diff --git a/src/components/Dropdown/Dropdown.scss b/src/components/Dropdown/Dropdown.scss index f90daf14c5f9b3..0f24a606a58a4d 100644 --- a/src/components/Dropdown/Dropdown.scss +++ b/src/components/Dropdown/Dropdown.scss @@ -7,6 +7,9 @@ // -------------------------------------------------- // Dropdown styles +$Dropdown-selectedItem-bg: $ms-color-neutralQuaternaryAlt; +$Dropdown-selectedItem-hover-bg: $ms-color-neutralLighter; + // Mixin for high contrast mode link states @mixin highContrastListItemState { @media screen and (-ms-high-contrast: active) { @@ -27,12 +30,9 @@ } .ms-Dropdown { - @include ms-baseFont; @include ms-u-normalize; - color: $ms-color-neutralPrimary; + @include ms-font-m; - font-size: $ms-font-size-m; - font-weight: $ms-font-weight-regular; margin-bottom: 10px; position: relative; outline: 0; @@ -50,7 +50,7 @@ &:hover, &:active { .ms-Dropdown-title { - border-color: $ms-color-neutralSecondaryAlt; + border-color: $ms-color-themePrimary; } } @@ -205,15 +205,20 @@ } &:hover { - background-color: $ms-color-neutralLight; + background-color: $Dropdown-selectedItem-hover-bg; color: $ms-color-black; @include highContrastListItemState; } + @include focus-border(); + + &:focus { + background-color: $ms-color-neutralLighter; + } + &:active { - background-color: $ms-color-neutralLight; - border-color: $ms-color-themePrimary; + background-color: $Dropdown-selectedItem-hover-bg; color: $ms-color-black; } @@ -229,12 +234,14 @@ // Use .is-selected instead. .ms-Dropdown-item.is-selected, .ms-Dropdown-item.ms-Dropdown-item--selected { - background-color: $ms-color-themeLight; + background-color: $Dropdown-selectedItem-bg; color: $ms-color-black; &:hover { - background-color: $ms-color-themeLight; + background-color: $Dropdown-selectedItem-bg; } + @include focus-border(); + @include highContrastListItemState; } \ No newline at end of file diff --git a/src/demo/pages/ContextualMenuPage/examples/ContextualMenu.Checkmarks.Example.tsx b/src/demo/pages/ContextualMenuPage/examples/ContextualMenu.Checkmarks.Example.tsx index c9dcd1f12051e8..73619d4bae9f97 100644 --- a/src/demo/pages/ContextualMenuPage/examples/ContextualMenu.Checkmarks.Example.tsx +++ b/src/demo/pages/ContextualMenuPage/examples/ContextualMenu.Checkmarks.Example.tsx @@ -40,7 +40,6 @@ export class ContextualMenuCheckmarksExample extends React.Component