Skip to content

Fix settings modal button sizing #23

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 25, 2025
Merged

Fix settings modal button sizing #23

merged 1 commit into from
Oct 25, 2025

Conversation

@built-by-as
Copy link
Owner

@built-by-as built-by-as commented Oct 25, 2025

Summary

  • Removed full-width styling from primary buttons in the settings modal
  • Buttons now size appropriately based on their content instead of stretching to fill the entire container width

Changes

  • Removed w-full class from .btn-primary in styles.css

Test plan

  • Open the settings modal and verify buttons (Cancel, Reset to Default, Save) are appropriately sized
  • Verify buttons still function correctly
  • Check other modals to ensure button styling is consistent

@built-by-as
Fix settings modal button sizing
e6de589 
Remove full-width styling from primary buttons to make them appropriately sized based on content rather than stretching to fill the container.
@built-by-as built-by-as merged commit 7f9a954 into main Oct 25, 2025
1 check passed
@built-by-as built-by-as deleted the settings-buttons branch October 25, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@built-by-as