Closed Bug 1397609 Opened 4 years ago Closed 4 years ago
Fix a11y of bookmark star and Pocket urlbar buttons
59 bytes, text/x-review-board-request
See bug 1395178 comment 8 and on.
Marco, could you please try the try build? Here's the archive directory, although I've recently noticed that builds often don't appear here anymore: https://firstname.lastname@example.org/ If they don't, I think you can click individual builds on treeherder and from there get build artifacts, from the Job Details tab, or the Inspect Task menu item in the ellipsis menu, or something? I'm sure you already know this, I'm just trying to decrease turnaround time in case you don't since we're in very different time zones. Here's the treeherder link: https://treeherder.mozilla.org/#/jobs?repo=try&revision=29785c2a92c9
The try builds didn't appear in the artifacts directory, but the jobs had links to target files. However, the builds were broken in a major way, every tab I opened immediately crashed no matter which site I loaded. However, I looked at your patch, and am confident this is the right markup.
Comment on attachment 8905363 [details] Bug 1397609 - Fix a11y of bookmark star and Pocket urlbar buttons. https://reviewboard.mozilla.org/r/177154/#review182282 rs=me based on code inspection and Marco's comment. Thanks!
Attachment #8905363 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/7e0c0b8a5c13 Fix a11y of bookmark star and Pocket urlbar buttons. r=Gijs
Thanks Marco. I should have mentioned to feel free to review the code too. :-)
This issue is verified fixed using Firefox 57.0b13 (BuildId:20171030163911) with NVDA on Windows 10 64bit.
You need to log in before you can comment on or make changes to this bug.