Skip to content

fix(website): prevent hero headline overlapping the film frame - #899

Merged
cameri merged 2 commits into
mainfrom
fix/website-hero-overlap
Oct 18, 2026
Merged

cameri merged 2 commits into
mainfrom
fix/website-hero-overlap

Conversation

@Ferryx349

@Ferryx349 Ferryx349 commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Description

On wide viewports, the home hero headline (“Nostr runs on relays…”) could sit under the top-left of the hero film frame, making the copy hard to read. This change reserves a headline band above the film and slightly scales down the title and frame so they stay visually separated without shrinking the hero much.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Non-functional change (docs, style, minor refactor)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my code changes.
  • I added a changeset, or this is docs-only and I added an empty changeset.
  • All new and existing tests passed.

Reserve headline space and slightly scale down the hero title and film on wide layouts.
@changeset-bot

changeset-bot Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7a37cbc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
nostream-website Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Ferryx349
Ferryx349 deployed to website-preview October 18, 2026 18:30 — with GitHub Actions Active
@github-actions

github-actions Bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

🧹 Site preview for this PR was deleted (PR closed).

@cameri
cameri merged commit 3ed70e4 into main Oct 18, 2026
11 of 12 checks passed
@cameri
cameri deleted the fix/website-hero-overlap branch October 18, 2026 18:32
@cameri
cameri deployed to website-preview October 18, 2026 18:32 — with GitHub Actions Active
@greptile-apps

greptile-apps Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge; no actionable issues were found.

Summary

The PR gives the hero headline more room above the film.

  • The wide home hero keeps its headline clear of the film frame.

No actionable issues were found. Visual checks were not run because the website dependencies are not installed.

Reviews (1) · Last reviewed commit: "fix(website): stop hero headline overlap..." · Reviewed by Greptile

This branch was successfully deployed

1 active deployment
website-preview — 7a37cbc2 Deployed Oct 18, 2026 by cameri via preview #84
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