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

Giving each nav item the same weight so items are aligned properly #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

savvasdalkitsis
Copy link
Contributor

Currently, items are not always centered correctly. If you have 3 items for example, the middle item's icon is not in the center of the screen when one of the other to tabs are pressed.

This PR fixes that

@commandiron
Copy link
Owner

commandiron commented Jul 25, 2022

I can see that the items are not centered correctly, but weighting the Box may prevent the titles from appearing correctly. When you try with 4 items after giving weight, you can see that the titles are not fully visible. Already navigation have a problem with long titles. maybe we can think of them together for reach a different solution.

@savvasdalkitsis
Copy link
Contributor Author

maybe pushing the text underneath the icon will allow for more icons to be placed next to each other?

@commandiron
Copy link
Owner

you are right, I will try to do it. Thanks for pr!

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