Skip to content

Fix broken devtools inspector source navigation #8058

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 15, 2025
Merged

Fix broken devtools inspector source navigation #8058

merged 1 commit into from
Apr 15, 2025

Conversation

pq
Copy link
Contributor

@pq pq commented Apr 14, 2025 •
edited
Loading

Restores source navigation from the embedded devtools inspector.

Fixes: #8041

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@pq pq requested a review from jwren April 14, 2025 23:46
@pq pq mentioned this pull request Apr 14, 2025
Closed
jwren
jwren approved these changes Apr 15, 2025
View reviewed changes
Copy link
Member

@jwren jwren left a comment

Choose a reason for hiding this comment

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

I just tested in Meerkat, LGTM

@jwren jwren mentioned this pull request Apr 15, 2025
Merged
@pq pq merged commit 2e70e33 into flutter:master Apr 15, 2025
9 checks passed
@pq pq deleted the bug_inspectorSourceLinking branch April 15, 2025 19:56
@jwren jwren mentioned this pull request Apr 15, 2025
Closed
jwren added a commit that referenced this pull request Apr 15, 2025
@jwren
Update the changelog.md for M85.2 (#8059)
2e8af12 
Update the changelog for
#8058

https://github.com/flutter/flutter-intellij/milestone/101
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@jwren jwren jwren approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@pq @jwren