Skip to content

DetailsList Example Accessibility Fixes, Bugs 5 & 12#7256

Merged
KevinTCoughlin merged 5 commits into
microsoft:masterfrom
natalieethell:detailsListA11y
Nov 29, 2018
Merged

DetailsList Example Accessibility Fixes, Bugs 5 & 12#7256
KevinTCoughlin merged 5 commits into
microsoft:masterfrom
natalieethell:detailsListA11y

Conversation

@natalieethell

@natalieethell natalieethell commented Nov 29, 2018

Copy link
Copy Markdown
Contributor

Pull request checklist

Description of changes

These changes fix some bugs in the DetailsList Documents Example. A user should now be able to use their keyboard to select multiple rows in the list without the modal selection toggle switching off.

Before:
detailslistmodalselectionnotworking

After:
detailslistmodalselectionworking

The aria label for the File Type column header is now more informative, letting the user know that it can be activated to sort the column on file type. The screen reader will now read out "Column operations for File type, Press to sort on File type".

Focus areas to test

Go to the main DetailsList example. Observe the changes.

Microsoft Reviewers: Open in CodeFlow

@natalieethell natalieethell changed the title DetailsList Example Accessibility Fixes DetailsList Example Accessibility Fixes, Bugs 5 & 12 Nov 29, 2018

@KevinTCoughlin KevinTCoughlin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fantastic usability improvement to this example! Works well in my testing now that I can actually test this example 😄.

:shipit:

@KevinTCoughlin

KevinTCoughlin commented Nov 29, 2018

Copy link
Copy Markdown
Member

bundlesize stalling... closed and re-opened in hopes of waking up 🤖.

@KevinTCoughlin
KevinTCoughlin merged commit 9043341 into microsoft:master Nov 29, 2018
@TestAccessibilityGIT TestAccessibilityGIT mentioned this pull request Nov 30, 2018
8 tasks
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants