Closed
Bug 1013708
Opened 10 years ago
Closed 10 years ago
in-content preferences advanced pane: remove text shadow from sub categories
Categories
(Firefox :: Settings UI, defect)
Tracking
()
VERIFIED
FIXED
Firefox 32
Tracking | Status | |
---|---|---|
firefox32 | --- | verified |
People
(Reporter: soeren.hentzschel, Assigned: Paenglab)
References
(Blocks 1 open bug)
Details
Attachments
(2 files, 4 obsolete files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
Paenglab
:
review+
|
Details | Diff | Splinter Review |
Please remove the text shadow of the in-content preferences advanced panel sub category labels, it difficult to read, see screenshot.
Assignee | ||
Comment 1•10 years ago
|
||
This patch uses the styles from Project Chameleon.
I had to change the pane background-color to white like Chameleon to let the tabs look better. And this needed on Windows to remove the caption background color. On OS X I removed also the line below the captions.
On Project Chameleon where was no spec for the dimensions and I choosed the best fitting ones.
Assignee: nobody → richard.marti
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #8430737 -
Flags: review?(jaws)
Assignee | ||
Comment 2•10 years ago
|
||
Found on Project Chameleon under Navigation - Horizontal inContent Navigation a different tabs background color for not active tabs. And also a updated transition.
Attachment #8430737 -
Attachment is obsolete: true
Attachment #8430737 -
Flags: review?(jaws)
Attachment #8430815 -
Flags: review?(jaws)
Comment 3•10 years ago
|
||
Comment on attachment 8430815 [details] [diff] [review]
horizontalTabs.patch
Review of attachment 8430815 [details] [diff] [review]:
-----------------------------------------------------------------
::: browser/themes/shared/incontentprefs/preferences.css
@@ +102,5 @@
> }
>
> +tab:hover {
> + background-color: #ebebeb;
> +}
nit: trailing whitespace
@@ +108,3 @@
> tab[selected] {
> + background-color: #ebebeb;
> + box-shadow: inset 0 -4px 0 0 #ff9500;
The linux theme adds a few rules for tab[selected] which need to be overriden.
Can you add a rule to the browser/themes/linux/incontentprefs/preferences.css for
tab[selected] {
/* Override the margin-bottom for tab[selected]
from toolkit/themes/linux/global/tabbox.css */
margin-bottom: 0;
border-bottom-left-radius: 0;
border-bottom-right-radius: 0;
}
Attachment #8430815 -
Flags: review?(jaws) → review-
Assignee | ||
Comment 4•10 years ago
|
||
Fixed comments and updated after landing of bug 1013904.
Attachment #8430815 -
Attachment is obsolete: true
Attachment #8431862 -
Flags: review?(jaws)
Assignee | ||
Comment 5•10 years ago
|
||
Found on OS X a unwanted padding on tabs. Removed with this patch.
Attachment #8431862 -
Attachment is obsolete: true
Attachment #8431862 -
Flags: review?(jaws)
Attachment #8432238 -
Flags: review?(jaws)
Comment 6•10 years ago
|
||
Comment on attachment 8432238 [details] [diff] [review]
horizontalTabs.patch
Review of attachment 8432238 [details] [diff] [review]:
-----------------------------------------------------------------
r+ with the following changes.
::: browser/themes/linux/preferences/in-content/preferences.css
@@ +4,5 @@
>
> %include ../../../shared/incontentprefs/preferences.css
>
> +tab[selected] {
> + /* Override the margin-bottom for tab[selected]
This comment should just be made more generic since we are overriding not just the margin-bottom but also the border-radii from toolkit. You can just change it to /* Override styles for tab[selected] from toolkit/themes/linux/global/tabbox.css */
::: browser/themes/shared/incontentprefs/preferences.css
@@ +110,5 @@
> }
>
> tab[selected] {
> + background-color: #ebebeb;
> + box-shadow: inset 0 -4px 0 0 #ff9500;
There is still a text-shadow on OSX with this patch applied. Please add a rule for `text-shadow: none;`.
Attachment #8432238 -
Flags: review?(jaws) → review+
Assignee | ||
Comment 7•10 years ago
|
||
Fixed the comments.
Try: https://tbpl.mozilla.org/?tree=Try&rev=798689fed57d
Attachment #8432238 -
Attachment is obsolete: true
Attachment #8432705 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Comment 9•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 32
Comment 10•10 years ago
|
||
Verified fixed on Mac OSX 10.8.5 using latest Nightly 32.0a1 (buildID: 20140609030202).
Status: RESOLVED → VERIFIED
status-firefox32:
--- → verified
You need to log in
before you can comment on or make changes to this bug.
Description
•