Closed Bug 888745 Opened 7 years ago Closed 7 years ago

Visually separate links without icons in bookmarks toolbar

Categories

(Firefox :: Theme, defect)

24 Branch
x86
Windows XP
defect
Not set

Tracking

()

RESOLVED FIXED
Tracking Status
firefox23 --- unaffected
firefox24 + verified

People

(Reporter: andershol, Unassigned)

References

Details

(Keywords: regression, uiwanted)

Attachments

(1 file)

When you have links without icons (e.g. bookmarklets) in the bookmarks toolbar, it seems to me that the toolbar buttons should be separated visually in some way so it is more clear where one button stops and where the next begins.

In the attached screen shot there are five toolbar buttons ("Google", "zap plugins", "+", "English", "todo").

I believe that the toolbar buttons without an icon was previously shown with a default icon or placeholder.
Status: UNCONFIRMED → NEW
Ever confirmed: true
I can confirm this bug.  But as a workaround, you can hover over the links.  This highlights the buttons.
May be related to bug 734326
Flagging UX/Dao here to help understand if this if this is shipping blocker from UX perspective for firefox given the decision made in Bug 691594 .
Flags: needinfo?(shorlander)
Flags: needinfo?(madhava)
Flags: needinfo?(dao)
(In reply to bhavana bajaj [:bajaj] from comment #3)
> Flagging UX/Dao here to help understand if this if this is shipping blocker
> from UX perspective for firefox given the decision made in Bug 691594 .

Yes, I think we should back it out.
Flags: needinfo?(dao)
Flags: needinfo?(shorlander)
Flags: needinfo?(madhava)
Flags: needinfo?
Flags: needinfo?
fixed by backing out bug 691594
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
I see this bug was fixed by the backout of bug 691594.  However that bug was reopened rather than wontfixed. So, on the chance that there would be another similar approach as before I wanted to point out another side effect. 
I had a bookmarklet named "DR" whose width was halved with the icon removal and clicking it suddenly became some sort of mouse-eye dexterity test.
For kicks I renamed it to "i" and the hover width became 9 pixels.  This seems like a clear UX issue to me.

I just wanted to make sure the width would be a part of discussions if there would be a future approach to removing the placeholder.
Keywords: verifyme
Verified as fixed on Firefox 24 beta 7 (BuildID: 20130829135643).
Mozilla/5.0 (Windows NT 5.1; rv:24.0) Gecko/20100101 Firefox/24.0
QA Contact: petruta.rasa
(In reply to Caspy7 from comment #6)
> I see this bug was fixed by the backout of bug 691594.  However that bug was
> reopened rather than wontfixed. 

Thanks a lot, Caspy7. Now bug 691594 is marked as WONTFIX, effectively saying "tough luck, suckers" to all of us for whom the space-wasting placeholder icons cause trouble.

Rather than fixing your problem by pushing for a long-standing bug to be set in stone, how about looking for a solution to both? I can think of three right off the top of my head:

- Use the same policy for separating bookmark toolbar items that is used for menu toolbar items. This would make the UI consistent.
- If a visible separator must be forced on everyone, at least make it narrow, like the separator lines used everywhere else in the menus and toolbars.
- Improving on that idea, make the use of a visible separator configurable, so those of us using bookmarkelts don't have them pushed off the edge of the browser window.

A group of students has chimed in on bug 691594 volunteering to tackle it. How about letting them do it instead of calling for a rather short-sighted WONTFIX?
Another idea:
- Define a minimum width for toolbar items. Simple. Probably easy. Doesn't screw other people.
You need to log in before you can comment on or make changes to this bug.