search

wetfish / sync

Synchronize audio and video between friends! Supports HTML5 video, YouTube, and Vimeo
27 stars 8 forks source link

refactor code #51

Open Meleeman01 opened 4 years ago

Meleeman01 commented 4 years ago

as it stands Sync is working and working well, however, the current code could be refactored and simplified and leverage the vue.js framework more. one instance could be moving the template code in vue-app.js into it's on vue file, and importing it into vue-app.js.

Meleeman01 commented 4 years ago

on the refactor list I think adding the media player to the data object in the mediaControl component would be a good idea, so we're not violating the dry principle by initializing a variable whenever we need to access the media player.