Make toolkit winstripe theme match Windows Classic

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
14 years ago
7 years ago

People

(Reporter: bugzilla-mozilla-20000923, Assigned: bugzilla-mozilla-20000923)

Tracking

(Depends on 2 bugs)

Trunk
x86
Windows Server 2003
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

()

Attachments

(1 attachment)

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.
Depends on: 313391
Status: NEW → ASSIGNED
No longer depends on: 313391
Depends on: 313391
Depends on: 314524
Blocks: 314524
No longer depends on: 314524
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)
Attachment #217713 - Flags: first-review?(neil) → first-review?(kevin)
Blocks: 313674
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?
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
The plan is ok, its trunk only, please make sure stuart knows this is coming
Attachment #217713 - Flags: first-review?(kevin) → first-review+
(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
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*
> (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.
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).
This has caused a regression on the trunk... on XP-luna there's a white line underneath the toolbar. 
No longer blocks: 313674, 336847
Depends on: 313674, 336847
Phase 1 complete. Phase 2 in bug 337771.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Blocks: 348173
No longer blocks: 348173
Depends on: 348173
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
Depends on: 454338
You need to log in before you can comment on or make changes to this bug.