Toolbars outside of the toolbox should be visible while customizing, just like those inside of the toolbox

VERIFIED FIXED in mozilla2.0b8

Status

()

VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: dao, Assigned: dao)

Tracking

Trunk
mozilla2.0b8
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

8 years ago
Created attachment 490880 [details] [diff] [review]
patch

This was fixed in bug 599325 for Firefox, but toolkit code should handle this.
Attachment #490880 - Flags: review?(enndeakin)

Updated

8 years ago
Attachment #490880 - Flags: review?(enndeakin) → review+
(Assignee)

Updated

8 years ago
Attachment #490880 - Flags: approval2.0?
Comment on attachment 490880 [details] [diff] [review]
patch

a+. thanks for whipping up the right fix for this!
Attachment #490880 - Flags: approval2.0? → approval2.0+
Comment on attachment 490880 [details] [diff] [review]
patch

Hrm, actually, there's no test for this. Is there already one in the tree? If so it should be modified for add-on bar.
Attachment #490880 - Flags: approval2.0+ → approval2.0-
(Assignee)

Comment 3

8 years ago
Comment on attachment 490880 [details] [diff] [review]
patch

I don't think this needs a test.
Attachment #490880 - Flags: approval2.0- → approval2.0?
Why not just leave the existing test, but have it use computed style rather than checking .collapsed ?
(Assignee)

Comment 5

8 years ago
The test makes me nervous, I'd rather get rid of it. I don't understand why it opens a "chromeless window", which seems entirely non-standard.
(Assignee)

Comment 6

8 years ago
Created attachment 492276 [details] [diff] [review]
patch

cleaned up the test
Attachment #490880 - Attachment is obsolete: true
Attachment #492276 - Flags: approval2.0?
Attachment #490880 - Flags: approval2.0?
Attachment #492276 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 7

8 years ago
http://hg.mozilla.org/mozilla-central/rev/6c4cca9bd105
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Verified fixed with Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b8pre) Gecko/20101204 Firefox/4.0b8pre ID:20101204030328
Status: RESOLVED → VERIFIED
Flags: in-testsuite+
Flags: in-litmus-
You need to log in before you can comment on or make changes to this bug.