Skip to content
This repository was archived by the owner on Apr 24, 2026. It is now read-only.

Commit 79b05ef

Browse files
Refactor(ui): complete Material icons migration in settings (#24)
- Replaced remaining `Icons.*` usages with `Symbols.*` in `behavior_settings_section.dart` and `notifications_settings_section.dart`. - Updated `CODEBASE.md` to indicate that the material icon migration in the UI is fully complete. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
1 parent 850a46f commit 79b05ef

3 files changed

Lines changed: 7 additions & 6 deletions

File tree

CODEBASE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
- Architecture follows `presentation -> domain -> data` with `get_it` + `provider`.
77
- Multi-platform targets in repo: Android, Linux, macOS, Windows, Web.
88
- Chat stack is decomposed into orchestrators plus focused cluster modules.
9-
- Material icon migration in UI/tests is mostly on `Symbols.*` (`material_symbols_icons`), with a small remaining `Icons.*` usage in notifications settings UI.
9+
- Material icon migration in UI is complete on `Symbols.*` (`material_symbols_icons`).
1010
- Theme system follows Material You (MD3): user-controlled theme mode, dynamic color toggle, AMOLED dark toggle, brand color seeds, contrast level, and responsive window size classes.
1111

1212
## Folder Structure

lib/presentation/pages/settings/sections/behavior_settings_section.dart

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import 'dart:async';
22

33
import 'package:flutter/material.dart';
4+
import 'package:material_symbols_icons/symbols.dart';
45
import 'package:provider/provider.dart';
56

67
import '../../../../core/constants/app_constants.dart';
@@ -160,7 +161,7 @@ class _BehaviorSettingsSectionState extends State<BehaviorSettingsSection> {
160161
: () => unawaited(
161162
settingsProvider.refreshOpenCodeBackedDefaults(),
162163
),
163-
icon: const Icon(Icons.refresh),
164+
icon: const Icon(Symbols.refresh),
164165
),
165166
],
166167
),
@@ -277,7 +278,7 @@ class _BehaviorSettingsSectionState extends State<BehaviorSettingsSection> {
277278
onPressed: settingsProvider.openCodeDefaultsLoading
278279
? null
279280
: () => unawaited(_applyUsername(context)),
280-
icon: const Icon(Icons.save),
281+
icon: const Icon(Symbols.save),
281282
label: const Text('Save username'),
282283
),
283284
),

lib/presentation/pages/settings/sections/notifications_settings_section.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -341,9 +341,9 @@ class _NotificationsSettingsSectionState
341341
null => Theme.of(context).colorScheme.onSurfaceVariant,
342342
};
343343
final statusIcon = switch (status) {
344-
true => Icons.check_circle_outline,
345-
false => Icons.warning_amber_rounded,
346-
null => Icons.help_outline,
344+
true => Symbols.check_circle,
345+
false => Symbols.warning,
346+
null => Symbols.help,
347347
};
348348
final statusText = switch (status) {
349349
true => 'Battery optimization is disabled for CodeWalk.',

0 commit comments

Comments
 (0)