Closed
Bug 115750
Opened 23 years ago
Closed 23 years ago
NS_THEME_TAB and NS_THEME_TAB_PANEL implementations (Windows)
Categories
(Core Graveyard :: Skinability, enhancement)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla0.9.8
People
(Reporter: ian, Assigned: hyatt)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
bryner
:
review+
bugzilla
:
superreview+
|
Details | Diff | Splinter Review |
This covers the implementation of NS_THEME_TAB and NS_THEME_TAB_PANEL for XUL
tab panels on Windows.
Assignee | ||
Comment 1•23 years ago
|
||
This patch adds support for tabs. It doesn't yet cover tabpanels, because I
have to write some custom code to handle not drawing a line under the tab
itself.
A known issue is called out in the bug, namely that I have no way of knowing
how thick the "edges" of tabs are, since all of the skins I tried don't
properly return content margins. This means I'm forced to make an unfortunate
assumption about the thickness of an edge (namely that the edge of a tab is 2
pixels thick).
Assignee | ||
Comment 2•23 years ago
|
||
IMO the skins themselves are buggy here, but since they're all behaving this
way (probably following a bad pattern set by Luna), there isn't much I can do.
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.8
Comment 3•23 years ago
|
||
Comment on attachment 63620 [details] [diff] [review]
Patch to add support for tabs.
sr=blake
Attachment #63620 -
Flags: superreview+
Comment 4•23 years ago
|
||
Comment on attachment 63620 [details] [diff] [review]
Patch to add support for tabs.
r=bryner
Attachment #63620 -
Flags: review+
Assignee | ||
Comment 5•23 years ago
|
||
Fix checked in. Now all I have left is the tab panel.
Assignee | ||
Updated•23 years ago
|
Attachment #63620 -
Attachment is obsolete: true
Assignee | ||
Comment 6•23 years ago
|
||
Tabpanels support. Known issues: handling not drawing the line under the
selected tab, line needs to overlap tabs by one pixel in order to be
pixel-perfect, and finally no way to specify a tabpanel (which isn't really a
big deal, the style builder doesn't even show you this as a separate entity in
the widget display).
Comment 7•23 years ago
|
||
Comment on attachment 63691 [details] [diff] [review]
Patch to add support for tabpanels.
r=bryner
Attachment #63691 -
Flags: review+
Comment 8•23 years ago
|
||
Comment on attachment 63691 [details] [diff] [review]
Patch to add support for tabpanels.
sr=blake
Attachment #63691 -
Flags: superreview+
Assignee | ||
Comment 9•23 years ago
|
||
Ok, marking this fixed. Will file separate bugs for the remaining drawing issue.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 10•23 years ago
|
||
david,
did this fix cause bug 118471 to be exposed?
Updated•16 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•