Closed
Bug 1413599
Opened 7 years ago
Closed 7 years ago
Update our current "switch to ESR" prompt version check to include JAWS versions 19 and lower
Categories
(Core :: Disability Access APIs, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla58
People
(Reporter: jimm, Assigned: jimm)
Details
(Whiteboard: aes+)
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
Felipe
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jimm
:
review+
ritu
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
We just finished up testing of the various JAWS screen readers that are available. We have:
1) older JAWS releases prior to version 18.0.2945, including 17.* versions
2) an updated 18 (18.0.4321) that received various bug fixes
3) the current beta of JAWS 2018 (19.0.3002)
In general we don't feel enough progress has been made here so we're going to update our version information to include newer releases. Hence, we plan to push JAWS users to ESR 52 rather than keep them on 57.
http://searchfox.org/mozilla-central/source/accessible/windows/msaa/Compatibility.cpp#171
Assignee | ||
Updated•7 years ago
|
status-firefox58:
affected → ---
Assignee | ||
Comment 1•7 years ago
|
||
Hey Ritu, this will require an uplift to 57. It's a very minor change involving JAWS version info. We have a conversation going with the vendor to nail down specific version information, after which we'll post a patch here.
Assignee | ||
Updated•7 years ago
|
Priority: -- → P1
Jim, DBolter and I met to discuss the rationale behind this uplift and it's clearly a must fix for 57. Tracked as blocker with the hopes that we'll push the patch in b14 on Friday.
Flags: needinfo?(rkothari)
Hi Tom, Ryan, if we are not already doing so in ESR smoke testing, consider adding JAWS + ESR52.5 testing to the ESR sign off plans. Especially since some release a11y users will be prompted to go use ESR.
Flags: needinfo?(tgrabowski)
Flags: needinfo?(ryanvm)
Assignee | ||
Comment 4•7 years ago
|
||
Assignee: nobody → jmathies
Assignee | ||
Updated•7 years ago
|
Attachment #8924246 -
Flags: review?(dbolter)
Assignee | ||
Comment 5•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Attachment #8924251 -
Flags: review?(felipc)
Comment 6•7 years ago
|
||
Sure, we can add that.
Flags: needinfo?(tgrabowski)
Flags: needinfo?(ryanvm)
Flags: needinfo?(andrei.vaida)
Assignee | ||
Comment 7•7 years ago
|
||
Updated•7 years ago
|
Attachment #8924251 -
Flags: review?(felipc) → review+
Comment 8•7 years ago
|
||
Comment on attachment 8924246 [details] [diff] [review]
part1 - enable the prompt for jaws version 19 or lower
Review of attachment 8924246 [details] [diff] [review]:
-----------------------------------------------------------------
r=me, this should work.
Attachment #8924246 -
Flags: review?(dbolter) → review+
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Pushed by jmathies@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5b1cc7fdfa40
Update the 'switch to ESR' prompt minimum version check to include JAWS versions 19 and lower. r=davidb
https://hg.mozilla.org/integration/mozilla-inbound/rev/7043cf41cae4
Disable the JAWS compatibility prompt in Nightly builds for testing. r=felipe
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Assignee | ||
Updated•7 years ago
|
Whiteboard: aes+
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5b1cc7fdfa40
https://hg.mozilla.org/mozilla-central/rev/7043cf41cae4
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox58:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Hi Jim, please nominate this for uplift to Beta. I would have liked to include this in b14 which we built yesterday. Since we missed that, this will most likely be included in Desktop RC1 (gtb Monday).
Flags: needinfo?(jmathies)
Comment 12•7 years ago
|
||
Pushed by jmathies@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d2d9c80e3684
Backed out changeset 5b1cc7fdfa40
https://hg.mozilla.org/integration/mozilla-inbound/rev/24f4d2216af9
Update the 'switch to ESR' prompt minimum version check to include JAWS versions 18 and lower. r=davidb
Assignee | ||
Comment 13•7 years ago
|
||
Reopening as we're adjusting this version check based on some feedback from Freedom Scientific.
Status: RESOLVED → REOPENED
Flags: needinfo?(jmathies)
Resolution: FIXED → ---
Assignee | ||
Comment 14•7 years ago
|
||
Attachment #8924246 -
Attachment is obsolete: true
Attachment #8925091 -
Flags: review+
Comment 15•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d2d9c80e3684
https://hg.mozilla.org/mozilla-central/rev/24f4d2216af9
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 16•7 years ago
|
||
Comment on attachment 8925091 [details] [diff] [review]
part1 - enable the prompt for jaws version 18 or lower
Approval Request Comment
[Feature/Bug causing the regression]:
Update version information related to JAWS accessibility client ESR prompting
[User impact if declined]:
Users will not receive the prompt, and potentially run into issues with Firefox.
[Is this code covered by automated tests?]:
no, a11y clients are manual test only.
[Has the fix been verified in Nightly?]:
Yes.
[Needs manual test from QE? If yes, steps to reproduce]:
Good to have - confirming current JAWS 18 users who manually install 57 receive the prompt on startup with JAWS running.
[List of other uplifts needed for the feature/fix]:
none
[Is the change risky?]:
no
[Why is the change risky/not risky?]:
well understood code
[String changes made/needed]:
none
Attachment #8925091 -
Flags: approval-mozilla-beta?
Comment on attachment 8925091 [details] [diff] [review]
part1 - enable the prompt for jaws version 18 or lower
This is a must for 57, we'd like Firefox a11y users on incompatible versions of JAWS to use Firefox ESR52 instead of 57.
Attachment #8925091 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 18•7 years ago
|
||
bugherder uplift |
Comment 19•7 years ago
|
||
This issue is verified fixed on 57.0-build1 (20171106194249) using Windows 7 x64 and Windows 10 x64. The switch to ESR prompt is received on startup, on 57 manual installation and with JAWS 18 running.
Flags: needinfo?(andrei.vaida)
You need to log in
before you can comment on or make changes to this bug.
Description
•