318053 - Overflow for Bookmarks Toolbar

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, defect)

Description

[Ben Goodger (use ben at mozilla dot org for email)]

Need to overflow into a chevron menu, preserving D&D capabilities into that overflow menu.

Comment 1

[Annie Sullivan]

Attachment: First pass at chevron menu

Comment 2

[Ben Goodger (use ben at mozilla dot org for email)]

Comment on attachment 204661 [details] [diff] [review]
First pass at chevron menu

>+          popup.popupShowingCallback = function() {t.chevronPopupShowing();};
>+          this._chevron.appendChild(popup);

Can't you replace this popupShowingCallback by doing

popup.addEventListener("popupshowing", function(e) { t.chevronPopupShowing(e.target); }, false);

... without requiring a change to menu.xml? (e.target is the popup element)

>+      <method name="getChevronWidth">

You call this function getChevronWidth...

>+          // hack -- collapse all children so that we can get the width of the hbox
>+          // if the buttons didn't stretch it past the edge of the window.
>+          // This still seems to return a minimum of 317 pixels.

Can't you also set overflow on the box in CSS to hidden? That will ensure the children are not used when determining the actual width of the box, but when the preferred width of the box is set to 0 and the flex set to 1 will ensure that clipping never actually happens. 

>+          var chevronWidth = this._chevron.boxObject.width;

... (from above) but then you don't use getChevronWidth to measure the chevron's width ...

>+            spaceLeft -= this.getChevronWidth(child);

... you use it to measure the other kids. Why not call the fn getElementWidth? or getButtonWidth?

One more rev to investigate the questions I made above about addEventListener/overflow.

Comment 3

[Annie Sullivan]

Attachment: Second pass at chevron menu

The event listener I tried to add for popupShowing gets called before the handler in menu.xml fills in the popup menu, so as far as I can tell the callback is necessary.  I set the overflow style to hidden and got rid of my hack, and changed getChevronWidth() to getElementWidth().

Comment 4

[Ben Goodger (use ben at mozilla dot org for email)]

Comment on attachment 204716 [details] [diff] [review]
Second pass at chevron menu

r=[e-mail]

Annie explained the callback needs to execute after popupshowing but before the popup has actually shown.

Comment 5

[Gervase Markham [:gerv]]

Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv