Closed Bug 526461 Opened 15 years ago Closed 13 years ago

When I hid menubar in a classic theme, there is an unnecessary highlight between title bar and navigation bar,

Categories

(Core :: Widget: Win32, defect)

1.9.2 Branch
x86
Windows Vista
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 388167

People

(Reporter: alice0775, Unassigned)

References

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2b2pre) Gecko/20091103 Firefox/3.5.1 (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2b2pre) Gecko/20091103 Firefox/3.5.1 (.NET CLR 3.5.30729)

When I hid menubar in a classic theme, there is an unnecessary highlight between title bar and navigation bar,

Reproducible: Always

Steps to Reproduce:
1.Set Windows Visual Style to "Windows Classic"
2.Start Namoroka with new profile
3.Hit alt key to hide menubar
Component: General → Theme
Version: unspecified → 3.6 Branch
Blocks: 477256
Attached image picture
Component: Theme → Themes
Product: Firefox → Toolkit
QA Contact: general → themes
Version: 3.6 Branch → 1.9.2 Branch
Status: UNCONFIRMED → NEW
Ever confirmed: true
There's an XXX comment for this in nsNativeThemeWin.cpp:

702     case NS_THEME_TOOLBAR: {
703       // Use -1 to indicate we don't wish to have the theme background drawn
704       // for this item. We will pass any nessessary information via aState,
705       // and will render the item using separate code.
706       aPart = -1;
707       aState = 0;
708       if (aFrame) {
709         nsIContent* content = aFrame->GetContent();
710         nsIContent* parent = content->GetParent();
711         // XXXzeniko hiding the first toolbar will result in an unwanted margin
712         if (parent && parent->GetChildAt(0) == content) {
713           aState = 1;
714         }
715       }
716       return NS_OK;
717     }
Component: Themes → Widget: Win32
Product: Toolkit → Core
QA Contact: themes → win32
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: