Make Personas feature more discoverable

RESOLVED FIXED in seamonkey2.5

Status

SeaMonkey
Themes
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: Karsten Düsterloh, Assigned: ewong)

Tracking

Trunk
seamonkey2.5

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

8 years ago
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?)

Comment 1

8 years ago
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.
(Reporter)

Comment 2

8 years ago
(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.

Updated

7 years ago
Whiteboard: [good first bug]

Comment 3

7 years ago
<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

Comment 4

7 years ago
(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)

Updated

7 years ago
Assignee: nobody → ewong
Status: NEW → ASSIGNED
(Assignee)

Comment 5

7 years ago
Created attachment 544824 [details] [diff] [review]
Added menu item "Get Persona" to Apply Theme.
Attachment #544824 - Flags: review?(neil)

Comment 6

7 years ago
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...

Comment 7

7 years ago
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

Comment 8

7 years ago
You probably want to whitelist getpersonas.com as well 
http://mxr.mozilla.org/comm-central/source/mozilla/browser/app/profile/firefox.js#208

Comment 9

7 years ago
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-
(Assignee)

Comment 10

7 years ago
Created attachment 544988 [details] [diff] [review]
Added menu item "Get Personas" to Apply Theme. (v2) [Checkin: comment 11]
Attachment #544824 - Attachment is obsolete: true
Attachment #544988 - Flags: review?(neil)

Updated

7 years ago
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+
(Assignee)

Updated

7 years ago
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
Last Resolved: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [good first bug]
Target Milestone: --- → Future

Updated

7 years ago
Attachment #544988 - Flags: approval-comm-aurora? → approval-comm-aurora+

Comment 13

7 years ago
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-

Comment 14

7 years ago
(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.