Closed Bug 515751 Opened 10 years ago Closed 10 years ago

when editing bookmarks, no easy way to determine a folder vs a link

Categories

(Firefox for Android Graveyard :: Bookmarks, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED
fennec1.0b4

People

(Reporter: jmaher, Assigned: vingtetun)

Details

Attachments

(1 file)

I clicked manage in my bookmarks and tapped on the first bookmark "mobile".  Little did I know this was a folder not a link.  I was very confused as there was no url to edit and tags to enter.

please made a visible highlight or "favicon" to make it a bit more obvious that it is a folder.  Right now it looks like some mixed up pixels and not much different than the default bookmarks and there default favicons.
We do use a different favicon for folders. Don't you see the folder favicon?
it is too small to see.  After very close examination I could make out the difference.
Hmm, those images are 16px - way too small
also look at https://bug508705.bugzilla.mozilla.org/attachment.cgi?id=399494, which probably help distinguish link vs folder (cause of the arrow on the middle right).

What i don't like in my patch is the fact i use: width instead of max-width / height instead of max-height, so, non-squared icon will be deformed...
Assignee: nobody → 21
Attachment #399999 - Flags: review?(mark.finkle)
Comment on attachment 399999 [details] [diff] [review]
Turn the favicon size 24x24 -awesomebar/bookmarklist

Hmm, I didn't realize the pixels sizes for wince were in the browser.css file. If they are the same for low and high, that's probably fine.

I should file a bug to actually move "same-sized" rules out of low and high css files anyway.
Attachment #399999 - Flags: review?(mark.finkle) → review+
Mark, do you want me to do that in this patch?
(In reply to comment #6)
> Mark, do you want me to do that in this patch?

no, it's bigger than this patch
pushed:
https://hg.mozilla.org/mobile-browser/rev/787ffc6f5ce2
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → B4
verified FIXED on builds:

Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.2a2pre) Gecko/20090914 Fennec/1.0a3

and

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a1pre) Gecko/20090914
Fennec/1.0b4pre
Status: RESOLVED → VERIFIED
Component: General → Bookmarks
You need to log in before you can comment on or make changes to this bug.