Move property_editor/reconnecting_overlay.dart -> not_connected_overlay.dart (#9594) The changes I made to this file prevented git from detecting it as a rename in my previous PR. To slightly simplify the review, I left the file where it was with a TODO to move after. This is just that move.
diff --git a/packages/devtools_app/lib/src/standalone_ui/ide_shared/property_editor/reconnecting_overlay.dart b/packages/devtools_app/lib/src/standalone_ui/ide_shared/not_connected_overlay.dart similarity index 90% rename from packages/devtools_app/lib/src/standalone_ui/ide_shared/property_editor/reconnecting_overlay.dart rename to packages/devtools_app/lib/src/standalone_ui/ide_shared/not_connected_overlay.dart index aac701f..0a67722 100644 --- a/packages/devtools_app/lib/src/standalone_ui/ide_shared/property_editor/reconnecting_overlay.dart +++ b/packages/devtools_app/lib/src/standalone_ui/ide_shared/not_connected_overlay.dart
@@ -6,11 +6,8 @@ import 'package:devtools_app_shared/ui.dart'; import 'package:flutter/material.dart'; -import '../../../shared/globals.dart'; -import '../../../shared/ui/common_widgets.dart'; - -// TODO(dantup): Rename and move this file one level up. Leaving as-is to -// make the review/diff simpler. +import '../../shared/globals.dart'; +import '../../shared/ui/common_widgets.dart'; /// An overlay to show when we are not connected to DTD based on the /// [DTDConnectionState] classes.
diff --git a/packages/devtools_app/lib/src/standalone_ui/standalone_screen.dart b/packages/devtools_app/lib/src/standalone_ui/standalone_screen.dart index 6a1f64a..0072eed 100644 --- a/packages/devtools_app/lib/src/standalone_ui/standalone_screen.dart +++ b/packages/devtools_app/lib/src/standalone_ui/standalone_screen.dart
@@ -8,8 +8,8 @@ import '../shared/globals.dart'; import '../shared/ui/common_widgets.dart'; +import 'ide_shared/not_connected_overlay.dart'; import 'ide_shared/property_editor/property_editor_panel.dart'; -import 'ide_shared/property_editor/reconnecting_overlay.dart'; import 'vs_code/flutter_panel.dart'; /// "Screens" that are intended for standalone use only, likely for embedding