Skip to content

Commit 5940828

Browse files
laurensuhLauren SuhDottenpixel
authored
feat(modal): close button bckg color accessibility (#5168)
* feat(modal): close button bckg color accessibility * feat(modal): release notes * fix release notes --------- Co-authored-by: Lauren Suh <[email protected]> Co-authored-by: dugg molidor <[email protected]>
1 parent dce6a68 commit 5940828

File tree

7 files changed

+139
-11
lines changed

7 files changed

+139
-11
lines changed

RELEASENOTES.general.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33

44

55
## 2.21.0 - Upcoming
6+
67
## Fixed
78

89
- Update transparency values (rgba and design tokens) with transparent SLDS global styling hooks

RELEASENOTES.md

Lines changed: 115 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,127 @@
44
## Release 2.21.0 - Upcoming
55

66
## Fixed
7+
- Update transparency values (rgba and design tokens) with transparent SLDS global styling hooks
78
- Changed anchor link ouline border black color to blue-50
9+
- Updated background colors for Standard and Actions icons
10+
- Updated icons to `v10.6.0`
11+
- Standard Set:
12+
- Added `bill_of_materials`
13+
- Added `buyer_group_qualifier`
14+
- Added `datashare_target`
15+
- Added `datashares`
16+
- Added `disclosure_and_compliance`
17+
- Added `document_preview`
18+
- Added `facility_bed`
19+
- Added `market`
20+
- Added `price_sheet`
21+
- Added `mulesoft`
22+
- Utility Set:
23+
- Added `add_source`
24+
- Added `app_web_messaging`
25+
- Added `bookmark_stroke`
26+
- Added `buyer_group_qualifier`
27+
- Added `document_preview`
28+
- Added `expired`
29+
- Added `favorite_alt`
30+
- Added `heart`
31+
- Added `hourglass`
32+
- Added `integration`
33+
- Added `market`
34+
- Added `more`
35+
- Added `notification_off`
36+
- Added `notification_snoozed`
37+
- Added `profile_alt`
38+
- Added `promotion_tiers`
39+
- Added `sender_email`
40+
- Added `service_appointment`
41+
- Added `service_report`
42+
- Added `toggle_off`
43+
- Added `toggle_on`
44+
- Added `mulesoft`
845

946
## Components
47+
### [Buttons](https://www.lightningdesignsystem.com/components/buttons)
48+
#### Added
49+
- Configured `--slds-c-button-text-color-hover` Styling hook in Button with dual-stateful to accurately apply the text color on-hover
50+
51+
### [Global Navigation](https://www.lightningdesignsystem.com/components/global-navigation)
52+
#### Added
53+
- Added `role="presentation"` for globlal navigation's overflow tab item
54+
55+
### [Menus](https://www.lightningdesignsystem.com/components/menus)
56+
## Fixed
57+
- Fixed invalid aria roles issue in subheadings with `role=presentation`
58+
<!-- ## [Unreleased] -->
59+
60+
### [Modals](https://www.lightningdesignsystem.com/components/modals)
61+
#### Fixed
62+
- Fixed `aria-labelledby` to meet accessibility requirements specifically for screen readers.
63+
1064
### [Modals](https://www.lightningdesignsystem.com/components/modals)
1165
#### Changed
1266
- Added full size option for modal. This will allow the modal to reach full screen for mobile screen size and devices. When screen is bigger than mobile screen width (from 768px), full size modal will take on large size modal behaviour.
67+
- Changed close button to meet accessibility requirement when in full size and in mobile width.
68+
69+
### [Path](https://www.lightningdesignsystem.com/components/path)
70+
#### Added
71+
- Added `aria-labelledby` to meet accessibility requirements specifically for screen readers.
72+
73+
### [Picklist](https://www.lightningdesignsystem.com/components/picklist)
74+
#### Added
75+
- Added aria-label and tabindex to fix aria-allowed-attr and aria-label sa11y issue.
76+
<!-- ## [Unreleased] -->
77+
78+
### [Picklist](https://www.lightningdesignsystem.com/components/picklist)
79+
#### Added
80+
- Added `aria-label` for picklist examples to meet accessibility requirements.
81+
82+
### [Popovers](https://www.lightningdesignsystem.com/components/popovers)
83+
#### Added button to the story `body-max-height-small` to fix scrollable-region-focusable wcag issue.
84+
## 2.19
85+
#### Fixed
86+
- Fixed hardcoded walk through header and alt image paths
87+
<!-- ## [Unreleased] -->
88+
89+
### [Progress Bar](https://www.lightningdesignsystem.com/components/progress-bar)
90+
#### Added
91+
- Added id to aria-labelledby to meet accessibility requirements.
92+
<!-- ## [Unreleased] -->
93+
94+
### [Progress Indicator](https://www.lightningdesignsystem.com/components/progress-indicator)
95+
#### Added
96+
- Added id to aria-labelledby to meet accessibility requirements.
97+
<!-- ## [Unreleased] -->
98+
99+
### [Progress Ring](https://www.lightningdesignsystem.com/components/progress-ring)
100+
#### Added
101+
- Added aria-label attribute to the progress ring to meet accessibility requirements.
102+
103+
### [Slider](https://www.lightningdesignsystem.com/components/slider)
104+
#### Added
105+
- Added unique id to fix duplicate-id-aria wcag issue
106+
<!-- ## [Unreleased] -->
107+
108+
### [Tabs](https://www.lightningdesignsystem.com/components/tabs)
109+
#### Added
110+
- Added `role="presentation"` for overflow tab item.
111+
112+
### [Vertical Navigation](https://www.lightningdesignsystem.com/components/vertical-navigation)
113+
#### Added
114+
- Configured `slds-nav-vertical__item` to add keyboard visual focus cue indicator
115+
116+
### [Welcome Mat](https://www.lightningdesignsystem.com/components/welcome-mat)
117+
#### Fixed
118+
- Fixed by adding tabindex="0" and role="region" for Scrollable region must have keyboard access wcag issue
119+
## 2.19
120+
#### Fixed
121+
- Fixed hardcoded base info background image path
122+
<!-- ## [Unreleased] -->
123+
124+
## Utilities
125+
### [Scrollable Utility](https://www.lightningdesignsystem.com/utilities/scrollable)
126+
#### Added
127+
- Added tabindex to fix Scrollable region must have keyboard access wcag issue
13128

14129
## Release 2.20.1 - December 16, 2022
15130

ui/components/menus/RELEASENOTES.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
# Menus Release Notes
44

5-
## 2.20.1
5+
## 2.21.0
66

77
## Fixed
88

ui/components/modals/RELEASENOTES.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
## 2.21.0
1212
### Changed
1313
- Added full size option for modal. This will allow the modal to reach full screen for mobile screen size and devices. When screen is bigger than mobile screen width (from 768px), full size modal will take on large size modal behaviour.
14+
- Changed close button to meet accessibility requirement when in full size and in mobile width.
1415

1516
## 2.19.0
1617
### Changed

ui/components/modals/base/_index.scss

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -345,6 +345,17 @@
345345
}
346346
}
347347

