Major Toonin Redesign - Githubissues

grey-software / toonin

Technology that allows you to tune in to your friends and family in realtime using peer-to-peer sharing.

https://toonin.grey.software

MIT License

67 stars 6 forks source link

Closed ArsalaBangash closed 4 years ago

ArsalaBangash commented 4 years ago

Lakshya2610 commented 4 years ago

I was testing the new UI and noticed a few bugs

Video player is bugged: I can't see any controls on the video player. It's just frame playing the video that pauses on click
The screen capture preview on the share tab doesn't display the video. It just stays white
The timeline changes status on clicks. It shouldn't change to any status on user clicks

Other than these bugs, the UI looks pretty neat!

alir128 commented 4 years ago

The name variable in the vuex store I think should be set to some value if we are not including that screen in this redesign as it might be an issue on server side not 100% sure.
Maybe some padding around the video player would look nicer.
If we have a different background shadow in light mode it would help make the card layout standout. Similar to how it is in dark mode.
196 #197 #200 code will need to be added manually to some components due to name changes.