Open Bug 1311742 Opened 8 years ago Updated 2 years ago

With dimmed highlighting, spaces are shifted around in the Find-In-Page highlighted result, at MozReview

Categories

(Toolkit :: Find Toolbar, defect)

defect

Tracking

()

People

(Reporter: dholbert, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

STR:
 1. Visit https://reviewboard.mozilla.org/r/86642/diff/3/
 2. Do Find-in-Page (Ctrl+F) and paste this into the findbar:
NS_PRECONDITION(mEnd == aEnd.mEnd

 Notice how the highlighted match looks.

 3. Hit "esc" to cancel the findbar-highlighting, and look to see what the result *actually* looks like in the page. Compare that to the findbar's highlighted rendering.

EXPECTED RESULTS:
 The region should look the same after step 2 and step 3 (modulo highlight color).

ACTUAL RESULTS:
The findbar-highlighted result looks like:
> NS_PRECONDITION(  mEnd== aEnd.m End
...vs. the actual text in the page looks like:
>     NS_PRECONDITION(mEnd == aEnd.mEnd
Attachment #8802984 - Attachment description: screenshot with find-in-page highlighting → screenshot with find-in-page highlighting (BAD)
Attachment #8802986 - Attachment description: screenshot after canceling out of find-in-page highlighting → screenshot after canceling out of find-in-page highlighting (GOOD)
For reference, here's a screenshot taken in a fresh profile. It's still bad, but not quite as bad as attachment 8802984 [details] (only one space has been shifted around, instead of several).

Not sure why I'm getting different-levels-of-bad behavior in two different profiles. *shrug*  Just posting for completeness, anyway.

I'm using "Nightly 52.0a1 (2016-10-17) (64-bit)" for all of these screenshots, BTW.
One more tangential comment:

(In reply to Daniel Holbert [:dholbert] from comment #0)
>  1. Visit https://reviewboard.mozilla.org/r/86642/diff/3/
>  2. Do Find-in-Page (Ctrl+F) and paste this into the findbar:
> NS_PRECONDITION(mEnd == aEnd.mEnd

Note that there are 2 results for that search term in the page. Both of them exhibit the same behavior.)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: