Closed
Bug 1089662
Opened 10 years ago
Closed 10 years ago
Consider moving /nominate javascript + template code outside of fireplace
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P4)
Tracking
(Not tracked)
RESOLVED
FIXED
2014-11-11
People
(Reporter: mat, Assigned: mat)
References
Details
(Keywords: perf, Whiteboard: [qa-])
/nominate lives in fireplace, which means its template and javascript code are inlined in include.js. That represents (before minification+gzip): 4.0K nominate.html (twice that size once inlined in include.js, lots of HTML + conditions & loops in that template) 8.0K /media/js/views/nominate.js 4.0K /media/js/views/nominate.styl We would reduce the size of include.{js,css} by having that view served outside fireplace somehow. It's non-essential to fireplace, I don't even know how to reach it from the homepage.
Updated•10 years ago
|
Priority: -- → P4
Comment 1•10 years ago
|
||
Scott says we can drop this page completely if we like.
Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•10 years ago
|
||
Fixed in https://github.com/mozilla/fireplace/commit/c49cebd2cd7d5297dac67773174cee35b5005293
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
Target Milestone: --- → 2014-11-11
Comment 4•10 years ago
|
||
We can't remove the page; the link is out there and I don't want people landing on a broken page. If there's a way to keep the page without too much trouble that would be great. Otherwise, we need to have a redirect. Scott can put up a blog post we can redirect to.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 5•10 years ago
|
||
(In reply to Amy Tsay [:amyt] from comment #4) > We can't remove the page; the link is out there and I don't want people > landing on a broken page. If there's a way to keep the page without too much > trouble that would be great. Otherwise, we need to have a redirect. Scott > can put up a blog post we can redirect to. Amy, on average how many submissions do we get right now? –– It sounds like it shouldn't, but if it has to go down, redirect to https://wiki.mozilla.org/Marketplace/Contributing/Apps/CommunityCuration But that still doesn't say how to nominate apps. Amy, what is the expected route for folks to nominate featured apps to the Marketplace? Is this Nominate page it? Or emailing Scott? It's at least unclear from the Marketplace homepage how to do that. So I'm assuming on the newsletters it's more clear.
Flags: needinfo?(atsay)
Comment 6•10 years ago
|
||
We get a dozen or so per month. You're right, it isn't clear, partly because v1 of the program was invite-only. Let's wait for Scott to write the blog post, which will clarify things, and redirect to that instead. Also want to give Scott a chance to chime in on whether he wants to keep the page or consolidate everything to the as-yet-unwritten blog post.
Flags: needinfo?(atsay)
The number of app noms we receive each month varies, but we got 14 in October. If we take down the Nomination page, I can write a blog post that essentially conveys the same info and asks folks to send app nominations to editor@mozilla.com, which comes directly to me. We can also link to the wiki page form the blog post.
Assignee | ||
Comment 8•10 years ago
|
||
Scott: please write the blog post then. Not having this page means less code to maintain and less content to download (since we bundle all consumer pages together) so it improves Marketplace performance. When the post is up, give the URL here and we'll set up the redirect.
Flags: needinfo?(sdevaney)
Cool. I'll have something written up by the end of this week.
Flags: needinfo?(sdevaney)
Comment 10•10 years ago
|
||
Here's the blog post we can redirect to: https://blog.mozilla.org/apps/2014/11/08/got-a-favorite-app-on-firefox-marketplace-lets-feature-it/
Assignee | ||
Comment 11•10 years ago
|
||
The redirection was added to dev/stage in bug 1096495, it should go in production with the next push.
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•