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

Improve sample app UI #173

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

AndrewKeepCoding
Copy link
Contributor

Improved and fixed some issues on the sample app.

@dotMorten
Copy link
Owner

I'm at Build right now so can't test it but what is this specifically addressing?

@AndrewKeepCoding
Copy link
Contributor Author

Just improved a couple of margin settings and fixed the Settings page title from "Windows Design" to "Settings". Nothing critical.

@dotMorten
Copy link
Owner

dotMorten commented May 21, 2024

Maybe I'm just not clear on what was wrong with them in the first place?
Also not sure I get the rename to settings - winuiex doesn't have any settings apis

@AndrewKeepCoding
Copy link
Contributor Author

For example, in the Windows Control page, the buttons have inconsistent look and spacing. One of them not showing its entire content:
image
This is how it looks after the change:
image

@AndrewKeepCoding
Copy link
Contributor Author

AndrewKeepCoding commented May 21, 2024

I also fixed the Settings page title:
image
to:
image

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