Add a negativeLineNumber to GridLine to be exposed to devtools

RESOLVED FIXED in Firefox 59

Status

()

enhancement
P1
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: gl, Assigned: bradwerth)

Tracking

unspecified
mozilla59
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox59 fixed)

Details

(Whiteboard: [designer-tools])

Attachments

(3 attachments)

Reporter

Description

2 years ago
No description provided.
Reporter

Updated

2 years ago
Blocks: 1387511
Priority: -- → P1
Whiteboard: [designer-tools]
Assignee

Comment 4

2 years ago
In implementing this, I've found a problem with the existing line numbers for grids where an implicit track is added before some removed repeat: auto-fit tracks. That needs to be fixed first to make this logic robust. I'll add a blocker bug soon.
Assignee

Updated

2 years ago
Depends on: 1416350
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Assignee

Updated

2 years ago
Attachment #8926656 - Flags: review?(bugs)
Attachment #8926657 - Flags: review?(mats)
Attachment #8926658 - Flags: review?(mats)

Comment 8

2 years ago
mozreview-review
Comment on attachment 8926657 [details]
Bug 1415670 Part 2: Calculate negativeNumber for each GridLine.

https://reviewboard.mozilla.org/r/197900/#review204670
Attachment #8926657 - Flags: review?(mats) → review+

Comment 9

2 years ago
mozreview-review
Comment on attachment 8926658 [details]
Bug 1415670 Part 3: Extend Grid tests to check negativeNumber property.

https://reviewboard.mozilla.org/r/197902/#review204674

Comment 10

2 years ago
mozreview-review
Comment on attachment 8926658 [details]
Bug 1415670 Part 3: Extend Grid tests to check negativeNumber property.

https://reviewboard.mozilla.org/r/197902/#review204676
Attachment #8926658 - Flags: review?(mats) → review+

Comment 11

2 years ago
mozreview-review
Comment on attachment 8926656 [details]
Bug 1415670 Part 1: Add a negativeNumber property to GridLine in webidl.

https://reviewboard.mozilla.org/r/197898/#review204870

rs+ for
Attachment #8926656 - Flags: review?(bugs) → review+
Reporter

Comment 13

2 years ago
Friendly reminder to land this soon.
Flags: needinfo?(bwerth)
Assignee

Comment 14

2 years ago
(In reply to Gabriel [:gl] (ΦωΦ) from comment #13)
> Friendly reminder to land this soon.

Blocking bugs have cleared. Trying to land it now.
Flags: needinfo?(bwerth)

Comment 15

2 years ago
Pushed by bwerth@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/92d2a5673bbc
Part 1: Add a negativeNumber property to GridLine in webidl. r=smaug
https://hg.mozilla.org/integration/autoland/rev/ec9fa6d9dca9
Part 2: Calculate negativeNumber for each GridLine. r=mats
https://hg.mozilla.org/integration/autoland/rev/812aaf31c577
Part 3: Extend Grid tests to check negativeNumber property. r=mats
Assignee

Comment 17

2 years ago
(In reply to Bogdan Tara[:bogdan_tara] from comment #16)
> Backed out 3 changesets (bug 1415670) for failing chrome test
> dom/grid/test/chrome/test_grid_implicit.html r=backout on a CLOSED TREE
> 
> https://treeherder.mozilla.org/logviewer.
> html#?job_id=149638383&repo=autoland&lineNumber=2633

This is a super-weird result from the try server. The failures are issues that should have been resolved by blocker Bug 
1416350. Probably my patches just need a better rebasing. Trying that first.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Assignee

Comment 21

2 years ago
(In reply to Brad Werth [:bradwerth] from comment #17)
> This is a super-weird result from the try server. The failures are issues
> that should have been resolved by blocker Bug 
> 1416350. Probably my patches just need a better rebasing. Trying that first.

No; there were logic problems in the patch which I have corrected. Attempting to land again. Sorry for the confusion!
Assignee

Updated

2 years ago
See Also: → 1423378

Comment 22

2 years ago
Pushed by bwerth@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f5961caa74ba
Part 1: Add a negativeNumber property to GridLine in webidl. r=smaug
https://hg.mozilla.org/integration/autoland/rev/dbb439bcdc03
Part 2: Calculate negativeNumber for each GridLine. r=mats
https://hg.mozilla.org/integration/autoland/rev/cfc0826e8cba
Part 3: Extend Grid tests to check negativeNumber property. r=mats

Comment 23

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f5961caa74ba
https://hg.mozilla.org/mozilla-central/rev/dbb439bcdc03
https://hg.mozilla.org/mozilla-central/rev/cfc0826e8cba
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.