Skip to content
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

Button arrangement on Options dialog is cumbersome #129

Open
7 of 10 tasks
InteXX opened this issue Feb 28, 2023 · 5 comments
Open
7 of 10 tasks

Button arrangement on Options dialog is cumbersome #129

InteXX opened this issue Feb 28, 2023 · 5 comments
Labels
enhancement New feature or request

Comments

@InteXX
Copy link

InteXX commented Feb 28, 2023


Prerequisites

  • I think my problem is NOT with youtube-dl/yt-dlp
  • I've verified and I assure that I'm running yt-dlg 1.8.4
  • I assure that I am using the latest version of youtube-dl/yt-dlp
  • Searched bugtracker
  • I've read the FAQs file

What is the purpose of your issue?

  • Bug report
  • Feature request (request for a new functionality)
  • Question
  • Other

Feature request (request for a new functionality)

  • I've verified and I assure that my requested feature is NOT in the TODO list

What operating system do you use?

Windows 11


Issue description

Could you move the Reset button over to the left-hand corner, change the Close button caption to OK and add an Apply/Cancel button set? The gold standard for dialogs features an OK/Apply/Cancel button set with the following ruleset: OK saves changes and exits; Apply saves changes but doesn't exit; Cancel exits without saving changes.

With the Reset button currently in the normal OK position, it's far too easy to click it by mistake and lose a whole host of painstakingly-crafted settings. Force of habit is powerful, and owing to the standard we've become very much accustomed to clicking the leftmost button in the lower right-hand corner in order to save changes. Unfortunately, however, yt-dlg's current button arrangement far too often leads to the exact opposite result.

Please consider moving the destructive Reset button to the lower left-hand corner and implementing an OK/Apply/Cancel button set in the lower right-hand corner.


@oleksis
Copy link
Owner

oleksis commented Feb 28, 2023

Can share the mockup how will look for the OS running?

@oleksis oleksis added the enhancement New feature or request label Feb 28, 2023
@InteXX
Copy link
Author

InteXX commented Feb 28, 2023

I expect it might look something like this:

image

OK would be the dialog's Accept button (initiated when Enter is pressed), and Cancel would be the dialog's Cancel button (initiated when Esc is pressed). The underlined hotkeys would initiate when their corresponding key combinations are pressed (Alt+R, Alt+O, etc).

The hotkeys are a good choice for accessibility design (keyboard only users, etc).

That's generally the ruleset.

@InteXX
Copy link
Author

InteXX commented Feb 28, 2023

I'd give it a stab myself, but I've not written a lick of Python.

I see Python is now supported in Visual Studio, so at least there's that.

@oleksis
Copy link
Owner

oleksis commented Mar 1, 2023

We will need the consensus for the legacy behavior. Need design a Dialog keep the actual behavior and follow the general guidelines

@InteXX
Copy link
Author

InteXX commented Mar 1, 2023

Sure, makes sense. You could skip the Apply button and swap the order of the OK/Cancel buttons.

But the Reset button is still best moved to an area all its own, its functionality perhaps even wrapped in a warning: "Are you sure you want to reset your configuration to the defaults?"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants