Closed
Bug 581238
Opened 14 years ago
Closed 12 years ago
Bookmarks button changes moves and changes form in Customize Toolbars mode (depending on Bookmarks toolbar and menu bar)
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jruderman, Assigned: mak)
References
Details
(Whiteboard: [places-next-wanted][fixed in bug 748894])
Bug 574511 made the "bookmarks button" customizable, but its tendency to move around makes it continue to seem like it cannot be removed.
Steps to reproduce:
1. Make a new Firefox profile (mkdir tp; firefox -profile tp)
2. Notice that the bookmarks toolbar has a "Bookmarks βΎ" at the right.
3. Go into "Customize toolbars" mode.
4. Notice that "Bookmarks βΎ" is gone; the entire bookmarks toolbar has turned into "Bookmarks Toolbar Items".
To actually remove it, you have to realize that the very different-looking "Star" button, which appears on a *different* toolbar, is actually the one you want to remove while in customize-toolbars mode. If I hadn't seen bug 574511 in the list of recent checkins, I would not have figured this out.
Assignee | ||
Comment 1•14 years ago
|
||
well, when you open customize the button moves, and it has the same icon.
The alternative would be to hide it and then you would be really unable to remove it.
Assignee | ||
Comment 2•14 years ago
|
||
notice we don't allow users to remove the button, but its container, and the button moves from that container (That can be re-positioned) to the bookmarks (wherever they are) if they are visible.
Reporter | ||
Comment 3•14 years ago
|
||
> well, when you open customize the button moves
So don't do that? It's really confusing.
Keywords: ux-implementation-level
Comment 4•14 years ago
|
||
What button moves where? Both Bookmarks Button and Star Button stays on their place when I open Cuztomize window. (And I'm not sure what do you mean in the description.)
Assignee | ||
Comment 5•14 years ago
|
||
(In reply to comment #4)
> What button moves where? Both Bookmarks Button and Star Button stays on their
> place when I open Cuztomize window. (And I'm not sure what do you mean in the
> description.)
it happens only if bookmarks toolbar is visible
(In reply to comment #3)
> > well, when you open customize the button moves
>
> So don't do that? It's really confusing.
There is not any not-hackish way to do that. I mean, the natural position of the button is on navigationbar, when we move to the bookmarks (notice it is not moved to bookmarks toolbar, but to bookmarks inside bookmarks toolbar) it is in a special position used to better use the space (and I've read nice feedback about this also in various articles about FX4b1). So when you enter customize it would disappear with bookmarks, thus I moved it to the natural position to allow user to remove it.
The only way I could think of is to add a fake movable button when customize starts and on customize change put the container there. I can try but it's an hack clearly and I'm unsure it would be approved.
Comment 6•14 years ago
|
||
so how, pray tell, do i remove the bookmarks menu item tacked on to the right side of the bookmarks bar? I've managed to get the bookmarks menu to show on the left, but i still have that tacky google chrome-ish bookmarks item on the right.
Comment 7•14 years ago
|
||
While this behavior is not ideal, this does not block the final release of Firefox.
blocking2.0: ? → -
Assignee | ||
Comment 8•14 years ago
|
||
(In reply to comment #6)
> so how, pray tell, do i remove the bookmarks menu item tacked on to the right
> side of the bookmarks bar?
just remove the bookmarks button from the toolbars wherever it is and put it in the buttons palette.
Comment 9•14 years ago
|
||
unfortunately the first 3 or 12 times i tried that it didn't work, but after x number of attempts it did allow me to put it where i wanted. sigh
Assignee | ||
Comment 10•14 years ago
|
||
what do you mean by "it did not work" and where di you put it?
Comment 11•14 years ago
|
||
i removed the bookmarks toolbar item from the bar, placed it inside the customize box, then tried placing it back on the toolbar. the first two or three times the only thing i was able to put back was the bookmarks toolbar item, with bookmarks menu attached. finally, oafter a few attempts bookmarks toolbar item and the bookmarks menu showed up as seperate items, at which point i was able to place both on the toolbar as i liked.
Assignee | ||
Comment 14•14 years ago
|
||
btw, if we want to retain the wise "move to bookmarks" behavior, fixing this could be tricky. The button is not an entity of bookmarks toolbar, is part of Bookmarks Toolbar Items, because we want it to follow bookmarks wherever the user puts them. We should put in a number of hacks to make the above happen since then the button would have a behavior when the user did not move it out of bookmarks and another one when he did move it out of bookmarks. would become mostly uinpredictable.
Comment 15•14 years ago
|
||
Also it will be nice when new bookmark button will occupies less space and will be only button without name (like you selected in "Customize...")
For at least it looks like this in mockups and I like it in that way
Home button have the same problem on bookmarks bar
Comment 16•14 years ago
|
||
This bug does not appear for me for both a clean profile and my month-old profile.
Anyone else?
RESOLVED WORKSFORME?
Assignee | ||
Comment 17•14 years ago
|
||
it does happen, it must happen since it is by design (itf it does not it's a regression), fixing this one requires a design change.
Reporter | ||
Comment 18•14 years ago
|
||
Still happens for me (if I enable the bookmarks toolbar after step 1).
Comment 19•14 years ago
|
||
Brand new install. FF4.0b7.
Removing "Bookmarks" from Toolbar into "Customize" works for current window or browser session only. It then reappears.
Aside from the redundancy of it all,(the word and function "Bookmarks" in 2 places) a procedure to change something in this manner should be a toggle - done or undone with no further toggles being performed "automatically".
Mozillazinr thread:
http://forums.mozillazine.org/viewtopic.php?f=23&t=2052017&start=0
Comment 20•14 years ago
|
||
This now appears to be fixed as of:
Mozilla/5.0 (X11; Linux i686; rv:2.0b8pre) Gecko/20101210 Firefox/4.0b8pre
Thanks!
Comment 21•14 years ago
|
||
I still get this on
Mozilla/5.0 (Windows NT 5.1; rv:2.0b8pre) Gecko/20101210 Firefox/4.0b8pre
Comment 22•14 years ago
|
||
This is the behavior we want for Firefox 4 at this point, because the risk of doing something different is very high.
blocking2.0: betaN+ → -
Comment 23•14 years ago
|
||
(In reply to comment #22)
> This is the behavior we want for Firefox 4 at this point, because the risk of
> doing something different is very high.
So this will be WONTFIX ? Seems a big odd...
Comment 24•14 years ago
|
||
(In reply to comment #23)
> So this will be WONTFIX ? Seems a big odd...
Only for Firefox 4.
Comment 25•14 years ago
|
||
(In reply to comment #24)
> (In reply to comment #23)
> > So this will be WONTFIX ? Seems a big odd...
> Only for Firefox 4.
So in the future releases like Firefox 4.X or 5 it can be fix as I understand correctly, yep ?
Comment 26•14 years ago
|
||
(In reply to comment #25)
> So in the future releases like Firefox 4.X or 5 it can be fix as I understand
> correctly, yep ?
Not 4.x, but the next major release, yes.
Assignee | ||
Updated•14 years ago
|
Whiteboard: [places-next-wanted]
Comment 31•14 years ago
|
||
What's the status on this ? Because it was planed for next major release and 12th deadline is approaching fast ;p
Will it make it in Firefox 5?
Comment 32•13 years ago
|
||
Any chance this can make into Firefox 6?
Branching is in a week.
Comment 33•13 years ago
|
||
(In reply to comment #32)
> Any chance this can make into Firefox 6?
> Branching is in a week.
Given that there is no patch here, that is unlikely.
Updated•13 years ago
|
tracking-firefox7:
--- → ?
Assignee | ||
Comment 35•13 years ago
|
||
what's the point on asking tracking on something that is not a regression in that specific version nor is a fix that should be observed by release drivers?
FYI tracking is not intended as a replacement for wanted or blocking.
Comment 36•13 years ago
|
||
It's marked as wanted in places, so maybe devs should track this if they forget about bugs and not even marking some distant target milestone for fixing it in near future ;)
Updated•13 years ago
|
tracking-firefox7:
? → ---
Updated•12 years ago
|
OS: Mac OS X → All
Summary: Bookmarks button changes moves and changes form in Customize Toolbars mode → Bookmarks button changes moves and changes form in Customize Toolbars mode (depending on Bookmarks toolbar [and menu bar])
Updated•12 years ago
|
Summary: Bookmarks button changes moves and changes form in Customize Toolbars mode (depending on Bookmarks toolbar [and menu bar]) → Bookmarks button changes moves and changes form in Customize Toolbars mode (depending on Bookmarks toolbar [and menu bar, which is bug 631330])
Comment 39•12 years ago
|
||
Sorry, bug 631330 and this one are probably same.
Summary: Bookmarks button changes moves and changes form in Customize Toolbars mode (depending on Bookmarks toolbar [and menu bar, which is bug 631330]) → Bookmarks button changes moves and changes form in Customize Toolbars mode (depending on Bookmarks toolbar and menu bar)
Updated•12 years ago
|
Comment 47•12 years ago
|
||
There is an add-on called bug582139 that fixes the design bug that overrides the user's wishes by moving the bookmarks button to the bookmarks toolbar.
Comment 48•12 years ago
|
||
Yeah, moving the bookmarks toolbar button is something that we (the UX team) thought was a good idea at the time but later realized was a bad idea.
I'll take this.
Assignee | ||
Comment 49•12 years ago
|
||
nothing should happen here, bug 748894 is on its way, please don't work on this.
Assignee | ||
Updated•12 years ago
|
Assignee: fyan → mak77
Comment 50•12 years ago
|
||
Perhaps this should be duped over to bug 748894, since that will fix this bug?
Comment 51•12 years ago
|
||
Oh, I didn't realize that. Thanks.
Comment 52•12 years ago
|
||
(In reply to Paul [sabret00the] from comment #50)
> Perhaps this should be duped over to bug 748894, since that will fix this
> bug?
No, it's a separate issue that can be tracked separately. When Marco fixes bug 748894, if his fix also fixes this bug, he'll get to close two bugs as FIXED and assigned to him, and that's fine. :)
Assignee | ||
Comment 53•12 years ago
|
||
fixed by bug 748894, the button has no more magic, just move it where you want it.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Whiteboard: [places-next-wanted] → [places-next-wanted][fixed in bug 748894]
Comment 54•12 years ago
|
||
Verified as fixed based on fixing bug 748894.
You need to log in
before you can comment on or make changes to this bug.
Description
•