47 bytes, text/x-phabricator-request
|Details | Review|
There are at least a couple of reasons why we would like to do this: - in bug 1493677 we would like to jump from the box-model to the rule-view, to tell users where a given margin, or padding or border was actually defined in the CSS - in bug 1478396 were are considering jumping from the flex item sizing properties list to the rule-view, to show users where a given base size, growth or max clamping were defined in CSS. So, the idea here is to have the rule-view expose an API that allows other panels to do just this. Something like ruleView.highlightProperty(name, value); This would find the first instance of this property (the one that's not overridden), scroll to it, and maybe flash it for a second to draw the user's attention towards it.
This is a feature enhancement that we were hoping to capture in the Flexbox inspector milestone 2 scope, but as we're getting near our deadline for M2 and as this is separate enough, I think we should move it to later. If we do manage to land it in time for 65, fine, but if we don't, no big deal.
No longer blocks: 1478396
Assignee: nobody → mtigley
Status: NEW → ASSIGNED
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/0a4aefb36aff Expose a method from rule view that scrolls to a rule property's name and value. r=gl
You need to log in before you can comment on or make changes to this bug.