Closed Bug 826074 Opened 7 years ago Closed 7 years ago

Update terminology for customizing about:home

Categories

(Firefox for Android :: General, defect)

x86
Linux
defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 20

People

(Reporter: wesj, Assigned: wesj)

Details

Attachments

(1 file)

We currently just use the world "Edit", "Clear", and "Clear All" for editing sites on about:home. This is a bit confusing for a UI that is (currently) basically just pinning and unpinning sites. We only show Clear on sites that are pinned (you can't clear sites that you just don't want to see on about:home), and only show Clear All if any of the sites are pinned (but not necessarily the one you long tapped on).

This is a bit confusing to users who might want to remove a site from our frecent list, or think editing will remove a particular site (it won't if its not a pinned site). Seems like it would be better to use something like "Pin Site", "Unpin Site", "Unpin all" and maybe "Change Pinned Site".
Attached patch PatchSplinter Review
This updates the strings to "Pin Site" "Unpin Site", and "Unpin All". I left "Edit" in on ibarlow's advice. "Pin Site" will pin whatever is already in that slot (i.e. its a fast path of sorts). We definitely need an indicator to make that more obvious that something happened in that case. Hopefully coming tomorrow.
Attachment #697537 - Flags: review?(mark.finkle)
Comment on attachment 697537 [details] [diff] [review]
Patch

clearSite() -> unpinSite()
clearAllSites() -> unpinAllSites()

Just to make things more clear

r+ with that
Attachment #697537 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mozilla-central/rev/9d119006ec64
Assignee: nobody → wjohnston
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Verified fixed on:
-build:  Firefox for Android 21.0a1 (2013-02-15)
-device: Samsung Galaxy Nexus
-OS: Android 4.2.1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.