Paratii-Video / paratii-mediaplayer

v0.0.2 of the Paratii Player, with a sleeker UI, plugin-oriented architecture
http://paratii.video
GNU General Public License v3.0
15 stars 2 forks source link

🔧Implement video quality icon and function #47

Closed felipegaucho closed 6 years ago

felipegaucho commented 6 years ago

Along the lines of https://github.com/Paratii-Video/paratii-mediaplayer/issues/46, we don't have this control yet. It was working pretty well on the legacy player, and I assume we just need to port the code.

@pedrocasa do we have specifications for what happens once the "engine icon" button is clicked?

bent0b0x commented 6 years ago

@pedrocasa I think you mentioned you have a design for this but can you post it here?

bent0b0x commented 6 years ago

FYI @felipegaucho I've built out the surrounding logic/functionality for the quality selector. Once the design is ready I can implement it and we'll be good to go:

playbackleveldemo

pedrocasa commented 6 years ago

Its here Ben https://github.com/Paratii-Video/paratii-mediaplayer/issues/32

bent0b0x commented 6 years ago

☝️ I do not think it is in there. Could you reference the design directly in this issue? It is much easier to work w/ when it is put in the specific issues 😄

bent0b0x commented 6 years ago

image

bent0b0x commented 6 years ago

@felipegaucho @pedrocasa this isn't perfect yet but lmk what you think so far

qualitydemo

pedrocasa commented 6 years ago

The dot should be locked on the center and the values should move

0

pedrocasa commented 6 years ago

Also, I don't know if u should focus on the small details but if u take a look at the specs (https://github.com/Paratii-Video/paratii-mediaplayer/issues/32) the panel of the player has 95% of opacity.

bent0b0x commented 6 years ago

Do we really want to do dragging here? Is clicking ok for now? @pedrocasa.

Just asking because it will be much quicker 😄

pedrocasa commented 6 years ago

Sure, don't need to drag. Analysing it again maybe it's not even a good idea to drag it. What I want to keep about this idea is the dot blocked and the values moving

bent0b0x commented 6 years ago

For sure, I like that too. I'll work on that tonight!

pedrocasa commented 6 years ago

Hen Ben, I've Updated the video resolution menu for screens bigger than 664px ( still working o screens that are bigger than this - but for now we've already a re-start)

link on #32