estuary / marketing-site

0 stars 5 forks source link

Brenosalv/feature/updated-menu-design #335

Closed Brenosalv closed 3 weeks ago

Brenosalv commented 1 month ago

303

314

324

Changes

[x] Add redesigned product menu;
[x] Add redesigned resources menu

Tests / Screenshots

github-actions[bot] commented 1 month ago

Visit the preview URL for this PR (updated for commit a5f8249):

https://estuary-marketing--pr335-brenosalv-feature-up-lom7xc5e.web.app

_{(expires Sun, 07 Jul 2024 22:08:20 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 76f6b095a0752e5d9c6c890267f9fdc3e392161e}

Brenosalv commented 1 month ago

@travjenkins Also, I removed the keen-slider package and replaced the connectors page carousel with the new one allowing the same colors to the dots.

Brenosalv commented 4 weeks ago

@travjenkins As we don't have a mobile version of the redesigned menus, I felt free to develop the responsiveness including an accordion when it's in mobile devices. Also, the mobile version in prod is bugged when clicking on the header links "Product" and "Resources" requiring us sometimes double clicking to open them. Please let me know your thoughts.

travjenkins commented 4 weeks ago

Only major feedback on mobile menu is I think the items should be spaced vertically a bit more

Current

Increasing gap to 12px