Skip to content

Accessibility improvements#250

Closed
maxx1128 wants to merge 2 commits into
mansona:mainfrom
SeeClickFix:notification-accessibility-improvements
Closed

Accessibility improvements#250
maxx1128 wants to merge 2 commits into
mansona:mainfrom
SeeClickFix:notification-accessibility-improvements

Conversation

@maxx1128

@maxx1128 maxx1128 commented Nov 8, 2019

Copy link
Copy Markdown

Two changes for making the notifications more accessible.

  1. Add aria elements to the container so screen readers alert users to new notifications.
  2. Make the close element a button so keyboard users can close it.

@maxx1128
maxx1128 force-pushed the notification-accessibility-improvements branch from 257af89 to 0023d41 Compare November 15, 2019 16:12
@hergaiety

Copy link
Copy Markdown

We currently extend the service to do very similar to this, it would be huge to have this merged into a new patch release and would provide an easier upgrade path for us such that we don't need to extend the service anymore.

Anything I can do to help this along @maxx1128 (@ynnoj @mansona)?

@maxx1128
maxx1128 force-pushed the notification-accessibility-improvements branch from 0023d41 to f4e5154 Compare October 5, 2020 18:37
@maxx1128

maxx1128 commented Oct 5, 2020

Copy link
Copy Markdown
Author

Just rebased this on top of the most recent version of the dependency. I agree with @hergaiety that it'd be a big time-saver to just get this merged into the main repo.

pichfl added a commit to pichfl/ember-cli-notifications that referenced this pull request Dec 15, 2021
pichfl added a commit to pichfl/ember-cli-notifications that referenced this pull request Dec 15, 2021
* Better variables
* Slightly more readable colors
* Support `prefers-reduced-motion` and reduces motion in Ember tests by default
* Added proper ARIA role to notification
* Partially replaces mansona#250
pichfl added a commit to pichfl/ember-cli-notifications that referenced this pull request Dec 15, 2021
pichfl added a commit to pichfl/ember-cli-notifications that referenced this pull request Dec 15, 2021
* Better variables
* Slightly more readable colors
* Support `prefers-reduced-motion` and reduces motion in Ember tests by default
* Added proper ARIA role to notification
* Partially replaces mansona#250
pichfl added a commit to pichfl/ember-cli-notifications that referenced this pull request Dec 15, 2021
* Better variables
* Slightly more readable colors
* Support `prefers-reduced-motion` and reduces motion in Ember tests by default
* Added proper ARIA role to notification
* Partially replaces mansona#250
pichfl added a commit to pichfl/ember-cli-notifications that referenced this pull request Jun 8, 2022
* Better variables
* Slightly more readable colors
* Support `prefers-reduced-motion` and reduces motion in Ember tests by default
* Added proper ARIA role to notification
* Partially replaces mansona#250
pichfl added a commit to pichfl/ember-cli-notifications that referenced this pull request Jun 8, 2022
@maxx1128

Copy link
Copy Markdown
Author

This has been ignored for about five years, going to close it.

@maxx1128 maxx1128 closed this May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants