Closed
Bug 1279707
Opened 9 years ago
Closed 8 years ago
the highlight will be accumulated and accumulated...
Categories
(Toolkit :: Find Toolbar, defect)
Tracking
()
VERIFIED
FIXED
mozilla51
Tracking | Status | |
---|---|---|
firefox49 | --- | unaffected |
firefox50 | --- | unaffected |
firefox51 | --- | verified |
People
(Reporter: alice0775, Assigned: mikedeboer)
References
(Blocks 1 open bug)
Details
(Keywords: regression)
Attachments
(2 files)
Steps To Reproduce:
1. Open a page (e.g. https://developer.mozilla.org/en-US/Add-ons/Code_snippets/Tabbed_browser )
2. Ctrl+F to open FindBar and input word ("browser") and hit ENTER
3. Click a link (e.g. "load listener" )
4. Select textfield of Findbar and Hit ENTER
Actual Results:
the new Find highlight is wrong
Comment 1•9 years ago
|
||
[Tracking Requested - why for this release]: Regression
Comment 2•9 years ago
|
||
Feature has been backed out so not tracking these bugs for 50.
tracking-firefox50:
? → ---
Comment 3•8 years ago
|
||
Bulk update to find bar bugs that won't be in 50 (according to https://bugzilla.mozilla.org/show_bug.cgi?id=1279695#c4 and local testing).
Assignee | ||
Comment 4•8 years ago
|
||
This should be working as expected now. If not, please re-open this bug. Thanks!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Verified this on Nightly 51.0a1, build ID:20160914030200, on Windows 10 x64 and Windows 7 x64 and I could not reproduce it with the term searched in the Description of the bug, but the issue is reproducible when searching for " to".
Reopening the bug.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → mdeboer
Status: REOPENED → ASSIGNED
Flags: qe-verify+
Flags: firefox-backlog+
Comment hidden (mozreview-request) |
Comment 7•8 years ago
|
||
mozreview-review |
Comment on attachment 8791627 [details]
Bug 1279707 - when the location changes, we need to be more thorough in clearing the state variables than we are now to make sure we start fresh in new documents.
https://reviewboard.mozilla.org/r/79006/#review77618
Attachment #8791627 -
Flags: review?(jaws) → review+
Pushed by mdeboer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f7acbdb1d9a6
when the location changes, we need to be more thorough in clearing the state variables than we are now to make sure we start fresh in new documents. r=jaws
Comment 9•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago → 8 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Comment 10•8 years ago
|
||
Verified as fixed (using the steps from the description) using the latest Nightly 52.0a1 Build ID:20160928030201 on Windows 10 X64 and Ubuntu 14.04.
I can still see some blank empty spaces if searching for "to" but that should be covered in bug 1302470.
Considering the above, setting the status to verified fixed.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•