Open Bug 1326809 Opened 7 years ago Updated 2 years ago

Inspector highlights random area and scrolls markup up by 1 screen if I inspect element outside of iframe

Categories

(DevTools :: Inspector, defect, P3)

Product:
DevTools
Component:
Inspector
Type:
defect

Tracking

(firefox50 affected, firefox51 affected, firefox52 wontfix, firefox53 unaffected)

Tracking Flags:
Tracking Status
firefox50 --- affected
firefox51 --- affected
firefox52 --- wontfix
firefox53 --- unaffected

People

(Reporter: arni2033, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [inspector-vs-scrolling-up]) 

>>>   My Info:   Win7_64, Nightly 49, 32bit, ID 20160526082509
STR_1:
1. Open url [1]
2. Open devtools -> inspector, switch to iframe
3. Scroll inspector to the end
4. Right-click on the button, click "Inspect element"

AR:  All content in inspector scrolls up by 1 screen. A random area in iframe is briefly highlighted
ER:  X and Y and probably Z
 X) Markup shouldn't scroll up. I filed several bugs about that, I hope it's enough for developers
    to even consider creating a reliable mechanism that would prevent that scrolling in all cases.
 Y) There should be no highlight saying "button", or it should highlight the actual button.
 Z) Some indication in UI explaining that element is outside of iframe (sometimes it's not obvious)


> [1] data:text/html,<button>Firefox regressions since version 28</button><iframe src='data:text/html,<body><script>var S="<div></div>";for(i=0;i<9;i++)S+=S;document.body.innerHTML=S;</script>'></iframe>
> [2] data:text/html,<button>Firefox regressions since version 28</button><iframe></iframe>
No longer blocks: 1277113
Component: Untriaged → Developer Tools: Animation Inspector
Component: Developer Tools: Animation Inspector → Developer Tools: Inspector
Not a recent regression. Already fixed on Nightly, potentially by Bug 1154645 ? 

Should be verified again after Bug 1326753 gets fixed.
Blocks: 1326753
status-firefox50: --- → affected
status-firefox51: --- → affected
status-firefox52: --- → affected
status-firefox53: --- → unaffected
Priority: -- → P3
(In reply to Julian Descottes [:jdescottes] from comment #1)
> Not a recent regression. Already fixed on Nightly, potentially by Bug
> 1154645 ? 

X) and Y) are fixed, i.e. the node view is not scrolled up anymore and the ancestor path is cleared. For Z), i.e. improving the UX, I've created bug 1332254.

Sebastian
Mass wontfix for bugs affecting firefox 52.
status-firefox52: affected → wontfix
Product: Firefox → DevTools
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.