Skip to content

Remove 'video-on-top' from VLC player launch options - #1286

Merged
feross merged 1 commit into
webtorrent:masterfrom
terryhau:vlc-remove-aot
Apr 19, 2018
Merged

feross merged 1 commit into
webtorrent:masterfrom
terryhau:vlc-remove-aot

Conversation

@terryhau

Copy link
Copy Markdown
Contributor

This command line option just sets the 'Always on top' setting in VLC.
The player will pop up in the foreground when playing a video from webtorrent without this option.

Having VLC always on top is very annoying when multi tasking.
If the user wants VLC always on top, they can set it in VLC preferences.

@Ropiel

Ropiel commented Dec 20, 2017

Copy link
Copy Markdown

I totally agree

With such a minimalist and handy software that's a pretty annoying option,please make that the "Floating on Top" option if unchecked works on vlc too.

@pastudan

Copy link
Copy Markdown

+1 This is a simple change. Any word on if this can be merged?

@feross

feross commented Apr 19, 2018

Copy link
Copy Markdown
Member

We added this option because the VLC window does not come to the front without it. We don't want the window to stay in the front, but there was no other way to focus it without the option.

All told, my opinion on this has changed and I think it's more annoying to have the option enabled so let's try disabling it and see how many complaints that generates :)

@feross feross left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@feross
feross merged commit cc78251 into webtorrent:master Apr 19, 2018
@lock lock Bot locked as resolved and limited conversation to collaborators Jul 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants