Open Bug 1294946 Opened 8 years ago Updated 2 years ago

Inspector does not focus the result two of three nodes in markup if using search string "identity-box"

Categories

(DevTools :: Inspector, defect, P3)

defect

Tracking

(firefox48 unaffected, firefox49 affected, firefox50 affected, firefox51 affected)

Tracking Status
firefox48 --- unaffected
firefox49 --- affected
firefox50 --- affected
firefox51 --- affected

People

(Reporter: magicp.jp, Unassigned)

References

Details

Attachments

(1 file)

Attached image invisible-node.png (deleted) —
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0 Build ID: 20160812030200 Steps to reproduce: 1. Start Nightly 2. Enable remote debugging 3. Open Browser Toolbox (Ctrl+Alt+Shift+I) 4. Type "identity-box" in inspector-searchbox, then hit [Enter] key (inspector-searchlabel is 1 of 3) 5. Check focusing node in markup 6. Hit [Enter] key again in inspector-searchbox Actual results: After step 6, Inspector does not focus 2 of 3 and 3 of 3 node. After using "Expand All", invisible node appears. (Please find attached file) Regression range: https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=e62b981d0cbe620420b0c6fb7af38a07be913afb&tochange=6191f8b4e99d5096c0060113ad71d6c4c5b7116b Expected results: Should focus search result in markup, and display all of nodes.
Blocks: 1264907
Has Regression Range: --- → yes
Has STR: --- → yes
Component: Untriaged → Developer Tools: Inspector
OS: Unspecified → All
Hardware: Unspecified → All
Flags: qe-verify?
Whiteboard: [devtools-html] [triage]
Hi Patrick, would like to confirm with you if this bug should belong to the Track #2 team. Thanks.
Flags: needinfo?(pbrosset)
It's apparently a regression from bug 1264907 which had been added to the track 2 backlog only because it had to be done before the breadcrumbs could be rewritten as HTML. I don't think it should have been added to the project in the first place, but maybe that helped planning. In any case, I don't think this one should be added to the track 2 backlog. It doesn't have anything in common with the de-XUL-ification project.
Flags: needinfo?(pbrosset)
(In reply to Patrick Brosset <:pbro> from comment #2) > It's apparently a regression from bug 1264907 which had been added to the > track 2 backlog only because it had to be done before the breadcrumbs could > be rewritten as HTML. > I don't think it should have been added to the project in the first place, > but maybe that helped planning. > In any case, I don't think this one should be added to the track 2 backlog. > It doesn't have anything in common with the de-XUL-ification project. Thanks Patrick.
No longer blocks: devtools-html-2
Flags: qe-verify?
Whiteboard: [devtools-html] [triage]
Triaging as P3 based on the fact that this seems browser-toolbox related only. If it occurs in the "normal" tab toolbox too, then we should re-evaluate.
Priority: -- → P3
Product: Firefox → DevTools
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: