Closed
Bug 1001402
Opened 10 years ago
Closed 10 years ago
[tarako] Consider stripping more fireplace code from yogafire
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
2014-05-06
People
(Reporter: mat, Assigned: mat)
References
Details
(Keywords: perf, Whiteboard: [qa-])
We might not want to do all of this because we need a plan to avoid maintaining 2 apps in the long run, but here is a small incomplete list of stuff we should consider stripping from Yogafire for performance or for package size reasons: - All unneeded fireplace templates - this makes yogafire template.js bigger than fireplace's (since we have extra templates on top), which sucks, it should be the other way around. - All unneeded fireplace javascript - some views are completely unnecessary. Some modules are also irrelevant. We currently have to run code that's useless for tarako (like fireplace categories). - All unneeded fireplace style - this is much harder to track down, but there are entire .styl files that we don't need. - All unneeded fireplace images - most of them referenced in the style files mentioned just above. All these optimizations could mean better load times (much less stuff to parse and load by the javascript and css engine) and smaller memory footprint.
Assignee | ||
Comment 1•10 years ago
|
||
Another (much smaller impact) one: - All unneeded translations
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → mpillard
Priority: -- → P2
Comment 3•10 years ago
|
||
I'm going to block v0 on this due to L10n reasons. Let's chat in person if concerned about it. Mathieu says it can be done tomorrow.
Blocks: 998811
Priority: P2 → P1
Assignee | ||
Comment 4•10 years ago
|
||
Fixed in https://github.com/mozilla/yogafire/commit/ddcec4865be84b17b60589932a5027d27d418fbd
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
Target Milestone: --- → 2014-05-06
You need to log in
before you can comment on or make changes to this bug.
Description
•