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

gui(installer): make edit key modal scrollable #1412

Merged

Conversation

jp1ac4
Copy link
Collaborator

@jp1ac4 jp1ac4 commented Oct 31, 2024

This is to fix #1402 following the suggestion in #1402 (comment).

This avoids problems with the "Apply" button when there are
many keys to choose from.
@pythcoiner
Copy link
Collaborator

here the actual rendering:
image

i was more thinking making the content of the "card" scrollable (or only the list of keys), but no strong opinions

@jp1ac4
Copy link
Collaborator Author

jp1ac4 commented Nov 1, 2024

i was more thinking making the content of the "card" scrollable (or only the list of keys), but no strong opinions

I'll give that a try.

@pythcoiner
Copy link
Collaborator

tACK 47a5d32

Copy link
Member

@edouardparis edouardparis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 47a5d32

@edouardparis edouardparis merged commit bd3422c into wizardsardine:master Nov 4, 2024
29 of 31 checks passed
@jp1ac4 jp1ac4 deleted the gui-installer-fix-apply-button branch November 6, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Installer - "Apply" button sometimes not filled when setting up a key
4 participants