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 Responsiveness #65

Merged
merged 7 commits into from Jun 20, 2021
Merged

Conversation

kinson
Copy link
Contributor

@kinson kinson commented Jun 14, 2021

Overview

Addresses #9 .

The primary focus of this pr is to update the header element to be more responsive. There are some other tweaks to page layouts to eliminate awkward gaps on smaller screens.

It turns out there were not many changes needed for the lessons menu!

I have added a few screenshots here taken from Chrome using the iPhone X viewport setting. I tested this with the iPad size and iPhone 5/SE screen size as well. Let me know if there are issues with specific device types/browsers that you can think of!

Also more generally if you think the layout of the menu should be different I am happy to make changes.

Screenshots

drawing

drawing

drawing

drawing

@doomspork doomspork added enhancement New feature or request tailwind This work requires TailwindCSS changes labels Jun 14, 2021
@doomspork
Copy link
Member

This is awesome to the max 🎉

@doomspork doomspork merged commit cc3b642 into elixirschool:master Jun 20, 2021
@kinson kinson deleted the responsive-menu branch June 20, 2021 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tailwind This work requires TailwindCSS changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants