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

Disable irrelevant buttons in the New Project dialog #1706

Closed
HeyMyian opened this issue Feb 18, 2024 · 4 comments · Fixed by #1707
Closed

Disable irrelevant buttons in the New Project dialog #1706

HeyMyian opened this issue Feb 18, 2024 · 4 comments · Fixed by #1707
Assignees
Labels
enhancement Request: New feature or improvement project management Component: Project or Project Tree usability Issue: Could do with some improvement

Comments

@HeyMyian
Copy link
Contributor

HeyMyian commented Feb 18, 2024

2024-02-18___22_33_20

Originally posted in #1699 (comment)

@vkbo
Copy link
Owner

vkbo commented Feb 18, 2024

The New Project doesn't do anything when you're in the New Project page, but the Browse and Open buttons perform their actions regardless of what page is visible, so one of the two solutions below is probably sensible.

When pressing the Browse or Open button when the New Project page is active:

  1. Do nothing at all. Just block the action.
  2. Switch back to the Project Overview page, but don't perform the actions requested.

I think 1 is the cleanest, but 2 gives some implied feedback about why the buttons aren't doing anything.

Since the two pages are just stacked on top of each other, dropping out of the New Project page doesn't do anything to the already filled in content, so you can just press "New Project" to go back again.

@vkbo vkbo self-assigned this Feb 18, 2024
@vkbo vkbo added enhancement Request: New feature or improvement project management Component: Project or Project Tree usability Issue: Could do with some improvement labels Feb 18, 2024
@vkbo vkbo added this to the Release 2.3 RC 1 milestone Feb 18, 2024
@HeyMyian
Copy link
Contributor Author

Do these buttons come with a distinct "inactive" style maybe?

@vkbo
Copy link
Owner

vkbo commented Feb 18, 2024

Not sure. Open and Cancel are standard buttons, and they are different between operating systems. These dialog button boxes are designed to integrate with the style of the host OS, so it's harder to predict how they turn out. As you've seen, disabling a button or control doesn't necessarily mean it looks inactive. The toggle switch looks inactive because I designed it, including the code that paints the different parts, and chose to make it very clear.

I think perhaps option 1 is the safest solution.

@vkbo
Copy link
Owner

vkbo commented Feb 18, 2024

Actually, it's only the Open button that needs handling. Browse has nothing to do with either page, and is a functionality all on its own, like Cancel. That's why I put it where it is in the first place. I put Open there because otherwise it would be inconsistent, since Open is a system standard button.

I think I will just block the open action when the project list isn't visible and leave it at that.

@vkbo vkbo linked a pull request Feb 20, 2024 that will close this issue
6 tasks
@vkbo vkbo closed this as completed Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Request: New feature or improvement project management Component: Project or Project Tree usability Issue: Could do with some improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants