Closed
Bug 619105
Opened 14 years ago
Closed 14 years ago
Persona preview screenshot doesn't match actual persona applied
Categories
(www.mozilla.org :: General, defect)
www.mozilla.org
General
Tracking
(Not tracked)
VERIFIED
WORKSFORME
2.1
People
(Reporter: abuchanan, Unassigned)
References
()
Details
(Whiteboard: [redesign])
Attachments
(2 files)
http://nova.stage.mozilla.com/en-US/firefox/customize/
Hover over the large persona screenshot. The screenshot doesn't match the persona the actually gets applied.
Comment 1•14 years ago
|
||
Is this really a blocker?
Reporter | ||
Comment 2•14 years ago
|
||
(In reply to comment #1)
> Is this really a blocker?
Oops, that was out of habit. Doesn't seem like a launch blocker to me.
Anyway, this is a bug with getpersonas, not mozilla.com
Severity: blocker → normal
Component: www.mozilla.com → getpersonas.com
QA Contact: www-mozilla-com → getpersonas-com
Comment 3•14 years ago
|
||
The image here <http://www.getpersonas.com/static/img/persona-sample.jpg> seems to be dynamically generated since it isn't on svn, and the script seems to be generating it from the wrong persona. The personas listed at the right, and the one that previews, are the top popular ones. I'm still haven't been able to figure out exactly what's making it though.
Comment 4•14 years ago
|
||
(In reply to comment #3)
> The image here <http://www.getpersonas.com/static/img/persona-sample.jpg> seems
> to be dynamically generated since it isn't on svn, and the script seems to be
> generating it from the wrong persona. The personas listed at the right, and
> the one that previews, are the top popular ones. I'm still haven't been able
> to figure out exactly what's making it though.
It's just a static image from Bug 522754 [1]. If that list is dynamic, we probably can't maintain a large main preview image like that. Maybe the large preview image should always be tied to a single editorially-chosen Persona?
[1] http://viewvc.svn.mozilla.org/vc/projects/getpersonas.com/production/static/img/persona-sample.jpg?view=log
Comment 5•14 years ago
|
||
Comment 6•14 years ago
|
||
the personas on the right (including the one being referenced for the large preview) are generated in [1]. Looking at it, it seems that list is personas marked as 'mozilla_personas' in the db, and has changed since that page was crafted. Turning it back is just a matter of setting the original personas to be mozilla_personas. I can't do that yet, but I believe :dria can.
Right now, the "Featured Personas" on getpersonas.com are determined by this same attribute, so I've added a few functions to storage.php that create a new featured_personas attribute that can be used for this in the future. Filed bug 620200 for that patch.
[1]
http://viewvc.svn.mozilla.org/vc/projects/getpersonas.com/production/server/lib/storage.php?view=log
Comment 7•14 years ago
|
||
If this can't be fixed for the Fx4 release, can we at least make the large image
not trigger a preview at all? It's confusing to preview a different persona
from the one in the image.
http://www.mozilla.com/en-US/firefox/customize/
Comment 8•14 years ago
|
||
I've fixed this temporarily by reassigning the first featured Persona for mozilla.com to be the same as the large image (Japanese Tattoo). This isn't ideal, but until that page is redesigned/rebuilt, it's a reasonable workaround.
The large image will have to be updated for Firefox 4 anyhow, since Personas now draw in the titlebar. Hopefully someone can fix it then. This requires a mozilla.com change, so reassigning to that component.
Component: getpersonas.com → www.mozilla.com
QA Contact: getpersonas-com → www-mozilla-com
Comment 9•14 years ago
|
||
Am linking this bug to the overall 'build' bug for the Fx4 version of the personalization page.
Blocks: 626671
Updated•14 years ago
|
Target Milestone: --- → 1.4
Comment 10•14 years ago
|
||
Any update on this? What needs to happen?
Updated•14 years ago
|
Target Milestone: 1.4 → 1.5
Comment 11•14 years ago
|
||
This has to be fixed on the getpersonas.com side. Can we move it to the appropriate component and who should we assign it to?
Depends on: 620200
Comment 12•14 years ago
|
||
Sorry, what needs to be fixed here? The preview image and persona match up for me on this page: http://www.mozilla.com/en-US/firefox/customize/
Comment 13•14 years ago
|
||
(In reply to comment #12)
> Sorry, what needs to be fixed here? The preview image and persona match up for
> me on this page: http://www.mozilla.com/en-US/firefox/customize/
Deb, I might be mixed up on this too, but I think they work now because of your temp fix in comment #8. Is that sufficient to close the bug, or do you think we need a more flexible long-term fix?
Comment 14•14 years ago
|
||
I think this is probably sufficient for now, but we can look at a more flexible long-term fix as part of the new site after we merge GetPersonas into AMO. I'll double-check to make sure Wil has this on his radar for that.
Thanks :)
Updated•14 years ago
|
Target Milestone: 1.5 → 2.0
Updated•14 years ago
|
Target Milestone: 2.0 → 2.1
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → WORKSFORME
Updated•14 years ago
|
Status: RESOLVED → VERIFIED
Assignee | ||
Updated•13 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
Assignee | ||
Updated•12 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in
before you can comment on or make changes to this bug.
Description
•