Skip to content

Commit

Permalink
Merge branch 'master' into 6935-overflowmenu-unnecessary-a11y-attrs
Browse files Browse the repository at this point in the history
  • Loading branch information
dakahn committed Oct 19, 2020
2 parents 724922a + b162614 commit 1fcf335
Show file tree
Hide file tree
Showing 29 changed files with 337 additions and 130 deletions.
2 changes: 1 addition & 1 deletion .github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ contributing environment. Please take a look at our
Before contributing to Carbon, you should make sure you have the following tools
installed:

- [Node.js](https://nodejs.org/en/download/) v10 or above here or follow their
- [Node.js](https://nodejs.org/en/download/) v12 or above here or follow their
installation through a package manager
[here](https://nodejs.org/en/download/package-manager/))
- If you're on macOS, we recommend using
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/deploy-react-storybook.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
name: Deploy React storybook to IBM Cloud

on:
workflow_dispatch:
push:
tags:
# Matches tags that have the shape `vX.Y.Z`. Reference:
Expand Down Expand Up @@ -37,9 +38,7 @@ jobs:
ibmcloud target -o 'carbon-design-system' -s 'production'
- name: Install IBM Cloud plugins
run: |
# We use v6.50.0 as v6.51.0 and v6.52.0 have a bug where we are unable to set
# the CF API
ibmcloud cf install -v 6.50.0
ibmcloud cf install
ibmcloud cf add-plugin-repo CF-Community https://plugins.cloudfoundry.org
ibmcloud cf install-plugin blue-green-deploy -f -r CF-Community
- name: Deploy React storybook
Expand Down
77 changes: 76 additions & 1 deletion packages/components/docs/sass.md
Original file line number Diff line number Diff line change
Expand Up @@ -13641,6 +13641,16 @@ Accordion styles
}
}

// Size styles
.#{$prefix}--accordion--xl .#{$prefix}--accordion__heading {
min-height: rem(48px);
}

.#{$prefix}--accordion--sm .#{$prefix}--accordion__heading {
min-height: rem(32px);
padding: rem(5px) 0;
}

// Disabled styles
.#{$prefix}--accordion__heading[disabled] {
color: $disabled-02;
Expand Down Expand Up @@ -14121,6 +14131,13 @@ Button styles

.#{$prefix}--btn.#{$prefix}--btn--icon-only.#{$prefix}--tooltip__trigger:focus {
border-color: $focus;

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
outline: 3px solid transparent;
outline-offset: -3px;
}
}

.#{$prefix}--btn.#{$prefix}--btn--icon-only.#{$prefix}--tooltip__trigger:focus
Expand Down Expand Up @@ -14365,6 +14382,13 @@ Button variant styles
border-color: $focus;
box-shadow: inset 0 0 0 $button-outline-width $focus, inset 0 0 0
$button-border-width $ui-02;

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
outline: 3px solid transparent;
outline-offset: -3px;
}
}

&:disabled:hover,
Expand Down Expand Up @@ -15141,7 +15165,11 @@ Combo box styles
.#{$prefix}--combo-box .#{$prefix}--list-box__field,
.#{$prefix}--combo-box.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field,
.#{$prefix}--combo-box.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field,
.#{$prefix}--combo-box.#{$prefix}--list-box--disabled.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field,
.#{$prefix}--combo-box.#{$prefix}--list-box--disabled.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field {
padding: 0;
}
Expand Down Expand Up @@ -17861,7 +17889,6 @@ Dropdown styles

.#{$prefix}--dropdown--disabled {
border-bottom-color: transparent;
cursor: not-allowed;

&:hover {
background-color: $field-01;
Expand All @@ -17888,6 +17915,11 @@ Dropdown styles
}
}

.#{$prefix}--dropdown--disabled .#{$prefix}--list-box__field,
.#{$prefix}--dropdown--disabled .#{$prefix}--list-box__menu-icon {
cursor: not-allowed;
}

