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

Project Wizard: Enter key doesn't work without tabbing past selection of a project type #3491

Closed
testlabauto opened this issue Jun 11, 2024 · 7 comments
Assignees
Labels
area: project wizard Issues related to New Project Wizard category. area: ui Issues related to UI category. bug Something isn't working

Comments

@testlabauto
Copy link
Contributor

Positron Version: 2024.06.0-50

Steps to reproduce the issue:

  1. Start to create a project with the project wizard
  2. Select an R or Python project using TAB and arrow keys
  3. Hit Enter
  4. Note that it won't function without tabbing over to the Next button

What did you expect to happen?

The enter key would immediately work after selecting a project type (when Next becomes active)

Were there any error messages in the output or Developer Tools console?

No

@testlabauto testlabauto added bug Something isn't working area: project wizard Issues related to New Project Wizard category. labels Jun 11, 2024
@testlabauto
Copy link
Contributor Author

Note from Sharon:
If you'd like, we can broaden the issue to all modals since the default accept functionality was generally removed (e.g. it'll impact the New Folder dialog too)

@sharon-wang sharon-wang added the area: ui Issues related to UI category. label Jun 11, 2024
@petetronic petetronic added this to the Release Candidate milestone Jun 12, 2024
@jonvanausdeln
Copy link
Contributor

Note for QA - If this flow changes (e.g. no need for next button), there are automated tests that will need to be updated.

@sharon-wang
Copy link
Member

Another test case: #4203

@petetronic
Copy link
Collaborator

Thomas also mentioned Enter was not working for the Missing R Package dialog, my assumption is there's something missing in the base modal components for these cases.

@sharon-wang
Copy link
Member

Yes, we disabled the default key Enter behaviour in #3442 as it was causing incorrect behaviour in some modals where pressing Enter would accept and close the modal, however the default button is not always for accepting and closing the modal (e.g. the default button could be a Next or Browse button).

We figured it was more disruptive for the Enter key to accept/close a modal whose default button is not OK (which can cause undefined behaviour and loss of user progress) than for the user to have to click the default button, at least until we add support for the default button action.

@softwarenerd softwarenerd changed the title Project Wizard: Enter key doesn't work without tabbing post selection of a project type Project Wizard: Enter key doesn't work without tabbing past selection of a project type Sep 18, 2024
@softwarenerd softwarenerd self-assigned this Sep 18, 2024
@softwarenerd
Copy link
Contributor

This issue is addressed in #4739.

@testlabauto
Copy link
Contributor Author

Verified Fixed

Positron Version(s) : 2024.09.0-81
OS Version          : OSX

Test scenario(s)

Enter key functions as expected in project wizard

Link(s) to TestRail test cases run or created:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: project wizard Issues related to New Project Wizard category. area: ui Issues related to UI category. bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants