-
Notifications
You must be signed in to change notification settings - Fork 492
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ToDo: toolbar improvements #862
Comments
Please don't fuck with the default toolbar, the one you linked is less readable than the current default due to the hollow centers with thin lines. At least leave a setting so i can restore it to the current look. |
Fluent UI toolbar is good and remove separators between buttons is reasonable. I personally think the main problem that affects the visual perception is seek bar and volume bar, They are both outdated with the Fluent style toolbar in the light theme. Are there any future plans for it? After all, we can easily change the style of the toolbar through external files. |
In dark mode the fluent-UI toolbar_dark.svg looks much better IMO. I don't think it's meant to be used in light mode ?
|
I have some ideas for the toolbar. I haven't gotten around to implementing any, yet, but I wanted dynamic options (we discussed having buttons 1, 2, 3 to assign to any action). What I would like to see ideally is:
The toolbar+ files could contain any number of icons that could be selected by the user for each custom button. If the button is missing when switching themes, the button could either disappear or just show a generic "missing" icon. I don't know if any or all of this will ever be done, but that's what I had thought of so far. |
If would not mind of the old bmp/png toolbar support would be dropped to simplify things, and replaced with just new SVG toolbars. I am sure people could make a few good looking ones for us. |
I am excited to see that additional custom buttons may be supported. I have always wanted to add audio and subtitle buttons to more easily select these tracks. |
You can also already easily toggle tracks by pressing A and S. |
Yes, that's what I do now. But toggling through subtitle tracks is not that efficient if the file has a lot of them. If I want to go from Traditional Chinese back to Simplified Chinese, toggling is not my preferred option.
|
Buttons would do exactly the same. If you want a menu, right-click on the video. |
My mistake then, as I was imagining the buttons giving direct access to the audio/subtitle selection menus. |
Option to select different toolbar size for windowed and full-screen modes would be useful, IMHO. |
The toolbars could scale between a few sizes depending on the size of the window. Especially if SVG that might be easy to support. |
Having a more dynamic toolbar (e.g. SVG-based icons) could open the door to users adding the buttons they want as well (seems that was already implied earlier, just adding that I need that feature). Would love to have a repeat, show playlist and subtitle toggle buttons in the toolbar for example! |
Personally the only things that don't like are the volume slider and the position slider. |
A light color version of the dark theme might be added in the future. |
I noticed that when the Dark Theme is enabled the external toolbar image isn't loaded, is there an option to enable this? |
Change filename to "toolbar_dark" |
This is modified version of That would be great if only one file used for toolbar and not separate <svg xmlns="https://www.w3.org/2000/svg" viewBox="0 0 640 640">
<style>
.light {stroke: black; fill: black}
.dark {stroke: white; fill: white}
</style>
<circle class="light" cx="320" cy="320" r="100"/>
</svg> |
You can avoid the dark theme overwriting the colors with white, depending on how you name the file. But the new design will get rid off auto recoloring. |
When I designed the original "dark" themed draw for the toolbar, I disabled the offset button press. However, it was always part of the classic theme. It's a "feature," you might say, of CToolBar to make it appear that 3d buttons are actually moving "into" the screen. Since there is no 3d border, etc., I didn't like it and went for the flat highlighted look. My general rule was to leave classic theme alone. Depending on the toolbar art, that style might look better. Although, with the blue square, probably not. Anyway, if the toolbars are all to look uniform across themes it has to be looked at. I went ahead and committed that change. It does make me think, though, about that code. One of the things I'm doing in the drawing is making "dark" themed buttons instead of using the windows blue highlighting. But if we want toolbars to look similar across themes, maybe we should throw out the classic behavior entirely, for the toolbar. Then both would follow an identical codepath. @clsid2, maybe you want to weigh in here. But there is a deeper issue with volume, seekbar, etc., that they are pretty dated and even if you prefer the classic theme, maybe the "modern" elements might be allowed at least in the player toolbar, volume control, seekbar, etc. It wouldn't be that hard to support a "light" modern theme, either. I just never got around to it. Basically someone has to define the colors and we have to make it a selectable option--and the code needs updating to not use constant color refs. I'm not sure dialogs really need any change--they are basically just native windows widgets and they look normal. Even if the "light" theme was different from classic windows in the player, the dialogs could "pass" as they are still light. If we wanted it all to be fully colorizable, it's a bit more work, but only because certain widgets like checkboxes and radios have some pre-rendered assets. |
I understand that. That was a good idea. I feel it's better to mention all these in a same place now, so that you guys can work on those when you touch this topic. |
@adipose Would it be possible to query the colors for those elements based on the active Windows theme? So we can keep being compatible with custom Windows themes and the special high contrast color themes that Windows offers. Offering multiple modern themes would of course also be good idea. Have a Themes subfolder, where users can provide the necessary data through: Dark and Light themes (and eventually maybe more) would be internally defined themes. What did you think of my idea that I posted a while ago to use two SVG images (empty+full) to combine for rendering the volume slider? |
The two space volume slider is a good idea. Once that is done, it will be consistent across themes. |
Please consider adding a It would be handy to use only mouse, of course I have mapped the "Adding Quit button on the toolbar" is the most requested feature in original MPC where they used to track all the tickets on a different website something called "trac". Why is it a big deal to add a quit button? Is it too much to ask ? Usability and Accessibility are important for any project. |
The player window has a close button. Adding it to toolbar does not make any sense and is not going to happen. I am not debating it. Any attempt to do that will result in a ban. VLC does not have a close button in its toolbar. So you might be confused with the titlebar. If that is somehow hidden for you for MPC-HC, then press 3 to reset to normal view. |
(English is not my first language, sorry If my comment hurt you in anyway, I'm not good at english phrases and sentence making) I'm talking about Fullscreen mode. (no titlebar in fullscreen) and VLC has the option to add/remove whatever buttons we want in fullscreen, just FYI. I know right-click menu has Exit entry (two clicks), we can save one extra unnecessary click by having an exit button on the toolbar. That's all. I'm not into any confusion or debate or to hurt anyone by pointing issues. I respect your opinion, you are the owner, and this is a free player so I won't ask anything, just suggested. My intention was to make the MPC better for all users. No need to be angry on me. This comment is only for making my previous comment clear, I'M NOT DEBATING. I respect everyone. Thanks for this awesome media player. It's been a great journey with this MPC player. |
I've added code to support a custom volume draw. Seems to work ok. I've started removing legacy/modern codeblocks as it's getting too confusing maintaining: Classic volume In particular the calculations for how to draw everything including the thumb scroller is getting convoluted so I've changed it to only support the last type of volume control. |
Can you make a test build? |
I noticed the same issues. Maybe this helps for SVG line smoothness: It might perhaps also be a good idea to round the height (after DPI scaling) of the toolbar stuff to nearest multiple of 4 (or maybe even 8). That could help reduce scaling quality issues, since Windows also allows uncommon scaling values. Custom toolbar designs could use a rectangle, different colors, etc. |
It's simply a proof of concept. This is loaded from the svg so a rectangular volume wouldn't really properly test it. The color issues must be due to rounding at certain resolutions. It worked well for me @4k. The rough edge might be an issue. It is initially rendered with nanosvg just like the toolbar. But alpha blending maybe doesn't work properly. Maybe it's just more obvious with the non 45 angle used on buttons. |
https://mega.nz/file/JVpxSaxa#m5sKWFu_yXTTIrvEXkoidqAMhPJcxOgpcom7CpJKxu4 This seems to fix at least one redraw issue. What I observed is that the invalidated region of the volctrl is insufficient for the larger draw area, when it's changed without using the mouse (using the mouse triggers some thumb revalidation, etc., so that was working). Shortcuts or mousewheel caused funny redraw in there. For now if there's any change I'm redrawing the whole thing. |
https://mega.nz/file/oVhh0aIJ#EJm_1JyQgHUW4R9KcLR_dzUpuqOan-qkfqQ9rSXJz74 Alpha blending is working now. That should make it smooth. |
This build removed the debug code, in case it caused any issues for testing. https://mega.nz/file/hch0kD4D#InLfLP5QeAomuoiihXHoF_n8CRBLfWNXcc5osuSAan4 |
Kind of a feature request regarding the customization. I'd like to see it change it to the middle of the screen, this is useful in ultrawide screens where you have them all the way to the left in a windowed video and also looks more aesthaetically pleasing. Having indicators about the file being played would be awesome too. Like how we can see H/W is we're using hardware decoding. Inficators for codec (H264, H265, AV1) and type (8-bit, 10-bit, HDR, etc) would be nice to have. The H/W status could also be placed in this section and maybe offer an option to highlight the word and use it as a button to disable or enable hw acceleration. This part can also host buttons for the most used features on video files like others suggested above: subtitle and audio tracks.. Havintg this bottom panel be more useful and not just host playback buttons on one side and volume on the other with lots of empty space that can be used for more functionality or ease of use would be good. I'm not a programmer but if it's not that hard to implement in SVG I will definitely be willing to make buttons, indicators, etc and post them here for everyone to benefit. |
There already is an option for showing codec in status bar. |
I missed that in advanced options, thanks. That's one example of information that could be implemented to show in the bottom toolbar instead of the status bar, or to replace it as mentioned in my previous comment. I'll keep an eye on this repo now that I'm using MPC-HC full time again. |
Putting information in toolbar is not going to happen. |
Feature Request: On both the Modern and Classic Seekbar can you add an option to specify a color? This is to help up the contrast when viewing the progress the video at a distance. The default light grey (classic) and blue (modern dark) are very difficult to see when viewing at a distance and using a 4K display. |
It sounds like an option worth adding, but maybe put it in as a separate issue. I've been very busy lately but I still monitor this project. |
Ah, there's already a ticket about this. I was going to also open a ticket to suggest that there be an option to hide the speed buttons, which I tend to hit by accident instead of the skip buttons since they look so similar, plus they're right next to each other. Other suggestions that might help without adding an option:
|
I came to add that I'd like to remove the speed buttons. It's just so weird to have that function as dedicated buttons directly on the toolbar. Do people use those that often? I just use hot keys when doing chapter work, so it seems like those would be hot keys as well. I would like jump to next/previous file as buttons. Right now I use the Pageup/Pagedown hot keys, but sometimes I'm just in a mouse mood. |
Jump to next/previous file buttons are literally already next to the speed buttons for two decades. If there are chapters, it jumps to chapters. But you can right-click to always jump file. Similarly you need to use Shift+PageDown in case chapters are present. |
The speed buttons look identical to the skip buttons. I constantly click the wrong button, making the toolbar useless for me. The toolbar should be customizable; buttons should be able to be removed or grouped differently. To make this player usable for me, here's my workaround:
|
The text was updated successfully, but these errors were encountered: