Closed Bug 1239929 Opened 8 years ago Closed 8 years ago
.5] Tutorial should be merged into marketplace
63 bytes, text/x-github-pull-request
|Details | Review|
In order to solve the redirection problem in tutorial of marketplace, we should merge the tutorial page into the single page application of marketplace.
Hi Davor, This patch might be hard to review because there are lots of refactoring inside. I removed the stand-alone tutorial page and merge it into the marketplace. Since there are two views now, I extract two modules (smart_button.js, spatial_navigation.js) so both views can share the function. Some of the event binding has changed the binding class name so it won't messed up the binding between two views. Also reduce some code in tutorial.html and tutorial.styl thanks to the preprocessor. You can leave any comments in the PR if you have any questions, thanks!
Attachment #8708236 - Flags: review?(dspasovski)
Hi Davor, Could you help to review the patch? We need to have marketplace ready for QA asap. Thank you!
Attachment #8708236 - Flags: review?(dspasovski) → review+
Landed in marketplace-tv-front-end master. https://github.com/mozilla-b2g/marketplace-tv-front-end/commit/286d3dcc79cb71ff975782034608f18665a3220c
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
verified and fixed with the latest build
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.