After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 708219 - Date & Time panel accessibility fixes
Date & Time panel accessibility fixes
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Date and Time
3.9.x
Other Linux
: Normal normal
: ---
Assigned To: Zeeshan Ali
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-09-17 10:11 UTC by Kalev Lember
Modified: 2013-09-17 14:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
datetime: Use the currently preferred way for setting accessible names (6.24 KB, patch)
2013-09-17 10:13 UTC, Kalev Lember
accepted-commit_now Details | Review
datetime: Use the currently preferred way for setting accessible names (5.01 KB, patch)
2013-09-17 13:29 UTC, Kalev Lember
committed Details | Review
datetime: Sync AM/PM button's accessible name with the text displayed (2.95 KB, patch)
2013-09-17 13:29 UTC, Kalev Lember
committed Details | Review

Description Kalev Lember 2013-09-17 10:11:49 UTC
.
Comment 1 Kalev Lember 2013-09-17 10:13:26 UTC
Created attachment 255082 [details] [review]
datetime: Use the currently preferred way for setting accessible names

For entries that have labels associated, use the mnemonic relationship; and for
those that don't, set up the accessible name.
Comment 2 Rui Matos 2013-09-17 10:29:38 UTC
Review of attachment 255082 [details] [review]:

ok
Comment 3 Kalev Lember 2013-09-17 13:29:00 UTC
Created attachment 255105 [details] [review]
datetime: Use the currently preferred way for setting accessible names

For entries that have labels associated, use the mnemonic relationship; and for
those that don't, set up the accessible name.
Comment 4 Kalev Lember 2013-09-17 13:29:04 UTC
Created attachment 255106 [details] [review]
datetime: Sync AM/PM button's accessible name with the text displayed

Instead of hardcoding the accessible name as "Switch between AM and
PM.", set it to "AM" or "PM" depending on which label the stack switcher
is currently showing.
Comment 5 Joanmarie Diggs (IRC: joanie) 2013-09-17 13:32:23 UTC
Review of attachment 255105 [details] [review]:

This makes good sense to me. Thanks for doing it! (not my place to accept-commit-now it)
Comment 6 Joanmarie Diggs (IRC: joanie) 2013-09-17 13:34:59 UTC
Review of attachment 255106 [details] [review]:

I've not tried this yet, but we discussed this approach in IRC as a means to notify Orca that the name has changed. Given that you tried it with Orca and that Orca is presenting the changed name when it changes and the correct name when you give it focus, I'm all in favor of it. :)

Thanks!
Comment 7 Rui Matos 2013-09-17 13:55:38 UTC
Review of attachment 255105 [details] [review]:

ok
Comment 8 Rui Matos 2013-09-17 14:04:10 UTC
Review of attachment 255106 [details] [review]:

Not tested but seems right
Comment 9 Kalev Lember 2013-09-17 14:38:24 UTC
Attachment 255105 [details] pushed as 538032d - datetime: Use the currently preferred way for setting accessible names
Attachment 255106 [details] pushed as 17a8595 - datetime: Sync AM/PM button's accessible name with the text displayed