Skip to content
This repository has been archived by the owner on May 7, 2020. It is now read-only.

Close button in toolbar UI for physically disabled persons (ticket #4328) #144

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

olegbevz
Copy link

Hello,
I've added a close button in toolbar after the "step" button and before the "volume" button.
I've checked mph working on Windows XP Professional, Windows 7 x64 and Windows 10 Preview x86.
Hope everything is ok.
Ticket:: https://trac.mpc-hc.org/ticket/4328#no4

@Stanzilla
Copy link

Your indenting looks wrong

@Underground78
Copy link
Contributor

I honestly don't really see how this can help physically disabled persons. I would think that it's easier for them to either use the hotkey or the usual close button in the title bar since it's common to all software.

@olegbevz
Copy link
Author

According to the instructions at Readme.md i've selected the easiest ticket number #4838 on the track and did the first part of it. I thought that if there is a ticket, then there must be a need for a close button.
If the feature is unuseful why there is a feature request for a close button with highest priority on the track?
Stanzilla, i've corrected indenting.

@Underground78
Copy link
Contributor

There are a lot of tickets on the tracker that haven't been assessed yet. The priority is fixed by the ticket opener and can then be changed by the team. The high priority here was set by the opener and nobody from the team had time to look at the ticket and decided on what to do with it.

@CrossVR
Copy link
Contributor

CrossVR commented Oct 12, 2014

The ticket was aimed specifically at the fullscreen UI. There is indeed no button to go back to windowed mode without stopping the video. Rather than introducing a redundant close button, it would be better to introduce a fullscreen toggle button on the toolbar.

@Underground78 We might want to prevent the ticket opener from setting the priority, it's unhelpful. I haven't seen a convenient way on trac to do this, but we can prevent it using a plugin.

@olegbevz
Copy link
Author

Maybe I can be helpfull with other real tasks? Especially with user interface?

@Underground78
Copy link
Contributor

Rather than introducing a redundant close button, it would be better to introduce a fullscreen toggle button on the toolbar.

Seems easier to me to just double click anywhere on the screen in that case than aiming at a button in a possibly hidden toolbar.

@CrossVR
Copy link
Contributor

CrossVR commented Oct 12, 2014

Seems easier to me to just double click anywhere on the screen in that case than aiming at a button in a possibly hidden toolbar.

So is clicking anywhere to pause, but we have a button for that. We should have buttons for the very basic functionality of the player. Anyway, I don't think we should suddenly introduce a new button in an old interface, better to have it as part of a larger change.

@XhmikosR
Copy link
Contributor

Won't this break things with current toolbar images we have on Trac?

@Armada651: I agree with the priority change.

@Underground78
Copy link
Contributor

Won't this break things with current toolbar images we have on Trac?

Yes it would.

@XhmikosR
Copy link
Contributor

I doubt we could get the toolbar creators to update their contrributions. I mean, we don't even have all of the contact details at all...

Anyway, I guess if we decide to merge this, it should be in the next major version bump. And let's hope people will just update their toolbars.

@olegbevz
Copy link
Author

I agree with Armada651. For me it is very uncomfortable to enter the full size mode by double click: video stops playing after clicking. So the toggle button like in VMWare or Remote Desktop can be useful. To add the toggle button we need to change the toolbar image.

4.Please let users see the thumbnails while mouse over the time-line..as seen in youtube videos.

May be this feature from ticket can be helpful? Can I implement it?

@LuanMax
Copy link
Contributor

LuanMax commented Oct 13, 2014

For me it is very uncomfortable to enter the full size mode by double click: video stops playing after clicking.

Hm, how about just pressing F11 (or any other user-defined key)?

By the way, there are some physically or intellectually disabled people from which I sometimes get some feedback about the German translation. And as far as I know, all of them are able to get along just fine with the actual MPC-HC version.

@ioannis-e
Copy link
Contributor

@LuanMax The problem arises when you are working on multiple screens.
In such a scenario, one screen will play mpc-hc full screen, while you might be working on something else on the second one (so mpc-hc has already lost focus).

So one way to give mpc-hc focus back again is to click on the video and thus pause the video (the other would be to Alt-Tab through all open windows), then use ESC (lets say) to exit full screen.

In my setup i use the middle button to enter/exit full screen, but most of the time that button is pretty sensitive and often i find my self skipping backwards/forwards to other videos in the folder...

Personally i agree that a toolbar button would be very convenient way to enter/exit full screen.

@LuanMax
Copy link
Contributor

LuanMax commented Oct 13, 2014

The problem arises when you are working on multiple screens.

OK, thanks, I missed this scenario.

@vBm
Copy link
Contributor

vBm commented Oct 13, 2014

May be this feature from ticket can be helpful? Can I implement it?

I'd suggest that you maybe comment on ticket per se or talk with guys via IRC if you want faster response, being that we're a bit off topic here xD

@kasper93
Copy link
Contributor

"Flybar" would probably solve the problem. It is perfect to hold toggle fullscreen button among other things.

stdedos pushed a commit to stdedos/mpc-hc that referenced this pull request Jan 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

9 participants