Closed Bug 584849 Opened 11 years ago Closed 10 years ago

Make Personas feature more discoverable

Categories

(SeaMonkey :: Themes, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.5

People

(Reporter: mnyromyr, Assigned: ewong)

Details

Attachments

(1 file, 1 obsolete file)

Now that lightweight theming works on trunk, it's very hard to find out that it's there and how to use it. "View->Apply Theme" will show installed personas, but "View->Apply Theme->Get New Themes" doesn't mention them at all...

How about adding "View->Apply Theme->Get New Personas", linking to http://www.getpersonas.com/?
(Should this URL be localizable?)
I really don't like having two links there for getting new themes/personas. I have asked the AMO team as I knew there had been talk of doing a single page with themes and personas on it, but apparently that never got done.

Still, something like that would be just right for this.

Also, we surely should do a more interesting first-run page, and refer to personas there in some prominent way.
(In reply to comment #1)
> Still, something like that would be just right for this.

Yes, it'd be only as long as AMO doesn't merge lightweight/personas into themes.
Whiteboard: [good first bug]
<RattyAway>	jorgev: which is the recommended landing page for personas?
	http://www.getpersonas.com/ or https://addons.mozilla.org/personas/
<jorgev>	they should have the same content
but only on getpersonas.com you can upload your own
might be best to recommend that one for now
(In reply to comment #0)
> How about adding "View->Apply Theme->Get New Personas", linking to
> http://www.getpersonas.com/?
IMHO "Get Personas" suffices, as we don't ship any.
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #544824 - Flags: review?(neil)
Comment on attachment 544824 [details] [diff] [review]
Added menu item "Get Persona" to Apply Theme.

>+<!ENTITY getPersonaCmd.label "Get Persona">
Nit: I'm sure I said "Get Personas". But you only managed to call the function getPersonas(), everything else is missing the s...
You probably want to whitelist getpersonas.com as well 
http://mxr.mozilla.org/comm-central/source/mozilla/browser/app/profile/firefox.js#208
Comment on attachment 544824 [details] [diff] [review]
Added menu item "Get Persona" to Apply Theme.

Looks like getpersonas.com is too Firefox-centric, and we should go with amo, but with s/themes/personas/ of course.

Also what I said earlier and what Ratty said, of course.
Attachment #544824 - Flags: review?(neil) → review-
Attachment #544988 - Attachment description: Added menu item "Get Persona" to Apply Theme. (v2) → Added menu item "Get Personas" to Apply Theme. (v2)
Attachment #544988 - Flags: review?(neil) → review+
Keywords: checkin-needed
Comment on attachment 544988 [details] [diff] [review]
Added menu item "Get Personas" to Apply Theme. (v2) [Checkin: comment 11]

http://hg.mozilla.org/comm-central/rev/f4ecb26d2773

This contains l10n changes. I don't know how strict the rules are for Aurora, but for Beta it's probably too late.
Attachment #544988 - Attachment description: Added menu item "Get Personas" to Apply Theme. (v2) → Added menu item "Get Personas" to Apply Theme. (v2) [Checkin: comment 11]
Attachment #544988 - Flags: approval-comm-aurora?
Bah, where's the TM seamonkey2.5 flag when you need it?
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [good first bug]
Target Milestone: --- → Future
Attachment #544988 - Flags: approval-comm-aurora? → approval-comm-aurora+
Comment on attachment 544988 [details] [diff] [review]
Added menu item "Get Personas" to Apply Theme. (v2) [Checkin: comment 11]

Sorry, no L10n changes are possible on aurora. The L10n freeze is the uplift from -central to -aurora.
Attachment #544988 - Flags: approval-comm-aurora+ → approval-comm-aurora-
(In reply to comment #12)
> Bah, where's the TM seamonkey2.5 flag when you need it?

Now it's there ;-)
Target Milestone: Future → seamonkey2.5
You need to log in before you can comment on or make changes to this bug.