416180 - Better wording for feedicon tooltip

Status: RESOLVED FIXED

(Camino Graveyard :: Location Bar & Autocomplete, defect)

Description

[Stefan [:stefanh]]

Attachment: Change the string

I'd suggest "Show list of available feeds" instead of the current "This page contains xml feed links". Samuel didn't liked "Show". I think the verb suits fine, because that's what happens when you click on the icon/button

Comment 1

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

I'm not completely happy with "Show", either, but I've been unable to arrive at something I'm really in love with yet :(

Part of the problem is that this "button" is both informational (it comes up only when needed) and functional (passes the selected feed to your reader).

The phrase I most liked was

"There are feeds available for this page"

Of Sam's suggestions, I most liked

"List available feeds"

Each of them seem to do exactly half of the job :p


When we arrive on a wording we all like, can you also get rid of

> /* Prefs window */
> "ShowAllPrefItems" = "Show All";

That string hasn't been used since 2004 :(

Comment 2

[Stefan [:stefanh]]

(In reply to comment #1)
> I'm not completely happy with "Show", either, but I've been unable to arrive at
> something I'm really in love with yet :(
> 
> Part of the problem is that this "button" is both informational (it comes up
> only when needed) and functional (passes the selected feed to your reader).
> 
> The phrase I most liked was
> 
> "There are feeds available for this page"
> 
> Of Sam's suggestions, I most liked
> 
> "List available feeds"
> 
> Each of them seem to do exactly half of the job :p

Hmm. And combining them will end up in things like "Feeds available. Click to show list", "Feeds. Click to show available feeds" etc. Perhaps we can find something that does the whole job... If we can't, I think it's worth considering:

What do you need to explain/inform? That the icon means there are feeds available or that you can do something with the icon (click the icon to show feeds)?

> 
> 
> When we arrive on a wording we all like, can you also get rid of
> 
> > /* Prefs window */
> > "ShowAllPrefItems" = "Show All";
>
> That string hasn't been used since 2004 :(
Oh. Sure, np.

Comment 3

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 301956 [details] [diff] [review]
Change the string

I finally caught Sam and we talked, and we want to go with

"List feeds offered by this page"

Comment 4

[Stefan [:stefanh]]

Attachment: Use "List feeds offered by this page" and remove pref string

Don't know if I need more reviews on this? http://wiki.caminobrowser.org/Development:Reviewing seems to indicate I do, though .

Comment 5

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 303535 [details] [diff] [review]
Use "List feeds offered by this page" and remove pref string

r=ardissone; thanks for doing this, and sorry about the delay.

> Don't know if I need more reviews on this?
> http://wiki.caminobrowser.org/Development:Reviewing seems to indicate I do,
> though .

I thought the last time I updated that page I added a note about "small patches", but I guess not.  We also typically waive the rule for established Mozilla contributors ;)

Comment 6

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 303535 [details] [diff] [review]
Use "List feeds offered by this page" and remove pref string

sr=pink

Comment 7

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Stefan, you have cvs privs, right?  If so, please land this on the MOZILLA_1_8_BRANCH and the trunk.

Comment 8

[Stefan [:stefanh]]

trunk:
Checking in camino/resources/localized/English.lproj/Localizable.strings.in;
/cvsroot/mozilla/camino/resources/localized/English.lproj/Localizable.strings.in,v  <--  Localizable.strings.in
new revision: 1.13; previous revision: 1.12
done

MOZILLA_1_8_BRANCH:
Checking in camino/resources/localized/English.lproj/Localizable.strings.in;
/cvsroot/mozilla/camino/resources/localized/English.lproj/Localizable.strings.in,v  <--  Localizable.strings.in
new revision: 1.1.2.13; previous revision: 1.1.2.12
don

Comment 9

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Thanks!