diff --git a/src/components/Breadcrumb/Breadcrumb.scss b/src/components/Breadcrumb/Breadcrumb.scss index 52da224092422c..980d626639da91 100644 --- a/src/components/Breadcrumb/Breadcrumb.scss +++ b/src/components/Breadcrumb/Breadcrumb.scss @@ -16,119 +16,115 @@ $Breadcrumb-itemMaxWidth-sm: 116px; .ms-Breadcrumb { @include ms-baseFont; margin: 23px 0 1px; +} - .ms-Breadcrumb-list { - white-space: nowrap; - padding: 0; - margin: 0; - } - - .ms-Breadcrumb-chevron { - font-size: $ms-font-size-s; - color: $ms-color-neutralSecondary; - vertical-align: top; - margin: 11px 7px; - } +.ms-Breadcrumb-list { + white-space: nowrap; + padding: 0; + margin: 0; +} - .ms-Breadcrumb-listItem { - list-style-type: none; - vertical-align: top; - margin: 0; - padding: 0; - display: inline-block; - position: relative; +.ms-Breadcrumb-chevron { + font-size: $ms-font-size-s; + color: $ms-color-neutralSecondary; + vertical-align: top; + margin: 11px 7px; +} - &:last-of-type { - .ms-Breadcrumb-chevron { - display: none; - } - } +.ms-Breadcrumb-listItem { + list-style-type: none; + vertical-align: top; + margin: 0; + padding: 0; + display: inline-block; + position: relative; - .ms-Breadcrumb-itemLink { - @include focus-border; - @include ms-font-xl; - color: $ms-color-neutralPrimary; - display: inline-block; - padding: 0 4px; - max-width: $Breadcrumb-itemMaxWidth; - white-space: nowrap; - text-overflow: ellipsis; - overflow: hidden; - vertical-align: top; + &:last-of-type { + .ms-Breadcrumb-chevron { + display: none; } } - .ms-Breadcrumb-overflow { + .ms-Breadcrumb-itemLink { + @include focus-border; + @include ms-font-xl; + color: $ms-color-neutralPrimary; display: inline-block; - position: relative; - - .ms-Breadcrumb-overflowButton { - font-size: $Breadcrumb-overflowButtonSize; - display: inline-block; - color: $Breadcrumb-overflowButtonColor; - padding: 8px; - cursor: pointer; - vertical-align: top; - } + padding: 0 4px; + max-width: $Breadcrumb-itemMaxWidth; + white-space: nowrap; + text-overflow: ellipsis; + overflow: hidden; + vertical-align: top; } +} - .ms-Breadcrumb-itemLink, - .ms-Breadcrumb-overflowButton { - text-decoration: none; - outline: transparent; +.ms-Breadcrumb-overflow { + display: inline-block; + position: relative; +} - &:hover { - background-color: $selectedHoverBackgroundColor; - color: $Breadcrumb-buttonHoverColor; - cursor: pointer; - } +.ms-Breadcrumb-overflowButton { + font-size: $Breadcrumb-overflowButtonSize; + display: inline-block; + color: $Breadcrumb-overflowButtonColor; + padding: 8px; + cursor: pointer; + vertical-align: top; +} - &:focus { - outline: $ms-color-neutralSecondaryAlt solid 1px; - color: $ms-color-neutralDark; - } +.ms-Breadcrumb-itemLink, +.ms-Breadcrumb-overflowButton { + text-decoration: none; + outline: transparent; - &:active { - outline: transparent; - background-color: $selectedActiveBackgroundColor; - color: $selectedActiveForegroundColor; - } + &:hover { + background-color: $selectedHoverBackgroundColor; + color: $Breadcrumb-buttonHoverColor; + cursor: pointer; } - @media screen and (max-width: $ms-screen-md-max) { - margin: 11px 0 1px; + &:focus { + outline: $ms-color-neutralSecondaryAlt solid 1px; + color: $ms-color-neutralDark; + } - .ms-Breadcrumb-listItem .ms-Breadcrumb-itemLink { - font-size: $ms-font-size-l; - } + &:active { + outline: transparent; + background-color: $selectedActiveBackgroundColor; + color: $selectedActiveForegroundColor; + } +} - .ms-Breadcrumb-chevron { - font-size: $ms-font-size-mi; - margin: 8px 5px; - } +@media screen and (max-width: $ms-screen-md-max) { + margin: 11px 0 1px; - .ms-Breadcrumb-overflow { - .ms-Breadcrumb-overflowButton { - font-size: $ms-font-size-m-plus; - padding: 5px 4px; - } - } + .ms-Breadcrumb-listItem .ms-Breadcrumb-itemLink { + font-size: $ms-font-size-l; } - @media screen and (max-width: $ms-screen-sm-max) { - .ms-Breadcrumb-listItem .ms-Breadcrumb-itemLink { - font-size: $ms-font-size-m; - max-width: $Breadcrumb-itemMaxWidth-sm; - } + .ms-Breadcrumb-chevron { + font-size: $ms-font-size-mi; + margin: 8px 5px; + } - .ms-Breadcrumb-chevron { - margin: 5px 4px; - } + .ms-Breadcrumb-overflowButton { + font-size: $ms-font-size-m-plus; + padding: 5px 4px; + } +} - .ms-Breadcrumb-overflow { - .ms-Breadcrumb-overflowButton { - padding: 2px 4px; - } - } +@media screen and (max-width: $ms-screen-sm-max) { + .ms-Breadcrumb-listItem .ms-Breadcrumb-itemLink { + font-size: $ms-font-size-m; + max-width: $Breadcrumb-itemMaxWidth-sm; + } + + .ms-Breadcrumb-chevron { + margin: 5px 4px; + } + + .ms-Breadcrumb-overflowButton { + padding: 2px 4px; } } diff --git a/src/components/Callout/Callout.scss b/src/components/Callout/Callout.scss index 91fee50c3f89fa..a97fe707774b8d 100644 --- a/src/components/Callout/Callout.scss +++ b/src/components/Callout/Callout.scss @@ -75,15 +75,15 @@ $Callout-beak-slant-width: 28px; .ms-Link.ms-Link--hero { position: relative; - left: -8px; // Move link inline with body text + @include left(-8px); // Move link inline with body text } } .ms-Callout-action { position: relative; top: 4px; - left: -8px; - margin-left: 0 !important; + @include left(-8px); + @include margin-left(0); &:hover .ms-Callout-actionIcon, &:focus .ms-Callout-actionIcon { @@ -92,18 +92,17 @@ $Callout-beak-slant-width: 28px; } .ms-Callout-button { - margin-right: 12px; + @include margin-right(12px); } //= Modifier: Close button callout // .ms-Callout.ms-Callout--close { .ms-Callout-title { - margin-right: 20px; // Avoid overlap with close button + @include margin-right(20px); // Avoid overlap with close button } } - //= Modifier: OOBE (Out of box experience) callout // .ms-Callout.ms-Callout--OOBE { @@ -152,8 +151,8 @@ $Callout-beak-slant-width: 28px; &.ms-Callout--arrowBottom { &:before, &:after { - left: 40px; - right: auto; + @include left(40px); + @include right(auto); } } @@ -161,7 +160,7 @@ $Callout-beak-slant-width: 28px; &.ms-Callout--arrowLeft { &:before, &:after { - top: calc('50% - 10px'); + top: calc(50% - 10px); } } diff --git a/src/components/Check/Check.scss b/src/components/Check/Check.scss index bf4c39cd179ff9..b84c48583025e3 100644 --- a/src/components/Check/Check.scss +++ b/src/components/Check/Check.scss @@ -17,25 +17,6 @@ $checkBoxHeight: 20px; height: $checkBoxHeight; text-align: center; - .ms-Check-background { - &:before { - position: absolute; - top: 0; - bottom: 0; - left: 0; - right: 0; - display: inline-block; - margin: auto; - height: $checkBoxHeight - 2px; - width: $checkBoxHeight - 2px; - background: $ms-color-white; - border: 1px solid $ms-color-neutralTertiaryAlt; - border-radius: 50%; - content: ' '; - } - - } - &.is-checked { .ms-Check-background { &:before { @@ -55,13 +36,31 @@ $checkBoxHeight: 20px; opacity: 1; } } +} - .ms-Check-check { +.ms-Check-check { + left: 0; + right: 0; + position: absolute; + line-height: $checkBoxHeight; + color: $ms-color-neutralTertiaryAlt; + opacity: 0; +} + +.ms-Check-background { + &:before { + position: absolute; + top: 0; + bottom: 0; left: 0; right: 0; - position: absolute; - line-height: $checkBoxHeight; - color: $ms-color-neutralTertiaryAlt; - opacity: 0; + display: inline-block; + margin: auto; + height: $checkBoxHeight - 2px; + width: $checkBoxHeight - 2px; + background: $ms-color-white; + border: 1px solid $ms-color-neutralTertiaryAlt; + border-radius: 50%; + content: ' '; } } \ No newline at end of file diff --git a/src/components/Dropdown/Dropdown.scss b/src/components/Dropdown/Dropdown.scss index d4b32dda095551..6287487b922a25 100644 --- a/src/components/Dropdown/Dropdown.scss +++ b/src/components/Dropdown/Dropdown.scss @@ -9,6 +9,8 @@ $Dropdown-selectedItem-bg: $ms-color-neutralQuaternaryAlt; $Dropdown-selectedItem-hover-bg: $ms-color-neutralLighter; +$DropDown-height: 32px; +$DropDown-item-height: 36px; // Mixin for high contrast mode link states @mixin highContrastListItemState { @@ -150,10 +152,10 @@ $Dropdown-selectedItem-hover-bg: $ms-color-neutralLighter; color: $ms-color-neutralDark; font-size: $ms-icon-size-s; position: absolute; - right: 13px; - bottom: 9px; - z-index: 1; + top: 0px; + @include right(12px); pointer-events: none; + line-height: $DropDown-height; } // Style the new, replacement component @@ -163,8 +165,9 @@ $Dropdown-selectedItem-hover-bg: $ms-color-neutralLighter; border: 1px solid $ms-color-neutralTertiaryAlt; cursor: pointer; display: block; - height: 32px; - padding: 5px 32px 0 10px; + height: $DropDown-height; + line-height: $DropDown-height - 2px; // height minus the border + @include padding(0, $DropDown-height, 0, 12px); position: relative; overflow: hidden; } @@ -215,8 +218,9 @@ $Dropdown-selectedItem-hover-bg: $ms-color-neutralLighter; box-sizing: border-box; cursor: pointer; display: block; - height: 36px; - padding: 7px 10px; + height: $DropDown-item-height; + line-height: $DropDown-item-height - 2px; // height minus the border + padding: 0px 12px; position: relative; border: 1px solid transparent; white-space: nowrap; diff --git a/src/components/Dropdown/Dropdown.tsx b/src/components/Dropdown/Dropdown.tsx index 12c3415759ee60..a336776059ddb4 100644 --- a/src/components/Dropdown/Dropdown.tsx +++ b/src/components/Dropdown/Dropdown.tsx @@ -101,8 +101,8 @@ export class Dropdown extends React.Component { aria-activedescendant={ selectedIndex >= 0 ? (id + '-list' + selectedIndex) : (id + '-list') } aria-controls={ id + '-list' } > - { selectedOption ? selectedOption.text : '' } +