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

TopNavBar adjustments #183 #275

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

Conversation

agrawalashita
Copy link

  • Fixed vertical alignment of the props and logo
  • Adjusted Donate button alignment in mobile view

@ddfridley ddfridley linked an issue Jan 21, 2025 that may be closed by this pull request
11 tasks
Copy link
Contributor

@ddfridley ddfridley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The alignment looks good.
In the Dark Mode Mobile case there is:
image

The donate button should be on the bottom not on the side.

You can easily switch back to master with storybook running and see the difference.

Also, I am going to add a couple of issues to this issue that are problem I see, not with your changes but with previous changes that weren't detected.

@agrawalashita
Copy link
Author

agrawalashita commented Jan 23, 2025

I have fixed the donate button issue @ddfridley. The new issues which you have added in issue #183, should that be fixed as part of this PR?

@ddfridley
Copy link
Contributor

Yes, please fix them too. They are existing bugs but since they are in this same code and really need to be fixed, we should keep our attention on it. But if it becomes too difficult, we can separate them out into another issue.

@ddfridley
Copy link
Contributor

@agrawalashita Are you able to address these 2 tasks I had added to the issue?

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.

TopNavBar layout handling + adjustments
2 participants