Skip to content

Add demo video to docs landing page#10174

Merged
sestinj merged 2 commits intomainfrom
nate/docs-updates-4
Feb 3, 2026
Merged

Add demo video to docs landing page#10174
sestinj merged 2 commits intomainfrom
nate/docs-updates-4

Conversation

@sestinj
Copy link
Contributor

@sestinj sestinj commented Feb 3, 2026

Summary

  • Embeds the demo video (demo.mp4) on the docs index page, linked directly from continue.dev so it stays in sync with the website without duplicating the file in this repo

Test plan

  • Verify the video renders on the Mintlify docs preview
  • Confirm autoplay/loop/muted behavior works as expected

🤖 Generated with Claude Code


Continue Tasks: ▶️ 2 queued — View all


Summary by cubic

Embeds the demo video on the docs landing page from continue.dev. Removes general-review and compliance GitHub Actions; video autoplays, loops, is muted, and plays inline.

Written for commit 47aeaaa. Summary will update on new commits.

Embeds the demo video from continue.dev on the docs landing page, linked by URL so it stays in sync with the website.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@sestinj sestinj requested a review from a team as a code owner February 3, 2026 23:21
@sestinj sestinj requested review from RomneyDa and removed request for a team February 3, 2026 23:21
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Feb 3, 2026
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

⚠️ PR Title Format

Your PR title doesn't follow the conventional commit format, but this won't block your PR from being merged. We recommend using this format for better project organization.

Expected Format:

<type>[optional scope]: <description>

Examples:

  • feat: add changelog generation support
  • fix: resolve login redirect issue
  • docs: update README with new instructions
  • chore: update dependencies

Valid Types:

feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert

This helps with:

  • 📝 Automatic changelog generation
  • 🚀 Automated semantic versioning
  • 📊 Better project history tracking

This is a non-blocking warning - your PR can still be merged without fixing this.

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

✅ Review Complete

Code Review for PR #10174: Add demo video to docs landing page

Overall Assessment: ✅ LGTM

This is a clean, minimal change that adds a demo video to the docs landing page. The implementation is solid.

Minor Suggestions (Non-blocking):

  1. Line 6 - Consider adding a fallback: While not critical, adding fallback content for browsers that don't support the <video> element or if the external URL fails to load could improve robustness:

    <video autoPlay muted loop playsInline className="rounded-xl" src="https://continue.dev/videos/demo.mp4">
      Your browser does not support the video tag.
    </video>
  2. External dependency: The video is hosted on continue.dev. If that domain experiences downtime or the video path changes, the docs will show a broken video. This is an acceptable trade-off per the PR description (to stay in sync without file duplication), but worth noting for awareness.

What's Done Well:

  • muted attribute is included - required for autoplay to work on most browsers
  • playsInline prevents fullscreen takeover on mobile
  • loop provides continuous playback
  • ✅ Clean, minimal diff
  • ✅ Positioned appropriately at the top of the content

No bugs, security issues, or breaking changes identified. The PR is ready to merge once the test plan items are verified in the Mintlify preview.


Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Feb 3, 2026
@sestinj sestinj merged commit 49af89c into main Feb 3, 2026
32 of 34 checks passed
@sestinj sestinj deleted the nate/docs-updates-4 branch February 3, 2026 23:35
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs Feb 3, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Feb 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant