Closed Bug 756759 Opened 13 years ago Closed 13 years ago

Remove gnomestripe's non-native fallback styling for toolbarbuttons

Categories

(Toolkit :: Themes, defect)

All
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla15

People

(Reporter: dao, Assigned: dao)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) — 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)
Attached patch patch v2Splinter Review
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 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)
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 on attachment 625395 [details] [diff] [review] patch v2 I don't really feel strongly either way!
Attachment #625395 - Flags: feedback?(gavin.sharp)
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
> 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.
(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.
Blocks: 1341048
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: