Created attachment 459360 [details] [diff] [review] v1 The changes in bug 573326 don't work for the new bookmarks button, since it doesn't expect a toolbaritem around the button.
Attachment #459360 - Flags: review?(dao)
Comment on attachment 459360 [details] [diff] [review] v1 This seems to be too much tailored to browser internals. I think we should: 1. Hide all .toolbarbutton-text, regardless of the widget structure. 2. Set the title attribute on the bookmarks menu and the back/forward widgets.
Attachment #459360 - Flags: review?(dao) → review-
Should I put title="&backCmd.label;/&forwardCmd.label;" on the back/forward toolbaritem, or should I create a new entity?
You should create a new entity.
Created attachment 462400 [details] [diff] [review] v2
Attachment #462400 - Flags: approval2.0? → approval2.0+
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b4
Now that we have an improved bookmarks button, shouldn't the old one be removed? It's basically become useless. The only advantage it has compared to the new one is that it only takes one click. This could be given to the new one by middle clicking it. And it's kind of confusing having two "bookmarks" in the toolbar customization window.
I don't know. Please file a new bug for that and CC Marco and Stephen.
bug 584669 Don't know which Marco to CC
You need to log in before you can comment on or make changes to this bug.