fix(home): always-visible homepage navbar (remove #229 reveal-on-scroll)#233
Merged
Merged
Conversation
After reverting the hero redesign (#230 via #232), #229's reveal-on-scroll navbar was left orphaned: the homepage navbar stayed hidden at the top (position: fixed; translateY(-100%)) and only slid in past 8px of scroll — behaviour that only made sense paired with #230's full-screen hero. On the restored normal-height hero it reads as a missing menu. Remove just the hiding behaviour: the scroll listener in index.tsx and the body.is-home .navbar fixed/translate rules in custom.css. The homepage navbar is now always visible like every subpage. Keeps the rest of #229 (nav links shown on the homepage, About us in the footer only).
marcel-veselka
added a commit
that referenced
this pull request
Jun 8, 2026
Rework the homepage hero into a two-column layout: the "start testing" form on the left, a full-colour autoplaying demo video on the right. This restores the demo-video hero from #230 (reverted in #232) and evolves it; rebased on main, so the reveal-on-scroll navbar removed in #233 stays out. - Two-column layout; HeroVideoInline gains a `dimmed` prop (default true). - Drop the video maximize/expand affordance, the "Watch 90-sec demo" link, and the now-unused HeroVideoModal (deleted). - Hovering the video fades the form's gradient border out as the video's own frame fades in (shared 500ms timing). - Tidy hero copy: remove the subtitle; replace the green trust row with a single "No credit card required" line under the CTA. - Trim the sign-in popup: drop the "Testing <host>" line and the "Sign in › Agents test › Get your report" breadcrumb. - Calm the trusted-by marquee: lower logo opacity (70→40), widen the edge fade, slow the scroll (35s→90s) — keeps motion but recedes behind the video. - Responsive vertical spacing + wider form (480→560px) + taller instructions textarea (3→5 rows).
marcel-veselka
added a commit
that referenced
this pull request
Jun 8, 2026
…deo (#235) Rework the homepage hero into a two-column layout: the "start testing" form on the left, a full-colour autoplaying demo video on the right. This restores the demo-video hero from #230 (reverted in #232) and evolves it; rebased on main, so the reveal-on-scroll navbar removed in #233 stays out. - Two-column layout; HeroVideoInline gains a `dimmed` prop (default true). - Drop the video maximize/expand affordance, the "Watch 90-sec demo" link, and the now-unused HeroVideoModal (deleted). - Hovering the video fades the form's gradient border out as the video's own frame fades in (shared 500ms timing). - Tidy hero copy: remove the subtitle; replace the green trust row with a single "No credit card required" line under the CTA. - Trim the sign-in popup: drop the "Testing <host>" line and the "Sign in › Agents test › Get your report" breadcrumb. - Calm the trusted-by marquee: lower logo opacity (70→40), widen the edge fade, slow the scroll (35s→90s) — keeps motion but recedes behind the video. - Responsive vertical spacing + wider form (480→560px) + taller instructions textarea (3→5 rows).
marcel-veselka
added a commit
that referenced
this pull request
Jun 8, 2026
- Rename the "Sign In" navbar button to "Log in". - Re-introduce the homepage reveal-on-scroll navbar removed in #233: the top bar is hidden at the top of the page and slides in once the user scrolls. It now pairs again with the full-screen hero restored in #235. - Trim the navbar to the essentials — logo + Wopee.io + Pricing. Remove the Solutions dropdown and the Blog link (Log in / Start for free CTAs kept).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to the #230 revert (#232).
Problem: #232 correctly reverted the #230 hero redesign, but the navbar behaviour was never part of #230 — it came from #229. With #230 gone, #229's reveal-on-scroll navbar is orphaned: the homepage menu stays hidden at the top (
position: fixed; translateY(-100%)) and only slides in after 8px of scroll. On the restored normal-height hero that looks like a missing menu.Fix: remove only the hiding behaviour —
src/pages/index.tsx: drop the scroll listeneruseEffect(and the now-unuseduseEffectimport).src/css/custom.css: drop thebody.is-home .navbarfixed/translate rules +.navbar--revealed.Homepage navbar is now always visible, same as every subpage. The rest of #229 is kept intact: Solutions/Pricing/Blog still render on the homepage, and 'About us' remains footer-only.