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

feat(settings): made it scrollable #117

Merged
merged 1 commit into from
Dec 24, 2023

Conversation

adhirajsinghchauhan
Copy link
Contributor

@adhirajsinghchauhan adhirajsinghchauhan commented Dec 24, 2023

Fixes #116 and #45.

Continuing from #97 (comment).

When content exceeds window height:
image

When content fits within window:
image

@adhirajsinghchauhan adhirajsinghchauhan requested a review from a team December 24, 2023 21:10
Frigyes06
Frigyes06 previously approved these changes Dec 24, 2023
@Frigyes06
Copy link
Member

Do you want to fix the font sizes in a different PR?

@adhirajsinghchauhan
Copy link
Contributor Author

Do you want to fix the font sizes in a different PR?

Yep because it may need other adjustments

@Frigyes06
Copy link
Member

As you mentioned, the code currently has 13, 15, 20, 23, 25 sizes.
I'd propose rounding 13 and 15 to 14, and 23 & 25 to 24

So we'd use 14, 20, 24

Frigyes06
Frigyes06 previously approved these changes Dec 24, 2023
@adhirajsinghchauhan adhirajsinghchauhan merged commit 7048eca into main Dec 24, 2023
24 checks passed
@adhirajsinghchauhan adhirajsinghchauhan deleted the feature/scrollable-settings branch December 24, 2023 22:04
@Frigyes06 Frigyes06 linked an issue Jan 3, 2024 that may be closed by this pull request
@Frigyes06 Frigyes06 added bug Something isn't working UI Related to the UI labels Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working UI Related to the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make settings window scrollable Backup / Restore UI Broken On Linux (OpenGL)
2 participants