Closed Bug 1369585 Opened 3 years ago Closed 3 years ago

Explicitly show the row / column in grid cell infobar

Categories

(DevTools :: Inspector, enhancement, P3)

enhancement

Tracking

(firefox55 fixed)

RESOLVED FIXED
Firefox 55
Tracking Status
firefox55 --- fixed

People

(Reporter: micah, Assigned: micah)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

We need to be clear about the row / column of a grid cell in the infobar. Instead of just displaying the row/ column numbers, we will show : `Row: ${rowNumber} / Column: ${colNumber}`
Assignee: nobody → tigleym
Status: NEW → ASSIGNED
Priority: -- → P3
Pushed by gabriel.luong@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5e9c3d9dd68e
Explicitly identify the row and columns in cell infobar. r=gl
Comment on attachment 8874073 [details]
Bug 1369585 - Explicitly identify the row and columns in cell infobar.

https://reviewboard.mozilla.org/r/145536/#review149526
Attachment #8874073 - Flags: review?(gl) → review+
https://hg.mozilla.org/mozilla-central/rev/5e9c3d9dd68e
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
> +layout.rowColumnPositions=Row %S \/ Column %S

This change is causing an error on the L10n dashboard:

"unknown escape sequence, \/ at line 48, column 34 for layout.rowColumnPositions"

How about using either only / or a dash here, or perhaps "Row %S, Column %S" to match a nearly similar string (console.properties:errLineCol)?
Flags: needinfo?(tigleym)
Flags: needinfo?(gl)
Pushed by gabriel.luong@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4ee2e37e9452
Remove L10n unknown escape sequence for layout.rowColumnPositions. r=me
Landed a fix.
Flags: needinfo?(tigleym)
Flags: needinfo?(gl)
(In reply to Gabriel [:gl] (ΦωΦ) from comment #8)
> Landed a fix.

Thanks!
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.