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

Updated appearance gap min #806

Merged
merged 2 commits into from
Dec 9, 2024
Merged

Updated appearance gap min #806

merged 2 commits into from
Dec 9, 2024

Conversation

bhh32
Copy link
Contributor

@bhh32 bhh32 commented Dec 7, 2024

Working on fixing issue #805

Updated the appearance page’s gap min to reflect the theme_builder.active_hint value instead of the default theme.active_hint value. I still need to test the change in this commit before merging.

bhh32 added 2 commits December 6, 2024 16:03
Updated the appearance page’s gap min to reflect the theme_builder.active_hint value instead of the default theme.active_hint value.
@bhh32
Copy link
Contributor Author

bhh32 commented Dec 7, 2024

I have tested locally and the behavior seems to be correct now based on my understanding of the given issue and what the expected functionality is. Please check the functionality updates of active hint and gaps to ensure I understood the expected behavior and their relationship.

@bhh32 bhh32 marked this pull request as ready for review December 7, 2024 02:46
@wash2 wash2 merged commit 00b8b2b into pop-os:master Dec 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants