Skip to content

Add video tutorial link to the Deep Links panel - #9925

Open
binSaed wants to merge 5 commits into
flutter:masterfrom
binSaed:add-deeplinks-video-tutorial-9425
Open

Add video tutorial link to the Deep Links panel#9925
binSaed wants to merge 5 commits into
flutter:masterfrom
binSaed:add-deeplinks-video-tutorial-9425

Conversation

@binSaed

@binSaed binSaed commented Jul 28, 2026

Copy link
Copy Markdown

Description

Most DevTools panels display a "Watch tutorial" link next to the docs link in the status line. The Deep Links panel is missing this link because it doesn't have a tutorialVideoTimestamp in its ScreenMetaData.

The Deep Links panel has its own dedicated video tutorial (https://www.youtube.com/watch?v=d7sZL6h1Elw) which is separate from the main "Dive in to DevTools" video used by other panels.

This PR adds a tutorialVideoUrl field to ScreenMetaData that allows a screen to specify a custom video URL. When set, VideoTutorialLink uses this URL instead of the default DevTools YouTube video. The "Watch tutorial" link now appears on the Deep Links panel.

Changes

  • packages/devtools_app/lib/src/shared/framework/screen.dart: Added tutorialVideoUrl field to ScreenMetaData and set it for the deepLinks screen.
  • packages/devtools_app/lib/src/framework/scaffold/status_line.dart: Updated showVideoTutorial check and VideoTutorialLink to use tutorialVideoUrl when available.

Fixes #9425

binSaed added 2 commits July 29, 2026 01:38
Use the tutorialVideoUrl from ScreenMetaData when available,
falling back to the default DevTools YouTube video timestamp.
@binSaed
binSaed requested a review from a team as a code owner July 28, 2026 22:39
@binSaed
binSaed requested review from srawlins and removed request for a team July 28, 2026 22:39

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request introduces support for custom video tutorial URLs (tutorialVideoUrl) in ScreenMetaData, allowing individual screens to link to dedicated video tutorials instead of falling back to a timestamp on the default DevTools YouTube video. Feedback on the changes highlights two main areas of improvement: handling cases where both tutorialVideoUrl and tutorialVideoTimestamp are null to prevent generating an invalid URL containing the string 'null', and using the shortened youtu.be domain format for the new URL to maintain consistency with other YouTube links in the codebase.

Comment thread packages/devtools_app/lib/src/framework/scaffold/status_line.dart Outdated
Comment thread packages/devtools_app/lib/src/shared/framework/screen.dart Outdated

@kenzieschmoll kenzieschmoll left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the contribution. Overall looks good. Just a couple of comments.

@binSaed

binSaed commented Jul 29, 2026

Copy link
Copy Markdown
Author

Thanks for the review! Both comments addressed — PTAL.

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.

Add video tutorial for deeplinks panel

2 participants