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

added a nav bar and fixed some spacing issues. Sticky position was no… #7

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

Ezra-Crowe
Copy link
Contributor

…t working the way I intended though so if this was configured to work on mobile before you may want to test it again now.

…t working the way I intended though so if this was configured to work on mobile before you may want to test it again now.
@AjaiKN
Copy link
Member

AjaiKN commented Feb 16, 2024

Looks good! It seems like this works fine, it's just that it's no longer sticky. I don't think the stickiness matters a whole lot compared to the changes you've made here, so I'll merge this. We can always make it sticky later.

In the future, it might be good for us to make it work for slightly smaller screens, but those probably aren't too common anymore, so it's not a high priority. (For example, it doesn't look great at 320x568, which is the size of an iPhone 5 or a 1st-gen iPhone SE.)

@AjaiKN AjaiKN merged commit 4e87bf5 into ObieSource:main Feb 16, 2024
1 check 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