Closed Bug 584849 Opened 12 years ago Closed 11 years ago
Make Personas feature more discoverable
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.
<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.
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...
Don't you need something like http://mxr.mozilla.org/comm-central/source/suite/browser/browser-prefs.js#576 as well? Then you can do this: http://mxr.mozilla.org/comm-central/source/suite/browser/navigator.js#2191
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-
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.
Bah, where's the TM seamonkey2.5 flag when you need it?
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
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.