Buttons in "type control" popup in reader mode should have tooltips or at least text for accessibility.
Categories
(Toolkit :: Reader Mode, enhancement, P2)
Tracking
()
People
(Reporter: mkaply, Assigned: MonikaMaheshwari, Mentored)
Details
(Keywords: access, Whiteboard: [reader-mode-firefox-integration])
Attachments
(1 file)
Currently when you click on the Aa in reader mode, you get a set of options, but the icons aren't 100% clear as to what they do. They should have tooltips at least. In addition, by not having any text, they are not accessible.
Updated•6 years ago
|
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Hi, would like to work on this issue. Can you help me and provide some more info.
Comment 2•5 years ago
|
||
(In reply to Monika Maheshwari [:MonikaMaheshwari] from comment #1)
Hi, would like to work on this issue. Can you help me and provide some more info.
Sure. The relevant reader mode code lives in:
https://searchfox.org/mozilla-central/source/toolkit/components/reader/AboutReader.jsm
and
https://searchfox.org/mozilla-central/source/toolkit/components/reader/content/aboutReader.html
and
We'll need to come up with tooltip text for the buttons in the the popup that appears when you click the "Aa" button on the sidebar in reader mode. We should define those as localizable strings in the aboutReader.properties file. You can then set a "title" attribute on the relevant buttons from the JS in the .jsm, using the strings fetched through the gStrings
variable in AboutReader.jsm (see other uses in that file for how to do so) .
Let me know if you get stuck or need more help on some of this.
Comment 4•5 years ago
|
||
(In reply to trushita from comment #3)
Hi,
Can I work on this issue?
Monika asked first, so perhaps there's a different issue you can work on, perhaps bug 1154295 ?
Assignee | ||
Comment 5•5 years ago
|
||
Hi I am going through the issue and will submit the patch after solving it.
Assignee | ||
Comment 6•5 years ago
|
||
Hey can you tell me the exact title that I need to put on each button. For example on first button I have gave the title Minus
.
Comment 7•5 years ago
|
||
The label should say what the button does. If it's represented by a graphical minus sign, it probably decreases something, so give it a title like "Decrease ..." where ... is what is actually being decreased. For example "Decrease font size" or such.
Assignee | ||
Comment 8•5 years ago
|
||
Thanks. One more doubt for linting what should I do or run.
Comment 9•5 years ago
|
||
In your build directory, run ./mach eslint <path_to_your_modified_file>
Assignee | ||
Comment 10•5 years ago
|
||
Thanks for help. I have submit my first patch (bug) and now I am submitting the second patch (bug) but when doing arc diff
it is giving details on initial patch. Am I doing it correctly?
Comment 11•5 years ago
|
||
(In reply to Monika Maheshwari [:MonikaMaheshwari] from comment #10)
Thanks for help. I have submit my first patch (bug) and now I am submitting the second patch (bug) but when doing
arc diff
it is giving details on initial patch. Am I doing it correctly?
It's easiest to use moz-phab submit .
to submit the commit for this second bug without affecting the earlier commit for the other bug. See https://github.com/mozilla-conduit/review .
Assignee | ||
Comment 12•5 years ago
|
||
Added tooltips in type control for accessibility
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 13•5 years ago
|
||
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/a423e8a55266 Buttons in "type control" popup in reader mode should have tooltips r=Gijs,MarcoZ
Comment 14•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Updated•5 years ago
|
Description
•