.#{$prefix}--dropdown--auto-width {
width: auto;
max-width: rem(400px);
Expand Down Expand Up @@ -18408,6 +18440,7 @@ Form styles
.#{$prefix}--date-picker-input__wrapper,
.#{$prefix}--time-picker--invalid,
.#{$prefix}--text-input__field-wrapper[data-invalid],
.#{$prefix}--text-input__field-wrapper--warning,
.#{$prefix}--text-input__field-wrapper--warning > .#{$prefix}--text-input,
.#{$prefix}--text-area__wrapper[data-invalid],
.#{$prefix}--select-input__wrapper[data-invalid],
Expand Down Expand Up @@ -19135,13 +19168,20 @@ List box styles

// invalid && populated input field
.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field
.#{$prefix}--text-input,
.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field
.#{$prefix}--text-input {
// to account for clear input button outline
padding-right: rem(98px);
}

.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field
.#{$prefix}--text-input
+ .#{$prefix}--list-box__invalid-icon,
.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field
.#{$prefix}--text-input
+ .#{$prefix}--list-box__invalid-icon {
Expand All @@ -19156,12 +19196,19 @@ List box styles

// invalid && empty input field
.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field
.#{$prefix}--text-input--empty,
.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field
.#{$prefix}--text-input--empty {
padding-right: carbon--mini-units(9);
}

.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field
.#{$prefix}--text-input--empty
+ .#{$prefix}--list-box__invalid-icon,
.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field
.#{$prefix}--text-input--empty
+ .#{$prefix}--list-box__invalid-icon {
Expand Down Expand Up @@ -20981,6 +21028,13 @@ Number input styles
align-items: center;
justify-content: center;
transform: translateY(-50%);

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
outline: 1px solid transparent;
outline-offset: -1px;
}
}

.#{$prefix}--number__control-btn {
Expand Down Expand Up @@ -22996,6 +23050,13 @@ Select styles
@include focus-outline('outline');

color: $text-01;

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
outline: 3px solid transparent;
outline-offset: -3px;
}
}

&:disabled,
Expand Down Expand Up @@ -25635,6 +25696,13 @@ Toggle styles
$duration--fast-01 motion(exit, productive);
content: '';
will-change: box-shadow;

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
// `ButtonText` is a CSS2 system color to help improve colors in HCM
border: 1px solid ButtonText;
}
}

// Toggle circle
Expand All @@ -25650,6 +25718,13 @@ Toggle styles
border-radius: 50%;
transition: transform $duration--fast-01 motion(exit, productive);
content: '';

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
// `ButtonText` is a CSS2 system color to help improve colors in HCM
border: 3px solid ButtonText;
}
}

.#{$prefix}--toggle-input__label & {
Expand Down
4 changes: 2 additions & 2 deletions packages/components/package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "carbon-components",
"description": "The Carbon Design System is IBM’s open-source design system for products and experiences.",
"version": "10.22.0-rc.0",
"version": "10.22.0",
"license": "Apache-2.0",
"main": "umd/index.js",
"module": "es/index.js",
Expand Down Expand Up @@ -79,7 +79,7 @@
"@babel/preset-env": "^7.10.0",
"@babel/preset-react": "^7.10.0",
"@babel/runtime": "^7.10.0",
"@carbon/elements": "^10.20.0",
"@carbon/elements": "^10.21.0",
"@carbon/icons-handlebars": "^10.19.0",
"@carbon/icons-react": "^10.19.0",
"@carbon/test-utils": "^10.11.0",
Expand Down
10 changes: 10 additions & 0 deletions packages/components/src/components/accordion/_accordion.scss
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,16 @@
}
}

// Size styles
.#{$prefix}--accordion--xl .#{$prefix}--accordion__heading {
min-height: rem(48px);
}

.#{$prefix}--accordion--sm .#{$prefix}--accordion__heading {
min-height: rem(32px);
padding: rem(5px) 0;
}

// Disabled styles
.#{$prefix}--accordion__heading[disabled] {
color: $disabled-02;
Expand Down
7 changes: 7 additions & 0 deletions packages/components/src/components/button/_button.scss
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,13 @@

.#{$prefix}--btn.#{$prefix}--btn--icon-only.#{$prefix}--tooltip__trigger:focus {
border-color: $focus;

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
outline: 3px solid transparent;
outline-offset: -3px;
}
}

