Skip to content

Show Performance and CPU Profiler screens without app connection - #6567

Merged
kenzieschmoll merged 6 commits into
flutter:masterfrom
kenzieschmoll:open-offline
Oct 23, 2023
Merged

kenzieschmoll merged 6 commits into
flutter:masterfrom
kenzieschmoll:open-offline

Conversation

@kenzieschmoll

@kenzieschmoll kenzieschmoll commented Oct 20, 2023 •

Copy link
Copy Markdown
Member

Without the app connection, these screens are here to provide users with a way to load offline data. This PR also removes the ImportFileAction from the set of scaffold actions, and replaces the existing "Export" buttons with a new OpenSaveButtonGroup, since we will keep these actions local to the screens that support Open / Save.

Screenshot 2023-10-20 at 12 47 44 PM
Screenshot 2023-10-20 at 12 47 53 PM

Error message when importing a DevTools file for the incorrect screen:
Screenshot 2023-10-20 at 12 50 51 PM

New Open / Save actions:
Screenshot 2023-10-20 at 2 26 13 PM
Screenshot 2023-10-20 at 2 26 49 PM

Fixes #4645

@kenzieschmoll
kenzieschmoll requested a review from a team as a code owner October 20, 2023 21:37
@kenzieschmoll
kenzieschmoll requested review from CoderDake and removed request for a team October 20, 2023 21:37
Comment thread packages/devtools_app/lib/src/screens/performance/performance_screen.dart Outdated
@kenzieschmoll
kenzieschmoll merged commit 3bf3eb3 into flutter:master Oct 23, 2023
@kenzieschmoll
kenzieschmoll deleted the open-offline branch October 23, 2023 17:35
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.

Import/Export (or Save/Open) should be discoverable and consistent

3 participants