Video thumbnail is now a fixed height, not width.

[ghost]

I noticed that when opening some videos the thumbnail created makes the window too big to display fully on-screen, in my case this meant I couldn't then click on play to 'cast the video. Having looked at the command-line passed to ffmpeg I realised that you were creating a thumbnail of a fixed width, with an auto-sized height, I've just swapped the command-line arguments so that the thumbnail is a fixed height with an auto-sized width.

The current fixed height value works for me, obviously you can tweak it further to suit.

Also now changed the seek position for the thumbnail, since some of videos that I want to 'cast weren't the defined 27 seconds long.

[keredson]

what screen resolution are you using?

[ghost]

@keredson I'm using a laptop with screen resolution: 1366x768 and found that if I tried to play some videos originally recorded in portrait orientation, the thumbnail would sometimes cause the window to extend below the visible desktop area, even moving the window up to the top of the screen left me unable to click on the media buttons. Also noticed that I couldn't resize the window. My first thought was to try to add accelerators to the media buttons, but then having looked at the code, as per the PR I just made ffmpeg produce a fixed height / variable width thumbnail, rather than a fixed width / variable height one.

The other tweak was just to grab the thumbnail earlier in the stream.

[keredson]

i appreciate the PR, but i like the constant width / variable height design, rather than the other way around. the rest if the UI is designed to fit a specific width. this PR will create weird gaps on the left/right side of the view window.

that being said, we should prob. have a max-height feature somewhere. 768 seems a reasonable value. if you wish to submit another PR for that i'd be game. i opened #61 to track.