.#{$prefix}--btn.#{$prefix}--btn--icon-only.#{$prefix}--tooltip__trigger:focus
Expand Down
7 changes: 7 additions & 0 deletions packages/components/src/components/button/_mixins.scss
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,13 @@
border-color: $focus;
box-shadow: inset 0 0 0 $button-outline-width $focus,
inset 0 0 0 $button-border-width $ui-02;

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
outline: 3px solid transparent;
outline-offset: -3px;
}
}

&:disabled:hover,
Expand Down
4 changes: 4 additions & 0 deletions packages/components/src/components/combo-box/_combo-box.scss
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,11 @@
.#{$prefix}--combo-box .#{$prefix}--list-box__field,
.#{$prefix}--combo-box.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field,
.#{$prefix}--combo-box.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field,
.#{$prefix}--combo-box.#{$prefix}--list-box--disabled.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field,
.#{$prefix}--combo-box.#{$prefix}--list-box--disabled.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field {
padding: 0;
}
Expand Down
6 changes: 5 additions & 1 deletion packages/components/src/components/dropdown/_dropdown.scss
Original file line number Diff line number Diff line change
Expand Up @@ -349,7 +349,6 @@

.#{$prefix}--dropdown--disabled {
border-bottom-color: transparent;
cursor: not-allowed;

&:hover {
background-color: $field-01;
Expand All @@ -376,6 +375,11 @@
}
}

.#{$prefix}--dropdown--disabled .#{$prefix}--list-box__field,
.#{$prefix}--dropdown--disabled .#{$prefix}--list-box__menu-icon {
cursor: not-allowed;
}

.#{$prefix}--dropdown--auto-width {
width: auto;
max-width: rem(400px);
Expand Down
1 change: 1 addition & 0 deletions packages/components/src/components/form/_form.scss
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,7 @@
.#{$prefix}--date-picker-input__wrapper,
.#{$prefix}--time-picker--invalid,
.#{$prefix}--text-input__field-wrapper[data-invalid],
.#{$prefix}--text-input__field-wrapper--warning,
.#{$prefix}--text-input__field-wrapper--warning > .#{$prefix}--text-input,
.#{$prefix}--text-area__wrapper[data-invalid],
.#{$prefix}--select-input__wrapper[data-invalid],
Expand Down
14 changes: 14 additions & 0 deletions packages/components/src/components/list-box/_list-box.scss
Original file line number Diff line number Diff line change
Expand Up @@ -315,13 +315,20 @@ $list-box-menu-width: rem(300px);

// invalid && populated input field
.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field
.#{$prefix}--text-input,
.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field
.#{$prefix}--text-input {
// to account for clear input button outline
padding-right: rem(98px);
}

.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field
.#{$prefix}--text-input
+ .#{$prefix}--list-box__invalid-icon,
.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field
.#{$prefix}--text-input
+ .#{$prefix}--list-box__invalid-icon {
Expand All @@ -336,12 +343,19 @@ $list-box-menu-width: rem(300px);

// invalid && empty input field
.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field
.#{$prefix}--text-input--empty,
.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field
.#{$prefix}--text-input--empty {
padding-right: carbon--mini-units(9);
}

.#{$prefix}--list-box[data-invalid]
.#{$prefix}--list-box__field
.#{$prefix}--text-input--empty
+ .#{$prefix}--list-box__invalid-icon,
.#{$prefix}--list-box--warning
.#{$prefix}--list-box__field
.#{$prefix}--text-input--empty
+ .#{$prefix}--list-box__invalid-icon {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,13 @@
align-items: center;
justify-content: center;
transform: translateY(-50%);

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
outline: 1px solid transparent;
outline-offset: -1px;
}
}

.#{$prefix}--number__control-btn {
Expand Down
7 changes: 7 additions & 0 deletions packages/components/src/components/select/_select.scss
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,13 @@
@include focus-outline('outline');

color: $text-01;

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
outline: 3px solid transparent;
outline-offset: -3px;
}
}

&:disabled,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,13 @@
+ .#{$prefix}--structured-list-td
.#{$prefix}--structured-list-svg {
fill: $icon-01;

// Windows, Firefox HCM Fix
@media screen and (-ms-high-contrast: active),
screen and (prefers-contrast) {
// `ButtonText` is a CSS2 system color to help improve colors in HCM
fill: ButtonText;
}
}

// Skeleton State
Expand Down
Loading

0 comments on commit 1fcf335

Please sign in to comment.