NS_THEME_TREEVIEW* implementations (Mac)

RESOLVED FIXED in mozilla0.9.9

Status

P1
enhancement
RESOLVED FIXED
17 years ago
5 years ago

People

(Reporter: ian, Assigned: mikepinkerton)

Tracking

Trunk
mozilla0.9.9
PowerPC
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

17 years ago
This covers the implementation of NS_THEME_TREEVIEW, NS_THEME_TREEVIEW_TREEITEM,
NS_THEME_TREEVIEW_TWISTY, NS_THEME_TREEVIEW_LINE, NS_THEME_TREEVIEW_HEADER,
NS_THEME_TREEVIEW_HEADER_CELL and NS_THEME_TREEVIEW_HEADER_SORTARROW for XUL
tree views on Mac.
(Reporter)

Updated

17 years ago
Summary: NS_THEME_TREEVIEW* implementation (Mac) → NS_THEME_TREEVIEW* implementations (Mac)
(Assignee)

Comment 1

17 years ago
-> 099
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.9
(Reporter)

Updated

17 years ago
Blocks: 117584

Updated

17 years ago
QA Contact: pmac
(Assignee)

Comment 2

17 years ago
Created attachment 66885 [details] [diff] [review]
trees, tabs, and twisties, oh my!

trees, tabs, twisties.
(Assignee)

Comment 3

17 years ago
needing r/sr here. hyatt? smfr?
Priority: -- → P1
(Assignee)

Comment 4

17 years ago
Created attachment 66904 [details] [diff] [review]
better twisty drawing

fix up twisties so that they have an empty eraseproc and don't blit over the
highlight color of a row.

r/sr needed
Attachment #66885 - Attachment is obsolete: true

Comment 5

17 years ago
Comment on attachment 66904 [details] [diff] [review]
better twisty drawing

r=sdagley
Attachment #66904 - Flags: review+

Comment 6

17 years ago
sr=hyatt

Comment 7

17 years ago
+static pascal void
+DoNothing(const Rect *bounds, UInt32 eraseData, SInt16 depth, Boolean 
isColorDev)

This could be a static method of the class. Minor.

+  nsCAutoString mode;
+  if ( GetAttribute(aFrame, mSortDirectionAtom, mode) )
+    return strlen(mode.get()) > 0;

strlen(mode.get() ? Use !mode.IsEmpty();

+  nsCAutoString mode;
+  if ( GetAttribute(aFrame, mSortDirectionAtom, mode) )
+    return strcmp(mode.get(), "descending") == 0;

mode.Equal()?

+  nsCAutoString mode;
+  if ( GetAttribute(aFrame, mClassAtom, mode) )
+    return strstr(mode.get(), "tab-bottom") != nsnull;

mode.FindMumble?

Deferring to hyatt on the outliner and CSS changes.
(Assignee)

Comment 8

17 years ago
Created attachment 66910 [details] [diff] [review]
address smfr's string comments
Attachment #66904 - Attachment is obsolete: true

Comment 9

17 years ago
Comment on attachment 66910 [details] [diff] [review]
address smfr's string comments

sr=sfraser
Attachment #66910 - Flags: superreview+
(Assignee)

Comment 10

17 years ago
landed.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.