Closed Bug 1239929 Opened 9 years ago Closed 9 years ago

[TV][2.5] Tutorial should be merged into marketplace

Categories

(Marketplace Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: jyeh, Assigned: jyeh)

References

Details

Attachments

(1 file)

In order to solve the redirection problem in tutorial of marketplace, we should merge the tutorial page into the single page application of marketplace.
Blocks: 1236810
Assignee: nobody → jyeh
Status: NEW → ASSIGNED
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)
QA Whiteboard: [COM=TV::Web Apps]
Hi Davor,
Could you help to review the patch? We need to have marketplace ready for QA asap. 
Thank you!
Flags: needinfo?(dspasovski)
done
Flags: needinfo?(dspasovski)
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: 9 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.

Attachment

General

Created:
Updated:
Size: