Closed
Bug 1300869
Opened 8 years ago
Closed 8 years ago
Dimmed highlighting mode on sometimes wraps the text line of the text being searched for
Categories
(Toolkit :: Find Toolbar, defect)
Toolkit
Find Toolbar
Tracking
()
People
(Reporter: jaws, Assigned: mikedeboer)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
For example:
If the text has the string "positionLabel" and I search for "positionLabel" then I may see:
[posi]_________
tionLabel
The [posi] part is yellow. The underscores is a blank white space, and the "tionLabel" is the rest of the search term but placed directly below the yellow area and without any highlighting or selection.
See attached screenshot. I can reproduce this easily on Firefox Nightly 2016-09-06 on Windows 10 when on MozReview.
Updated•8 years ago
|
Reporter | ||
Comment 2•8 years ago
|
||
Reporter | ||
Updated•8 years ago
|
Summary: Dimmed highlighting mode on Mozreview sometimes wraps the text line of the text being searched for → Dimmed highlighting mode on sometimes wraps the text line of the text being searched for
Updated•8 years ago
|
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → mdeboer
Status: NEW → ASSIGNED
Iteration: --- → 51.3 - Sep 19
Points: --- → 5
Flags: qe-verify+
Flags: firefox-backlog+
Assignee | ||
Updated•8 years ago
|
Points: 5 → 2
Comment hidden (mozreview-request) |
Reporter | ||
Comment 7•8 years ago
|
||
mozreview-review |
Comment on attachment 8790694 [details]
Bug 1300869 - don't allow whitespace wrapping in find toolbar modal highlighting boxes.
https://reviewboard.mozilla.org/r/78394/#review77352
Attachment #8790694 -
Flags: review?(jaws) → review+
Pushed by mdeboer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8f7f5ce39b7d
don't allow whitespace wrapping in find toolbar modal highlighting boxes. r=jaws
Updated•8 years ago
|
tracking-firefox51:
--- → +
Comment 10•8 years ago
|
||
As I just noted over in bug 1282752 comment 12, I suspect the "white-space:nowrap" fix here might be preventing that other bug's fix from being effective.
Perhaps we should back out this bug and come up with a different fix here? Unless I'm misunderstanding, "white-space:nowrap" seems like a pretty arbitrary/severe restriction, since we're allowing for search results that are legitimately wrapped to several lines.
Updated•8 years ago
|
Flags: needinfo?(mdeboer)
Comment 11•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Updated•8 years ago
|
QA Contact: brindusa.tot
Assignee | ||
Comment 12•8 years ago
|
||
I don't think it's a severe restriction, but it might be better to put the style on the 'outlineText' instead of its parent node.
But the meat of your issue, Daniel, is being discussed back in bug 1282752. Thanks for helping out!
Flags: needinfo?(mdeboer)
Comment 13•8 years ago
|
||
Right -- it's not actually as severe as I thought, if we can manually slice up the range and position the pieces independently. Thanks!
Comment 14•8 years ago
|
||
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0
I reproduced the initial issue using Nightly 51.0a1 (Build ID: 20160911030419) on Windows 10 x86_64. Reproduced also all the duplicates issues from bugs: 1301939, 1302022, 1302161, 1302281 and 1302743.
Verified fixed on Windows 10, Ubuntu 16.04 and Mac OS X 10.11 using the latest Nightly 52.0a2 (Build ID: 20160922030437). Also, I can confirm the fixes for all the duplicates of this bug.
status-firefox52:
--- → verified
Comment 15•8 years ago
|
||
Since this is disabled in Firefox 51, shouldn't the tracking flag for Firefox 51 be removed and set to Firefox 52?
Status: RESOLVED → VERIFIED
Target Milestone: mozilla51 → mozilla52
You need to log in
before you can comment on or make changes to this bug.
Description
•