Extend visibility for currently highlighted line
Categories
(DevTools :: Debugger, enhancement, P3)
Tracking
(firefox71 fixed)
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: Harald, Assigned: davidwalsh)
References
(Blocks 1 open bug)
Details
User Story
When jumping to a line in the debugger, I want to clearly see the target line highlighted even in a busy UI like the debugger, so that I can understand which line I jumped to.
Attachments
(2 files)
Follow up from https://github.com/firefox-devtools/debugger/issues/4318
Ideas:
- Extend duration
- Ease out with longer high visible state
- Only start fading out when the UI is fully loaded and/or after the user interacted with the debugger/document
- More vibrant color (Chrome uses yellow)
Reporter | ||
Comment 1•6 years ago
|
||
Reporter | ||
Comment 2•6 years ago
|
||
One option to not have the highlight start when the debugger is still churning on loading (animation ends up choppy and noisy because of ongoing loading): The highlight should be applied as early as possible but the fade out should only be triggered in a requestIdleCallback or an event that the debugger fires when it is fully loaded.
Comment 3•6 years ago
|
||
Agreed on everything! Maybe just not for the first line when first opening a file.
Also: we should try a subtle blue row highlight based on the Inspector hover color.
Other places where this can be applied, off the top of my head: link box model to rules, Markup changed DOM highlight.
Reporter | ||
Comment 4•6 years ago
|
||
Also: we should try a subtle blue row highlight based on the Inspector hover color.
That does not seem very consistent across editors. Style Editor highlights the current line, while Inspector highlights the currently focused element in the tree. To keep this bug focused, it is maybe worth filing a separate issue with a better problem statement.
Related: https://github.com/firefox-devtools/debugger/issues/8007
Updated•6 years ago
|
Assignee | ||
Comment 5•6 years ago
|
||
Victoria: Inconsistencies aside, can you recommend a better (yellow) highlight color and opacity for this line?
Updated•6 years ago
|
Updated•5 years ago
|
Assignee | ||
Comment 6•5 years ago
|
||
Victoria: Inconsistencies aside, can you recommend a better (yellow) highlight color and opacity for this line?
Comment 7•5 years ago
|
||
Hi! There's new yellow flash styling currently in progress for the Markup view: https://bugzilla.mozilla.org/show_bug.cgi?id=1541278
I'm not sure what it looks like yet - was planning to take a closer look after it lands. But the yellow should be 'Yellow 50' at 40% opacity.
I also think all our flash effects should have a longer duration so it would be great if you move forward on that as well.
Assignee | ||
Comment 8•5 years ago
|
||
Comment 10•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Updated•5 years ago
|
Description
•