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

double arrow key required to move select export format #3787

Closed
Irev-Dev opened this issue Sep 5, 2024 · 3 comments · Fixed by #3809
Closed

double arrow key required to move select export format #3787

Irev-Dev opened this issue Sep 5, 2024 · 3 comments · Fixed by #3809
Labels
bug Something isn't working high-priority

Comments

@Irev-Dev
Copy link
Collaborator

Irev-Dev commented Sep 5, 2024

Screenshare.-.2024-09-05.1_51_22.PM.mp4
@Irev-Dev Irev-Dev added the bug Something isn't working label Sep 5, 2024
@franknoirot
Copy link
Collaborator

I think I can explain what's happening here. This line sets the initially-selected combobox option, and whichever is the currently-active value is highlighted by the classnames ui-active:bg-primary/10 dark:ui-active:bg-chalkboard-90 here. We could remove the line that sets the initial value and it would be more honest UX I think?

Explainer video:

Screenshare.-.2024-09-05.3_40_18.PM-compressed.mp4

@franknoirot
Copy link
Collaborator

This has always been the case well before Electron, and I see it as a UX papercut, so I'd recommend not having as high priority

@Irev-Dev
Copy link
Collaborator Author

Irev-Dev commented Sep 5, 2024

Fix in #3809, it's not pretty though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high-priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants