Closed
Bug 313388
Opened 20 years ago
Closed 19 years ago
Make toolkit winstripe theme match Windows Classic
Categories
(Toolkit :: UI Widgets, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bugzilla-mozilla-20000923, Assigned: bugzilla-mozilla-20000923)
References
(Depends on 2 open bugs, )
Details
Attachments
(1 file)
14.78 KB,
patch
|
kevin
:
first-review+
|
Details | Diff | Splinter Review |
This bug is a sub-bug of bug 243078, for dealing with making the Toolkit
'winstripe' theme (menu, toolbar and popups only) match the Windows Classic
appearance when using pure CSS. This is the fallback position for when no
theming is available.
BTW, this component sucks, I want a Toolkit: Themes component. :-)
Please do NOT comment on this bug unless you have a technical contribution, or
it will become impossible to work on this bug.
Updated•19 years ago
|
Assignee | ||
Comment 1•19 years ago
|
||
This patch turns off the native code that was checked-in against my wishes some time ago, and corrects the CSS to visually match the Windows XP Classic appearance.
Just to pre-empt one question: the EM-based values are used where the spacing is based on the font-size. This means that the sizes of various gaps continue to match with native menus when using large fonts (or simply different font sizes).
Picking randomly for first-review flag.
Attachment #217713 -
Flags: first-review?(neil)
Assignee | ||
Updated•19 years ago
|
Attachment #217713 -
Flags: first-review?(neil) → first-review?(kevin)
Comment 2•19 years ago
|
||
After applying your patch on the trunk, I see that classic menu styles are applied even when the Luna theme is selected. James, I'm having trouble following the story here. Please bear with me. Is this part of a larger patch that you're working on to get the correct appearance on both Luna and Classic, or do you intend this patch to go in by itself?
Assignee | ||
Comment 3•19 years ago
|
||
This is phase 1, as I've stated time and again. This patch needs to go in on its own so regressions with it can be found before I do the native theming. See:
http://twpol.dyndns.org/weblog/2005/11/01/01
Comment 4•19 years ago
|
||
The plan is ok, its trunk only, please make sure stuart knows this is coming
Updated•19 years ago
|
Attachment #217713 -
Flags: first-review?(kevin) → first-review+
Assignee | ||
Comment 5•19 years ago
|
||
(In reply to comment #4)
> The plan is ok, its trunk only, please make sure stuart knows this is coming
Who exactly and why?
And what other review shenanigans are needed?
(In reply to comment #4)
> The plan is ok, its trunk only, please make sure stuart knows this is coming
Checkin has been approved. Can someone please check this in so we can get this moving?
~B
Assignee | ||
Comment 7•19 years ago
|
||
I'm waiting for someone to answer the question about reviews, so I know if I *can* check it in or not. I'm not sure it's worth my effort if people don't answer critical (and supposedly easy) questions, though. *sigh*
Comment 8•19 years ago
|
||
> (In reply to comment #4)
> > The plan is ok, its trunk only, please make sure stuart knows this is coming
> Who exactly and why?
That would be Stuart Parmenter, which is one of the guys working on the Cairo graphics backend. I'm not sure how this would affect him, but I've cc'ed him, so now he knows :-)
> And what other review shenanigans are needed?
Kevin has reviewed the patch and mconnor says the plan is ok. That's enough.
Assignee | ||
Comment 9•19 years ago
|
||
Ok, this has landed on trunk.
Any regressions or issues that show up as a result should be filed as "depends on" bugs to keep tracking sane.
I'm going to leave this bug open /for the moment/ as I've not dealt with bug 313391 yet (not sure what the status of it is with Places, either).
Comment 10•19 years ago
|
||
This has caused a regression on the trunk... on XP-luna there's a white line underneath the toolbar.
Assignee | ||
Updated•19 years ago
|
Assignee | ||
Comment 11•19 years ago
|
||
Phase 1 complete. Phase 2 in bug 337771.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•19 years ago
|
Comment 12•15 years ago
|
||
Comment on attachment 217713 [details] [diff] [review]
Phase 1 - XP Classic using only CSS
>- background-color: Menu;
> min-width: 1px;
>+ background: Menu;
Why this change?
Attachment #217713 -
Attachment description: Phase 1 - XP Classic using only CSS → Phase 1 - XP Classic using only CSS
You need to log in
before you can comment on or make changes to this bug.
Description
•