536904 - Add toggletoolbar="true" to the windows that should have a Mac toolbar toggle pill

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Stefan [:stefanh]]

As philor said in bug #520338:

"After bug 269410, we need to add toggletoolbar="true" to the window element in windows where we want the pill button to toggle the toolbar...."

This is 1.9.2/1.9.3-only.

Comment 1

[Stefan [:stefanh]]

I think we should have this on the radar, so it doesn't get lost.

Comment 2

[Stefan [:stefanh]]

Attachment: Add pill button and remove top border

Re the css: the border is not needed anymore (mstange have fixed it). There are some problems, though - see bug 579250. I'll remove it anyway, since I think things will finally be solved:

<mstange>	roc will probably fix it again, so you might just remove it

Comment 3

[[e-mail]]

Comment on attachment 458093 [details] [diff] [review]
Add pill button and remove top border

You are allowed to add the attribute in the middle of the list on all the windows, thus avoiding editing the existing last line ;-)

Comment 4

[Karsten Düsterloh]

Comment on attachment 458093 [details] [diff] [review]
Add pill button and remove top border

I wonder if the pill button maybe should also toggle the sidebar, like it does in the Finder?

r=me without the change noise Neil mentioned.

Comment 5

[Stefan [:stefanh]]

http://hg.mozilla.org/comm-central/rev/c08135ef6a07 (with fixed noise).

Comment 6

[Stefan [:stefanh]]

(In reply to comment #4)
> 
> I wonder if the pill button maybe should also toggle the sidebar, like it does
> in the Finder?

I think that would be an overkill :-)