Closed
Bug 1384933
Opened 7 years ago
Closed 7 years ago
Can't iterate through search results after switching to a different file
Categories
(DevTools :: Debugger, defect)
Tracking
(firefox54 unaffected, firefox55 unaffected, firefox56 wontfix, firefox57 verified)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox54 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | --- | wontfix |
firefox57 | --- | verified |
People
(Reporter: hyacoub, Unassigned)
References
()
Details
[Affected versions]: Nightly 56.0a1 [Affected platforms]: Platforms: Windows 10 x 64, Ubuntu 16.04 and Mac OS X 10.12 [Steps to reproduce]: 1. Go to https://devtools-html.github.io/debugger-examples/examples/todomvc/ 2. Open the Debugger. 3. From "Source list pane" section, select a file that has the ".js" termination. 4. On Windows - Press CTRL + F On Mac - Press CMD + F Search for a term that can be found (e.g listenTo). 5. Navigate to another file (e.g app-view.js). 6. Press Enter to iterate forward or press Shift+Enter to iterate back. [Expected result]: Can iterate through search results after switching to a different file. [Actual result]: Can't iterate through search results for a different file.
Reporter | ||
Updated•7 years ago
|
Blocks: 1294139
status-firefox54:
--- → unaffected
status-firefox55:
--- → unaffected
status-firefox56:
--- → affected
Comment 1•7 years ago
|
||
added to GH: https://github.com/devtools-html/debugger.html/issues/3475
Updated•7 years ago
|
Comment 2•7 years ago
|
||
This has been fixed in GitHub, not sure if this landed in Nightly yet. :jason, did that fix land in Nightly? And did that land make the 56 train?
Flags: needinfo?(jlaster)
Comment 3•7 years ago
|
||
Yep, this is now fixed.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(jlaster)
Resolution: --- → FIXED
Comment 4•7 years ago
|
||
This issue is verified fixed in Firefox 57.0a1 (Build Id:20170905220108). Unfortunately the land did not made the 56 train since this is still reproducible in Firefox 56.0b9 (BuildId:20170903140023).
status-firefox57:
--- → verified
Comment 5•7 years ago
|
||
Too late for 56. Mass won't fix for 56.
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•