Grid highlighter disappears after editing a property and focusing another one

RESOLVED FIXED in Firefox 52

Status

defect
RESOLVED FIXED
3 years ago
Last year

People

(Reporter: pbro, Assigned: gl)

Tracking

(Blocks 1 bug)

unspecified
Firefox 53
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed, firefox53 fixed)

Details

Attachments

(1 attachment, 2 obsolete attachments)

STR:

- Open http://labs.jensimmons.com/examples/grid-content-1.html
- open the inspector
- find and select the <main> element
- in the rule-view, click on the grid icon next to display:grid;
- click on the "repeat(5, 16vw)" value
- change this value to something else, like "repeat(5, 17vw)"
- click on another value, like the one just below it "repeat(12, 16vw)"

=> The grid highlighter is hidden. It should remain visible.

This is most probably because the whole rule is refreshed and we loose some piece of state in the meantime.

This is related to bug 1302536. Not sure if it should block it, but it seriously impacts the usefulness of this feature.
In fact, in the last step of the STR in the previous comment, you do not need to click inside another value. Just blurring the first value will do it.

Gabriel: can you think of a way to fix this?
Flags: needinfo?(gl)
Assignee

Updated

3 years ago
Flags: needinfo?(gl)
Assignee

Comment 2

3 years ago
Posted patch 1318019.patch (obsolete) — Splinter Review
Assignee: nobody → gl
Attachment #8811566 - Flags: review?(pbrosset)
Assignee

Updated

3 years ago
Blocks: dt-grid
Assignee

Comment 3

3 years ago
Posted patch 1318019.patch [1.0] (obsolete) — Splinter Review
Attachment #8811566 - Attachment is obsolete: true
Attachment #8811566 - Flags: review?(pbrosset)
Attachment #8811573 - Flags: review?(gasolin)
Assignee

Comment 4

3 years ago
Attachment #8811573 - Attachment is obsolete: true
Attachment #8811573 - Flags: review?(gasolin)
Attachment #8811575 - Flags: review+
Assignee

Comment 5

3 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/ef23bf8110953349a23ba2e344c20643b392eda0
Bug 1318019 - Add missing () to call isDisplayGrid function when a property value editor closes. r=me
Assignee

Comment 6

3 years ago
Comment on attachment 8811575 [details] [diff] [review]
1318019.patch [2.0]

Approval Request Comment
[Feature/regressing bug #]: 1302536
[User impact if declined]: Grid highlighter disappears after editing any property values in the rule-view.
[Describe test coverage new/current, TreeHerder]:
[Risks and why]: This is a trivial fix for a function that should've been called.
[String/UUID change made/needed]: None.
Attachment #8811575 - Flags: approval-mozilla-aurora?

Comment 7

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ef23bf811095
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
I've Successfully reproduce this bug in Nightly 53.0a1 (2016-11-16) on windows 10 ,32 bit.


This Bug is now verified as fixed on Latest Nightly 53.0a1 (2016-11-19).

Build ID : 20161119030204.
User Agent : Mozilla/5.0 (Windows NT 10.0; rv:53.0) Gecko/20100101 Firefox/53.0.
Comment on attachment 8811575 [details] [diff] [review]
1318019.patch [2.0]

followup fix for bug 1302536, should land in aurora52 together
Attachment #8811575 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Updated

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