Closed
Bug 1113639
Opened 10 years ago
Closed 10 years ago
Search engines table header in preferences is highlighted
Categories
(Firefox :: Search, defect)
Firefox
Search
Tracking
()
RESOLVED
FIXED
Firefox 38
People
(Reporter: phorea, Assigned: abdelrahman, Mentored)
References
Details
Attachments
(1 file)
|
4.14 KB,
patch
|
florian
:
review+
|
Details | Diff | Splinter Review |
Reproduced using Nightly 37.0a1 2014-12-18 under Mac OSX 10.9.5, Ubuntu 14.04 64-bit and Win 7 64-bit.
Steps to reproduce:
1. Open Preferences/Search in both in-content and window mode
2. Hover over the table's header
3. Try to click the table's columns
Expected results:
No highlight on table header if there's no action on the columns.
Actual results:
In-content preferences highlights the header on all platforms; on Ubuntu the columns can be clicked too.
Preferences/Search in separate window allows the columns to be clicked (all platforms) - such as they were buttons. Under Ubuntu and Windows, the text is moved 1px when this happens.
Comment 1•10 years ago
|
||
The CSS rule that causes the color of the header to change on hover is: http://hg.mozilla.org/mozilla-central/annotate/3d846527576f/toolkit/themes/shared/in-content/common.inc.css#l644
I think we could add an attribute on the treecol elements to indicate that no sorting is possible, and tweak that CSS selector.
Mentor: florian
| Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8554052 -
Flags: review?(florian)
Comment 3•10 years ago
|
||
Comment on attachment 8554052 [details] [diff] [review]
rev 1 - Stop highlighting search engines table header
Review of attachment 8554052 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good to me, thanks!
Attachment #8554052 -
Flags: review?(florian) → review+
Updated•10 years ago
|
Assignee: nobody → a.ahmed1026
Comment 4•10 years ago
|
||
Whiteboard: [fixed-in-fx-team]
Comment 5•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 38
Updated•10 years ago
|
QA Whiteboard: [good first verify]
Comment 6•10 years ago
|
||
Its fixed in my browser Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Firefox/38.0 , it does highlight
You need to log in
before you can comment on or make changes to this bug.
Description
•