Skip to content

Commit 79f0a20

Browse files
authored
Remove new inspector welcome banner (#9569)
1 parent 9f7bf22 commit 79f0a20

File tree

3 files changed

+0
-48
lines changed

3 files changed

+0
-48
lines changed

packages/devtools_app/lib/src/screens/inspector_v2/inspector_screen_body.dart

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ import '../../shared/analytics/constants.dart' as gac;
1414
import '../../shared/analytics/metrics.dart';
1515
import '../../shared/console/eval/inspector_tree_v2.dart';
1616
import '../../shared/globals.dart';
17-
import '../../shared/managers/banner_messages.dart';
1817
import '../../shared/managers/error_badge_manager.dart';
1918
import '../../shared/primitives/blocking_action_mixin.dart';
2019
import '../../shared/ui/common_widgets.dart';
@@ -56,8 +55,6 @@ class InspectorScreenBodyState extends State<InspectorScreenBody>
5655
static const inspectorTreeKey = Key('Inspector Tree');
5756
static const minScreenWidthForText = 900.0;
5857

59-
static const _welcomeShownStorageId = 'inspectorV2WelcomeShown';
60-
6158
@override
6259
void initState() {
6360
super.initState();
@@ -107,12 +104,6 @@ class InspectorScreenBodyState extends State<InspectorScreenBody>
107104
}
108105

109106
_inspectorTreeController.setSearchTarget(searchTarget);
110-
111-
unawaited(
112-
_maybeShowWelcomeMessage(context).catchError((_) {
113-
// Ignore errors.
114-
}),
115-
);
116107
}
117108

118109
@override
@@ -223,16 +214,6 @@ class InspectorScreenBodyState extends State<InspectorScreenBody>
223214
}),
224215
);
225216
}
226-
227-
Future<void> _maybeShowWelcomeMessage(BuildContext context) async {
228-
final welcomeAlreadyShown = await storage.getValue(_welcomeShownStorageId);
229-
if (welcomeAlreadyShown == 'true') return;
230-
// Mark the welcome message as shown.
231-
await storage.setValue(_welcomeShownStorageId, 'true');
232-
if (context.mounted) {
233-
pushWelcomeToNewInspectorMessage(InspectorScreen.id);
234-
}
235-
}
236217
}
237218

238219
class InspectorTreeControls extends StatelessWidget {

packages/devtools_app/lib/src/shared/analytics/constants.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,6 @@ const inspectorHoverEvalMode = 'inspectorHoverEvalMode';
131131
const inspectorV2Enabled = 'inspectorV2Enabled';
132132
const inspectorV2Disabled = 'inspectorV2Disabled';
133133
const inspectorAutoRefreshEnabled = 'inspectorAutoRefreshEnabled';
134-
const inspectorV2Docs = 'inspectorV2Docs';
135134
const clearLogs = 'clearLogs';
136135
const copyLogs = 'copyLogs';
137136

packages/devtools_app/lib/src/shared/managers/banner_messages.dart

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -521,30 +521,6 @@ The $codeType DevTools debugger is in maintenance mode. For the best debugging e
521521
);
522522
}
523523

524-
class WelcomeToNewInspectorMessage extends BannerInfo {
525-
WelcomeToNewInspectorMessage({required super.screenId})
526-
: super(
527-
key: Key('WelcomeToNewInspectorMessage - $screenId'),
528-
buildTextSpans: (context) => [
529-
const TextSpan(
530-
text: '''
531-
👋 Welcome to the new Flutter inspector! To get started, check out the ''',
532-
),
533-
GaLinkTextSpan(
534-
link: GaLink(
535-
display: 'documentation',
536-
url: 'https://docs.flutter.dev/tools/devtools/inspector#new',
537-
gaScreenName: screenId,
538-
gaSelectedItemDescription: gac.inspectorV2Docs,
539-
),
540-
context: context,
541-
style: Theme.of(context).linkTextStyle,
542-
),
543-
const TextSpan(text: '.'),
544-
],
545-
);
546-
}
547-
548524
void maybePushDebugModePerformanceMessage(String screenId) {
549525
if (offlineDataController.showingOfflineData.value) return;
550526
if (serviceConnection.serviceManager.connectedApp?.isDebugFlutterAppNow ??
@@ -573,10 +549,6 @@ void pushDebuggerIdeRecommendationMessage(String screenId) {
573549
);
574550
}
575551

576-
void pushWelcomeToNewInspectorMessage(String screenId) {
577-
bannerMessages.addMessage(WelcomeToNewInspectorMessage(screenId: screenId));
578-
}
579-
580552
extension BannerMessageThemeExtension on ThemeData {
581553
TextStyle get warningMessageLinkStyle => regularTextStyle.copyWith(
582554
decoration: TextDecoration.underline,

0 commit comments

Comments
 (0)