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

Rework mobile header #61

Closed
wants to merge 2 commits into from
Closed

Conversation

Clete2
Copy link

@Clete2 Clete2 commented Feb 17, 2021

Mobile header was too bulky. I made a more minimized version. Up to you if you want to merge this, but personally I did some testing and I think it looks better on a phone screen. Thanks for making such an awesome theme!

Old:
Screen Shot 2021-02-17 at 2 41 13 PM

New:

Screen Shot 2021-02-17 at 2 41 26 PM

Live look: https://clete2.com

Mobile header was too bulky. I made a more minimized version.
Re-add media query for desktop. Accidentally removed.
@vaga
Copy link
Owner

vaga commented Mar 3, 2021

Thanks for your PR!
Sure, it's too big but your solution doesn't convinced me. It's a personal feelings. 😄
I'm working on it.

@vaga vaga added the wontfix This will not be worked on label Mar 3, 2021
@Clete2
Copy link
Author

Clete2 commented Mar 3, 2021

Sounds good to me @vaga ! I will follow your issue #64 and see what you come up with. 😊

@Clete2 Clete2 closed this Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants