Page MenuHomePhabricator

Split home.js into different components
Open, Needs TriagePublic


Currently, home.js looks too large and became difficult to debug and solve issues so split home.js into different components.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 11 2020, 11:00 AM

@Gopavasanth Hi! As this task is tagged with good first task, it might be ideal if you can add to the task description, what home.js component does, and how to go about splitting it. For the scope of this task, contributing to one new component might be reasonable that you can advise what that should be. Please update the task description accordingly. Thanks!

Hsync7 added a subscriber: Hsync7.Apr 18 2020, 1:45 PM

Can I do this task, in the direction of contributing for the project idea ?

We can split the code into many components for instance Header, Footer, Player, VideoSettings (menu) and many more, For the acceptance of the code, at least one component should be made without any code duplication.

Hello @Hsync7 If you are working on this task, Please claim the task and feel free to ask any questions if you have !

Soda claimed this task.Sep 29 2020, 4:58 AM
Soda added a subscriber: Soda.

I'll try to get some part of this done.

Change 630981 had a related patch set uploaded (by Sohom Datta; owner: Sohom Datta):
[labs/tools/VideoCutTool@master] Seperated header, footer, notificationsUtils and VideoSettings

Change 630981 merged by jenkins-bot:
[labs/tools/VideoCutTool@master] Seperated header, footer, notificationsUtils and VideoSettings

Hsync7 removed a subscriber: Hsync7.Oct 5 2020, 9:57 AM