[l10nstats] make errors and warnings navigatable in compare-locales view

RESOLVED FIXED

Status

Webtools
Elmo
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: Pike, Assigned: Pike)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 8843386 [details]
going to first issue, and then next etc.

In particular with locales with a lot of missing strings, it's hard to find what errors or warnings there are exactly.

One way to fix this is to add anchors to each, and be able to navigate from one to the other.

I've made a POC, and did a screencast.

It doesn't try to stop adding links on the last issue, you just don't have an anchor to find at that point.
Attachment #8843386 - Flags: feedback?(francesco.lodolo)
Comment on attachment 8843386 [details]
going to first issue, and then next etc.

It looks good.

Maybe "go/jump to the first" instead of "details of the first"?

You probably need to remove the decoration on the arrow when focused (I see a small horizontal line under the last one).
Attachment #8843386 - Flags: feedback?(francesco.lodolo) → feedback+

Comment 2

a year ago
Commit pushed to develop at https://github.com/mozilla/elmo

https://github.com/mozilla/elmo/commit/673348a6b239a3f4602dd402982a918340a1139c
bug 1344306, make errors and warnings in compare-locales discoverable, f=flod

Comment 3

a year ago
Commit pushed to develop at https://github.com/mozilla/elmo

https://github.com/mozilla/elmo/commit/6bfb12b5cb53194a3080d53b662c9c6b7044cb56
bug 1344306, follow-up to make navigation only show when skipping blocks

Add some js to hide those markers that would just skip to the next
line.
As I'm already in js, also hide the last marker, as it's much easier
to do in js than in the django template.
Inspired by feedback from flod.
This is now in both prod and stage, marking FIXED.
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.