Make access key tests more verbose in reporting affected nodes

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

unspecified

Firefox Tracking Flags

(firefox13 fixed, firefox14 fixed, firefox15 fixed, firefox16 fixed, firefox-esr10 fixed)

Details

(Whiteboard: [mozmill-l10n][lib][qa-])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Given the changes to _reportNode over on bug 741762 we should also improve the access key test by using the same output. Axel, would you mind also fixing that? I strongly believe it will help localizers a lot.
(Assignee)

Comment 1

5 years ago
Actually it's a trivial fix and I will take it.
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
(Assignee)

Updated

5 years ago
Whiteboard: [mozmill-l10n]
(Assignee)

Comment 2

5 years ago
Created attachment 631293 [details] [diff] [review]
Patch v1

This is trivial and produces way better output. Here an example report:
http://mozmill-crowd.blargon7.com/#/l10n/report/87961186c2b807b7747aa7e6d40701bf
Attachment #631293 - Flags: review?(l10n)
(Assignee)

Updated

5 years ago
Whiteboard: [mozmill-l10n] → [mozmill-l10n][lib]

Updated

5 years ago
Attachment #631293 - Flags: review?(l10n) → review+
(Assignee)

Comment 3

5 years ago
Pushed:
http://hg.mozilla.org/qa/mozmill-tests/rev/1bd8e9016b02 (default)
http://hg.mozilla.org/qa/mozmill-tests/rev/5587c99e49dc (aurora)
http://hg.mozilla.org/qa/mozmill-tests/rev/cabf7204f5d9 (beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/4d1613eb3adf (release)
http://hg.mozilla.org/qa/mozmill-tests/rev/dde4fb2d7f3b (esr10)
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-firefox-esr10: --- → fixed
status-firefox13: --- → fixed
status-firefox14: --- → fixed
status-firefox15: --- → fixed
status-firefox16: --- → fixed
Resolution: --- → FIXED
Whiteboard: [mozmill-l10n][lib] → [mozmill-l10n][lib][qa-]
You need to log in before you can comment on or make changes to this bug.