Skip to content

[ Widget Preview ] Respond to IDE navigation events and show previews from the currently focused script - #174466

Merged
auto-submit[bot] merged 6 commits into
masterfrom
widget_previews_selected_script_responsive
Sep 2, 2025
Merged

[ Widget Preview ] Respond to IDE navigation events and show previews from the currently focused script#174466
auto-submit[bot] merged 6 commits into
masterfrom
widget_previews_selected_script_responsive

Conversation

@bkonyi

@bkonyibkonyi commented Aug 26, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

This change allows for the widget previewer to react to activeLocationChanged events sent over the Editor DTD service to automatically filter the set of visible previews based on the currently selected file in the IDE. This functionality can be turned on or off depending on whether or not the developer wants to see all previews in the project at once.

This change also includes some minor refactoring and UI changes to move the widget preview environment controls to a reserved area at the bottom of the preview window.

Demo:

Screen.Recording.2025-08-26.at.4.40.22.PM.mov

Fixes#175661

@bkonyi
[ Widget Preview ] Response to IDE navigation events and show preview…
9ae9d5d
…s from the currently focused script
This change allows for the widget previewer to react to
`activeLocationChanged` events sent over the `Editor` DTD service to
automatically filter the set of visible previews based on the currently
selected file in the IDE. This functionality can be turned on or off
depending on whether or not the developer wants to see all previews in
the project at once.
This change also includes some minor refactoring and UI changes to move
the widget preview environment controls to a reserved area at the bottom
of the preview window.
@github-actionsgithub-actionsBot added the tool Affects the "flutter" command-line tool. See also t: labels. label Aug 26, 2025
@bkonyi
bkonyi requested a review from matanlureyAugust 26, 2025 21:17
@bkonyi
bkonyi marked this pull request as ready for review August 26, 2025 21:17
@bkonyibkonyi changed the title [ Widget Preview ] Response to IDE navigation events and show previews from the currently focused script[ Widget Preview ] Respond to IDE navigation events and show previews from the currently focused scriptAug 29, 2025
LongCatIsLooong
LongCatIsLooong approved these changes Aug 29, 2025
View reviewed changes

@LongCatIsLooongLongCatIsLooong 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.

LGTM with nits.

Comment threadpackages/flutter_tools/lib/src/widget_preview/dependency_graph.dart
Comment thread...s/flutter_tools/templates/widget_preview_scaffold/lib/src/widget_preview_rendering.dart.tmpl Outdated
Comment thread...tools/templates/widget_preview_scaffold/lib/src/widget_preview_scaffold_controller.dart.tmpl Outdated
Comment thread...tools/templates/widget_preview_scaffold/lib/src/widget_preview_scaffold_controller.dart.tmpl
Comment thread...tools/templates/widget_preview_scaffold/lib/src/widget_preview_scaffold_controller.dart.tmpl
Comment threadpackages/flutter_tools/templates/widget_preview_scaffold/lib/src/controls.dart.tmpl
@justinmc

justinmc commented Aug 29, 2025

Copy link
Copy Markdown
Contributor

Thanks for reviewing @LongCatIsLooong

@bkonyi

bkonyi commented Sep 2, 2025

Copy link
Copy Markdown
ContributorAuthor

Thanks so much for the review!

@bkonyibkonyi added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 2, 2025
@auto-submit
auto-submitBot added this pull request to the merge queueSep 2, 2025
Merged via the queue into master with commit 1d19f97Sep 2, 2025
154 checks passed
@auto-submit
auto-submitBot deleted the widget_previews_selected_script_responsive branch September 2, 2025 17:28
@flutter-dashboardflutter-dashboardBot removed the autosubmit Merge PR when tree becomes green via auto submit App label Sep 2, 2025
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 2, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 2, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
5ba89e0
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 2, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 2, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
0d6236e
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 2, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 2, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
0556081
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 2, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 2, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
b094fe8
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 3, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 3, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
685c1de
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 3, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 3, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
eea3c98
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 3, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 3, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
cb07a96
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 4, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
9276872
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 4, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
b4c2b16
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 4, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
21ca9a3
…previews from the currently focused script (flutter/flutter#174466)
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Sep 4, 2025
Closed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
fa14c5c
…previews from the currently focused script (flutter/flutter#174466)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 5, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
fdc4cf4
…previews from the currently focused script (flutter/flutter#174466)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 5, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
8968c83
…previews from the currently focused script (flutter/flutter#174466)
@bkonyibkonyi mentioned this pull request Sep 15, 2025
Open
mboetger pushed a commit to mboetger/flutter that referenced this pull request Sep 18, 2025
@bkonyi@mboetger
[ Widget Preview ] Respond to IDE navigation events and show previews…
e4435ba
… from the currently focused script (flutter#174466)
This change allows for the widget previewer to react to
`activeLocationChanged` events sent over the `Editor` DTD service to
automatically filter the set of visible previews based on the currently
selected file in the IDE. This functionality can be turned on or off
depending on whether or not the developer wants to see all previews in
the project at once.
This change also includes some minor refactoring and UI changes to move
the widget preview environment controls to a reserved area at the bottom
of the preview window.
**Demo:**
https://github.com/user-attachments/assets/c3b93826-8437-4655-8264-6beed6651fe7
Jaineel-Mamtora pushed a commit to Jaineel-Mamtora/flutter_forked that referenced this pull request Sep 24, 2025
@bkonyi@Jaineel-Mamtora
[ Widget Preview ] Respond to IDE navigation events and show previews…
97bccb6
… from the currently focused script (flutter#174466)
This change allows for the widget previewer to react to
`activeLocationChanged` events sent over the `Editor` DTD service to
automatically filter the set of visible previews based on the currently
selected file in the IDE. This functionality can be turned on or off
depending on whether or not the developer wants to see all previews in
the project at once.
This change also includes some minor refactoring and UI changes to move
the widget preview environment controls to a reserved area at the bottom
of the preview window.
**Demo:**
https://github.com/user-attachments/assets/c3b93826-8437-4655-8264-6beed6651fe7
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 12, 2025
@engine-flutter-autoroll
1d19f97[ Widget Preview ] Respond to IDE navigation events and show …
3a104dd
…previews from the currently focused script (flutter/flutter#174466)
lucaantonelli pushed a commit to lucaantonelli/flutter that referenced this pull request Nov 21, 2025
@bkonyi@lucaantonelli
[ Widget Preview ] Respond to IDE navigation events and show previews…
7cc7efd
… from the currently focused script (flutter#174466)
This change allows for the widget previewer to react to
`activeLocationChanged` events sent over the `Editor` DTD service to
automatically filter the set of visible previews based on the currently
selected file in the IDE. This functionality can be turned on or off
depending on whether or not the developer wants to see all previews in
the project at once.
This change also includes some minor refactoring and UI changes to move
the widget preview environment controls to a reserved area at the bottom
of the preview window.
**Demo:**
https://github.com/user-attachments/assets/c3b93826-8437-4655-8264-6beed6651fe7
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LongCatIsLooongLongCatIsLooongLongCatIsLooong approved these changes
@matanlureymatanlureyAwaiting requested review from matanlurey

Assignees

No one assigned

Labels

toolAffects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support filtering preview set based on the currently selected source file in the IDE

3 participants

@bkonyi@justinmc@LongCatIsLooong