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

RESOLVED FIXED in Firefox 67

Status

()

enhancement
P2
normal
RESOLVED FIXED
a year ago
24 days ago

People

(Reporter: mkaply, Assigned: MonikaMaheshwari, Mentored)

Tracking

({access})

Trunk
mozilla67
Points:
---

Firefox Tracking Flags

(firefox59 affected, firefox67 fixed)

Details

(Whiteboard: [reader-mode-firefox-integration])

Attachments

(1 attachment)

(Reporter)

Description

a year ago
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

a year ago
Keywords: access
Priority: -- → P2
Whiteboard: [reader-mode-firefox-integration]

Updated

2 months ago
Mentor: gijskruitbosch+bugs

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

Comment 2

2 months 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

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.

Comment 3

2 months ago

Hi,
Can I work on this issue?

Comment 4

2 months 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 ?

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?

Comment 11

a month 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 .

Added tooltips in type control for accessibility

Updated

a month ago
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

Comment 13

a month 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

a month ago
bugherder
Status: NEW → RESOLVED
Last Resolved: a month 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.