Opened 8 years ago
Closed 8 years ago
#42037 closed defect (bug) (fixed)
Nearby events dashboard widget "city updated" speak message announced on page load
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.9 | Priority: | normal |
| Severity: | normal | Version: | 4.8 |
| Component: | Administration | Keywords: | has-screenshots has-patch commit |
| Focuses: | accessibility, javascript | Cc: |
Description
Just noticed (a bit late...) that when loading the dashboard page, the Nearby events widgets renders its template and also sends to the ARIA live region the assertive message "city updated"
this is not ideal on page load, since users haven't taken any action and there's no need to confirm an action that hasn't happened. It's also confusing and a bit annoying hearing that message every time the dashboard screen is loaded.
Attachments (1)
Change History (6)
#2
@
8 years ago
- Summary changed from Nearby events dashboard widget "city updated" speak message announced on page lod to Nearby events dashboard widget "city updated" speak message announced on page load
#3
@
8 years ago
- Focuses accessibility added
- Keywords commit added
- Owner set to afercia
- Status changed from new to assigned
Good catch!
I was able to reproduce this before applying 42037.diff, and applying it fixes the issue for me. The message is still announce when manually changing the city, too.
The diff itself also looks good, so I'd say go ahead and commit it :)
Note: See
TracTickets for help on using
tickets.

42037.diff checks for
initiatedByto send the "city updated" message. This way, the message should be announced only when the city gets updated by users. Not sure if it's the best option or there are better ways. Also, I guess the similarunknownCitydoesn't need the same check. /cc @iandunn (hey)