Closed Bug 1429442 Opened 6 years ago Closed 5 years ago

Buttons in "type control" popup in reader mode should have tooltips or at least text for accessibility.

Categories

(Toolkit :: Reader Mode, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox59 --- wontfix
firefox67 --- fixed

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.
Keywords: access
Priority: -- → P2
Whiteboard: [reader-mode-firefox-integration]
Mentor: gijskruitbosch+bugs

Hi, would like to work on this issue. Can you help me and provide some more info.

(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

https://searchfox.org/mozilla-central/source/toolkit/locales/en-US/chrome/global/aboutReader.properties

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.

Summary: Icons in reader mode settings should have tooltips or at least text for accessibility. → Buttons in "type control" popup in reader mode should have tooltips or at least text for accessibility.

Hi,
Can I work on this issue?

(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 ?

Hi I am going through the issue and will submit the patch after solving it.

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.

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.

Thanks. One more doubt for linting what should I do or run.

In your build directory, run ./mach eslint <path_to_your_modified_file>

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?

(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 .

Added tooltips in type control for accessibility

Assignee: nobody → monikamaheshwari1996
Attachment #9049479 - Attachment description: Bug #1429442 Buttons in "type control" popup in reader mode should have tooltips or at least text for accessibility → Bug 1429442 Buttons in "type control" popup in reader mode should have tooltips or at least text for accessibility
Attachment #9049479 - Attachment description: Bug 1429442 Buttons in "type control" popup in reader mode should have tooltips or at least text for accessibility → Bug 1429442 - Buttons in "type control" popup in reader mode should have tooltips or at least text for accessibility
Attachment #9049479 - Attachment description: Bug 1429442 - Buttons in "type control" popup in reader mode should have tooltips or at least text for accessibility → Bug 1429442 - Buttons in "type control" popup in reader mode should have tooltips
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
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
QA Whiteboard: [qa-67b-p2]
You need to log in before you can comment on or make changes to this bug.