Closed Bug 976006 Opened 11 years ago Closed 11 years ago

Bookmark indicator does not update when adding a bookmark

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Android
defect
Not set
normal

Tracking

(firefox29 unaffected, firefox30 verified, fennec30+)

VERIFIED FIXED
Firefox 30
Tracking Status
firefox29 --- unaffected
firefox30 --- verified
fennec 30+ ---

People

(Reporter: catalin.suciu, Assigned: bnicholson)

References

Details

(Keywords: regression, uiwanted)

Attachments

(1 file)

Nightly 30.0a1 (2014-02-23) Bookmark star indicator does not update when adding a bookmark from the bookmarks submenu
Assignee: nobody → bnicholson
Status: NEW → ASSIGNED
tracking-fennec: --- → ?
Keywords: regression
Is this actually a bug? We already have a checkbox next to Bookmark in the submenu. The star no longer just means "bookmark", it means "save using one of these actions". Do we really want the star to change just from bookmarks being saved? This creates a visual disconnect between the other items in the submenu.
Flags: needinfo?(ibarlow)
Keywords: uiwanted
I think we should fill in the star if any one of these actions have been used to save something. Technically, Reading List and Homescreen items are both bookmarks, and Subscribed pages are close enough where it still warrants some visual indicator that the user has done something mark this page as important.
Flags: needinfo?(ibarlow)
tracking-fennec: ? → 30+
As suggested by ibarlow on IRC, this highlights the star if the page is bookmarked and/or in the reading list.
Attachment #8384186 - Flags: review?(wjohnston)
Attachment #8384186 - Flags: review?(wjohnston) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
Verified as fixed in build 30.0a1 (2014-03-06); Device: Lenovo Yoga Tab 10 (Android 4.2.2).
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: