Open Bug 301060 Opened 20 years ago Updated 2 years ago

Selecting a treerow/twisty in a tree causes all twisties to highlight

Categories

(Core :: Widget: Cocoa, defect)

All
macOS
defect

Tracking

()

People

(Reporter: stefanh, Unassigned)

Details

(Keywords: regression, testcase)

Attachments

(2 files)

It seems that the way twisties behave have changed since the 1.8a2 release. If you select a treerow/click on a single twisty all the twisties in the tree will behave like they've been selected (they will be highlighted). This behaviour can be seen on both Firefox and SeaMonkey (classic theme) recent nightlies. However, the 1.8a2 release of seamonkey and Firefox 1.0.4 doesn't have this behaviour. A more narrowed-down regression range: Occurs in a seamonkey cvs build from 2004-07-15 18:00 PDT Does not occur in the 1.8a2 release, build ID 2004071408
Attached file testcase
To reproduce the issue, just load the testcase and select a treerow. In SeaMonkey, the bug can be seen (classic theme) on any tree in the ui (Help, Bookmarks manager etc) as well.
Keywords: testcase
Assignee: joshmoz → nobody
tescase WFM Build identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a9pre) Gecko/2007102203 SeaMonkey/2.0a1pre
(In reply to comment #2) > tescase WFM > Build identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a9pre) > Gecko/2007102203 SeaMonkey/2.0a1pre See the component, this is a mac bug :-)
Assignee: nobody → joshmoz
Component: Widget: Mac → Widget: Cocoa
QA Contact: mac → cocoa
Hardware: Macintosh → All
Attached patch fix v0.5Splinter Review
The problem here is that when we draw a twisty in our native theme code we get active state from the frame, which is for the whole tree - not per-twisty. We probably need to have different types of widget backgrounds for the active state, this patch goes down that road but it doesn't actually work. This patch needs to be fixed so that it works and then we need to make sure Linux is doing the same thing since they also draw twisties with nsITheme. Note that the twisties paint in "nsTreeBodyFrame::PaintTwisty".
Assignee: joshmoz → nobody
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: