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

Cleanup some UI #78

Merged
merged 1 commit into from
Oct 22, 2023
Merged

Cleanup some UI #78

merged 1 commit into from
Oct 22, 2023

Conversation

kitajchuk
Copy link
Contributor

@kitajchuk kitajchuk commented Oct 22, 2023

👋 Hey, been awhile!

I was just checking out the site and noticed a couple things that don't have good responsive anymore.

Home page CTA:

Before:
Screen Shot 2023-10-22 at 11 11 38 AM

After:
I took liberty with the layout and wording so feel free to update it now that the pattern is in place here 🤔
Screen Shot 2023-10-22 at 11 11 48 AM

Main navigation:

Before:
Screen Shot 2023-10-22 at 11 12 05 AM

After:
My solution here is to just switch to responsive mobile version (hamburger menu) a bit earlier at 1024. There's just too many items in the navigation for it to remain unpacked any longer than that 🤔

Screen Shot 2023-10-22 at 11 12 18 AM

@kitajchuk kitajchuk requested a review from davehorton as a code owner October 22, 2023 18:15
@kitajchuk kitajchuk temporarily deployed to Production October 22, 2023 18:15 — with GitHub Actions Inactive
@vercel
Copy link

vercel bot commented Oct 22, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
next-static-site ✅ Ready (Inspect) Visit Preview Oct 22, 2023 6:15pm

@davehorton davehorton merged commit ffcae20 into main Oct 22, 2023
2 checks passed
@davehorton
Copy link
Contributor

hey there brandon its great to hear from you! Thanks so much for the PR.

@davehorton davehorton deleted the kita/ui_cleanup branch October 22, 2023 18:23
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