Closed
Bug 489732
Opened 16 years ago
Closed 16 years ago
Add start of no-native-theme theme
Categories
(Toolkit :: Themes, defect)
Toolkit
Themes
Tracking
()
RESOLVED
FIXED
People
(Reporter: vlad, Assigned: vlad)
Details
Attachments
(1 file)
55.08 KB,
patch
|
vlad
:
review+
|
Details | Diff | Splinter Review |
This patch adds a "faststripe" theme that can be enabled via --enable-faststripe. It's built on top of winstripe, but gets rid of all native theme and all ugly moz-border-color and other pieces. It's still rough, but can be useful for places where we don't have or don't want to use native theme.
Attachment #374210 -
Flags: review+
Assignee | ||
Comment 1•16 years ago
|
||
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 2•16 years ago
|
||
(In reply to comment #0)
> It's built on top of winstripe, but gets rid of all native theme
Then why do you have so many -moz-appearance rules?
Assignee | ||
Comment 3•16 years ago
|
||
Because it's a work-in-progress, and things are being removed as I run into them.
Can I ask what the point of this is? "Add start of no-native-theme theme" isn't a very useful summary.
Assignee | ||
Comment 5•16 years ago
|
||
It's a theme that doesn't rely on -moz-appearance or, ideally, any -moz-* attributes that will ask the system to render anything. The target is for platforms where there either is no good native theme API, where it's too slow to use, or where the native theme i just too ugly to even bother trying to use.
Comment 6•16 years ago
|
||
Note, there are many of such themes on AMO.
Particular examples are LittleFox, Nautipolis, W3v8, Walnut, Walnut2, etc...
I always didn't like -moz-appearance, and always wanted to have themes that look the same on ALL platforms...
Comment 7•15 years ago
|
||
Why did you add an actual configure option for this? Seems like the kind of thing you'd just want to toggle per-app or per-platform or something.
Also, not to be a pedant, but did you really check this in with r=yourself?
Assignee | ||
Comment 8•15 years ago
|
||
It's not necessarily per app/per platform, since you might not want native theme on some platforms that could do it, etc. Ideally we could have it be some more elaborate mechanism (runtime perhaps?), but that wasn't necessary for solving the actual problem. And yeah; there was some time pressure, and given that any of the interesting bits weren't going to be part of any current build, I just dumped it in.
Comment 9•15 years ago
|
||
I'm just currently trying to fight our accumulation of configure options, most of which are useless for 99% of people, is all. (bug 513924)
Updated•14 years ago
|
Assignee: nobody → vladimir
You need to log in
before you can comment on or make changes to this bug.
Description
•