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)

x86
Windows XP
enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla0.9.8

People

(Reporter: ian, Assigned: hyatt)

References

Details

Attachments

(1 file, 1 obsolete file)

This covers the implementation of NS_THEME_TAB and NS_THEME_TAB_PANEL for XUL tab panels on Windows.
Blocks: 40270
Blocks: 40269
Blocks: 117584
Attached patch Patch to add support for tabs. (obsolete) (deleted) — Splinter Review
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).
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.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.8
QA Contact: pmac
Comment on attachment 63620 [details] [diff] [review] Patch to add support for tabs. sr=blake
Attachment #63620 - Flags: superreview+
Comment on attachment 63620 [details] [diff] [review] Patch to add support for tabs. r=bryner
Attachment #63620 - Flags: review+
Fix checked in. Now all I have left is the tab panel.
Attachment #63620 - Attachment is obsolete: true
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 on attachment 63691 [details] [diff] [review] Patch to add support for tabpanels. r=bryner
Attachment #63691 - Flags: review+
Comment on attachment 63691 [details] [diff] [review] Patch to add support for tabpanels. sr=blake
Attachment #63691 - Flags: superreview+
Ok, marking this fixed. Will file separate bugs for the remaining drawing issue.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
david, did this fix cause bug 118471 to be exposed?
Blocks: 185435
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: