Skip to content

Add animated video heroes to industry pages#5378

Open
dimitrieh wants to merge 4 commits into
split/hero-mediafrom
split/industry
Open

Add animated video heroes to industry pages#5378
dimitrieh wants to merge 4 commits into
split/hero-mediafrom
split/industry

Conversation

@dimitrieh

@dimitrieh dimitrieh commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Description

Outcome: industry pages open with an animated video hero and show a derived grid of relevant use cases, a more credible, tailored story per industry.

Adds the industry layout markup that renders the <video> hero from front matter (with reduced-motion handling), per-page front matter (eyebrow, icon, video/poster), and a dev image-handler fix that disambiguates output by source path.

Builds on: the hero-media assets beneath it (the video and poster files), the use-case includes from the use-case PR (the derived grid), and the navigation PR's shared styles.

Related Issue(s)

Split from #5371.

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes

Stack (left to right is merge order; #5373 punctuation is independent, off main):

flowchart LR
    main([main])
    main --> P73["#5373 punctuation"]
    main --> P74["#5374 shop-floor"] --> P75["#5375 nav"] --> P76["#5376 use-cases"] --> P77["#5377 hero-media"] --> P78["#5378 industry"] --> P79["#5379 AI"] --> P80["#5380 homepage"] --> P81["#5381 polish"]
    classDef here stroke-width:4px;
    class P78 here;
Loading

@netlify

netlify Bot commented Jul 24, 2026

Copy link
Copy Markdown

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit 65ee907
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/6a63dbab6eaf31000862d657
😎 Deploy Preview https://deploy-preview-5378--flowforge-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 62 (🟢 up 2 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 91 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment on lines -164 to -168
<div class="max-w-screen-lg mx-auto mt-16 text-center">
<p class="text-gray-500 text-lg font-semibold">{{ hero.trustedByText }}</p>
<div class="mx-auto text-center -mt-0.5 -mb-10">
{% include "social-proof.njk" %}
</div>

@Yndira-E Yndira-E Jul 24, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dimitrieh this section needs to be where it is, it was one of @JoycePlaysFootball's request that the logos where right below the hero section, and also, we had a more meaningful copy above the icons than the one proposed in this PR.

@Yndira-E

Copy link
Copy Markdown
Contributor

I like the eyebrow above the title, it helps provide more context.

The videos… I’m not sure we need them. The current layouts already have visuals that support the message, so adding background videos feels redundant, and I don’t think they add enough value in their current state:

  • The content already sits alongside a contextual visual, either a product image or a testimonial (only automotive), so introducing a full-width background video means we now have two competing visuals mostly telling the same story.
Screenshot 2026-07-24 at 17 32 30
  • Since the content sits inside a large semi-transparent card for readability, most of the video ends up obscured, leaving only the edges visible. At that point it feels more like decoration than content that supports the message.
  • The quality isn’t good enough; they look blurry/pixelated. I suspect that’s the tradeoff of compressing them enough to keep their size reasonable, which makes me question whether they’re adding enough value to justify introducing them in the first place.
  • They’re not seamless loops, so there’s a noticeable jump when they restart, which makes the experience feel less polished.
  • I’m not convinced the additional motion adds enough value to justify the extra visual complexity.

Regarding the animated hover effect for the links, same feedback as in the previous PRs: I’d keep it only for the main navigation.

Co-authored-by: Zeger-Jan van de Weg <ZJvandeWeg@users.noreply.github.com>
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