348+
/* Close button style for modal size="full" in mobile */
349+
.slds-modal_full-close-button {
350+
border: none;
351+
352+
@include mq-medium-min {
353+
/* Revert back to non-mobile sizing style.
354+
All variants' closing button should look the same once out of mobile sizing. */
355+
background-color: transparent;
356+
}
357+
}
358+
348359
.slds-modal__close {
349360
right: $spacing-medium;
350361

ui/components/modals/base/example.jsx

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,8 @@ export let Modal = (props) => (
5959
props.closeButton !== false ? (
6060
<ButtonIcon
6161
className={classNames('slds-modal__close',
62-
{'slds-button_icon-inverse': props.closeButtonInverse }
62+
{'slds-button_icon-inverse': !props.isFullSize,
63+
'slds-button_icon-border-filled slds-modal_full-close-button': props.isFullSize }
6364
)}
6465
iconClassName="slds-button__icon_large"
6566
symbol="close"
@@ -75,13 +76,13 @@ export let Modal = (props) => (
7576
Modal.propTypes = {
7677
tabIndex: PropTypes.oneOf(['-1', '0']),
7778
closeButtonDisabled: PropTypes.bool,
78-
closeButtonInverse: PropTypes.bool,
79+
isFullSize: PropTypes.bool,
7980
};
8081

8182
Modal.defaultProps = {
8283
tabIndex: '-1',
8384
closeButtonDisabled: false,
84-
closeButtonInverse: true,
85+
isFullSize: false,
8586
};
8687

8788
export let ModalHeader = (props) => (
@@ -184,10 +185,9 @@ export let ModalSizes = (props) => (
184185
props.size === 'small' && 'slds-modal_small',
185186
props.size === 'medium' && 'slds-modal_medium',
186187
props.size === 'large' && 'slds-modal_large',
187-
props.size === 'full' &&
188-
'slds-modal_full'
188+
props.size === 'full' && 'slds-modal_full'
189189
)}
190-
closeButtonInverse={props.size !== 'full'}
190+
isFullSize={props.size === 'full'}
191191
aria-labelledby="modal-heading-01"
192192
>
193193
<ModalHeader>
@@ -225,7 +225,7 @@ export let FullTaglines = (props) => (
225225
props.size === 'full' &&
226226
'slds-modal_full'
227227
)}
228-
closeButtonInverse={props.size !== 'full'}>
228+
isFullSize={props.size === 'full'}>
229229
<ModalHeader>
230230
<h1 id="modal-heading-01" className="slds-modal__title slds-hyphenate">
231231
Modal header
@@ -265,7 +265,7 @@ export let FullHeadless = (props) => (
265265
'slds-modal_full'
266266
)}
267267
aria-labelledby="modal-heading-01"
268-
closeButtonInverse={props.size !== 'full'}
268+
isFullSize={props.size === 'full'}
269269
>
270270
<ModalContent className={classNames(
271271
props.size === 'full' && 'slds-modal_full-content slds-p-around_medium slds-modal__content_headless'
@@ -297,7 +297,7 @@ export let FullFootless = (props) => (
297297
'slds-modal_full'
298298
)}
299299
aria-labelledby="modal-heading-01"
300-
closeButtonInverse={props.size !== 'full'}
300+
isFullSize={props.size === 'full'}
301301
>
302302
<ModalHeader>
303303
<h1 id="modal-heading-01" className="slds-modal__title slds-hyphenate" tabindex="-1">

ui/components/progress-indicator/RELEASENOTES.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
# Progress Indicator Release Notes
44

5-
## 2.20.0
5+
## 2.21.0
66

77
### Added
88

0 commit comments

Comments
 (0)