Open Bug 1302387 Opened 8 years ago Updated 2 years ago

Find highlighting doesn't reflect page changes

Categories

(Toolkit :: Find Toolbar, defect)

defect

Tracking

()

Tracking Status
firefox48 --- unaffected
firefox49 --- unaffected
firefox50 --- unaffected
firefox51 --- disabled
firefox52 --- disabled
firefox53 --- disabled
firefox54 --- disabled
firefox55 --- disabled

People

(Reporter: bj, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: regression)

Attachments

(1 file)

Attached image Linux screen shot
Steps to reproduce:
1) Add some items to an eBay watch list.
2) View the watch list
3) Use Find in Page to search for "e" (using Highlight All).
4) Click "Add Note" for an item.

Expected:
* The visible letters are highlighted.
* No visible highlighting for non-visible letters.

Actual:

3) On Windows 7 this hangs Firefox. This may be bug 1302011. If not a new ticket should be created.

4) On Linux, letters under the Add Note box are highlighted while not visible, and the "e" in "Add Note" isn't highlighted.

See the attached screen shot near the middle.
Blocks: 1291278
Keywords: regression
This feature will be disabled in 51.
Depends on: 1302470
B.J., this behavior should be much improved in the most recent Nightly (currently at version 58). Can you confirm this for me? Thanks!
Flags: needinfo?(bj)
Yes, thank you Mike. The page is much better.

The problem with highlights appearing for non-visible text is fixed.

The page still doesn't highlight the search target for newly displayed text. I believe this is an old bug and not from the search changes in the past year, but it regularly annoys me on dynamic pages. (It would be fine to close this bug and document that issue in another bug. I'd be annoyed but accept it if the problem was closed as "functions as designed".)
Flags: needinfo?(bj)
Thanks for getting back to me, B.J., and I'm glad this is at least better.
The bug where newly displayed text is not included in the search result is indeed quite old and I'm continuously looking for improvements in that area. It's complex (otherwise it would've been fixed ages ago ;-) ), but not impossible.
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: