Closed
Bug 756759
Opened 13 years ago
Closed 13 years ago
Remove gnomestripe's non-native fallback styling for toolbarbuttons
Categories
(Toolkit :: Themes, defect)
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: dao, Assigned: dao)
References
Details
Attachments
(1 file, 1 obsolete file)
7.75 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
The fallback styling shouldn't really be needed on Linux and is annoying to override because of the inflationary use of !important in the [checked="true"] rules.
While I'm at it, I'm also getting rid of Lighten.png.
Attachment #625392 -
Flags: review?(neil)
Assignee | ||
Comment 1•13 years ago
|
||
Oops, I removed too much. The text colors are still needed.
Attachment #625392 -
Attachment is obsolete: true
Attachment #625392 -
Flags: review?(neil)
Attachment #625395 -
Flags: review?(neil)
Comment 2•13 years ago
|
||
Comment on attachment 625395 [details] [diff] [review]
patch v2
>+ background-color: rgba(255,255,255,0.5);
rgba(255, 255, 255, 0.5); unless you or gavin can point to some style guideline that says that you must cram all the values together.
Attachment #625395 -
Flags: review?(neil)
Attachment #625395 -
Flags: review+
Attachment #625395 -
Flags: feedback?(gavin.sharp)
Assignee | ||
Comment 3•13 years ago
|
||
It doesn't matter here, but in places where you combine multiple color values, spaces within these values reduce the overall readability.
e.g.
-moz-linear-gradient(bottom, rgba(10%, 10%, 10%, .4) 1px, transparent 1px)
vs.
-moz-linear-gradient(bottom, rgba(10%,10%,10%,.4) 1px, transparent 1px)
Comment 4•13 years ago
|
||
Comment on attachment 625395 [details] [diff] [review]
patch v2
I don't really feel strongly either way!
Attachment #625395 -
Flags: feedback?(gavin.sharp)
Assignee | ||
Comment 5•13 years ago
|
||
Target Milestone: --- → mozilla15
Comment 6•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
![]() |
||
Comment 7•13 years ago
|
||
> It doesn't matter here, but in places where you combine multiple color values,
> spaces within these values reduce the overall readability.
Actually spaces *increase* readability. Us old fogeys tend not to have 20/20 vision unlike you pesky kids.
Assignee | ||
Comment 8•13 years ago
|
||
(In reply to Philip Chee from comment #7)
> > It doesn't matter here, but in places where you combine multiple color values,
> > spaces within these values reduce the overall readability.
> Actually spaces *increase* readability. Us old fogeys tend not to have 20/20
> vision unlike you pesky kids.
This has nothing to do with vision. It's easier to skim the whole thing when all the spaces clearly delimit the values.
You need to log in
before you can comment on or make changes to this bug.
Description
•