diff --git a/lib/res/dimens.dart b/lib/res/dimens.dart index 430d0d5..0e03b22 100644 --- a/lib/res/dimens.dart +++ b/lib/res/dimens.dart @@ -7,6 +7,11 @@ class Dimens { static const double borderRadiusL = 24; static const double borderRadiusXL = 32; + static const double elevationLevel0 = 0; + static const double elevationLevel1 = 1; + static const double elevationLevel2 = 3; + static const double elevationLevel3 = 6; + static const double grid4 = 4; static const double grid8 = 8; static const double grid16 = 16; diff --git a/lib/res/theme.dart b/lib/res/theme.dart index 52e0e92..2ae0e13 100644 --- a/lib/res/theme.dart +++ b/lib/res/theme.dart @@ -29,14 +29,16 @@ ThemeData themeFrom(Color primaryColor, Brightness brightness) { primaryColor: primaryColor, colorScheme: scheme, appBarTheme: AppBarTheme( - elevation: 4, + elevation: Dimens.elevationLevel0, + scrolledUnderElevation: Dimens.elevationLevel2, color: scheme.surface, + foregroundColor: scheme.onBackground, surfaceTintColor: scheme.surfaceTint, ), cardTheme: CardTheme( clipBehavior: Clip.antiAlias, color: scheme.surface, - elevation: 4, + elevation: Dimens.elevationLevel1, margin: EdgeInsets.zero, shadowColor: Colors.transparent, shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(Dimens.borderRadiusL)), @@ -46,7 +48,7 @@ ThemeData themeFrom(Color primaryColor, Brightness brightness) { dialogTheme: DialogTheme( backgroundColor: scheme.surface, surfaceTintColor: scheme.surfaceTint, - elevation: 6, + elevation: Dimens.elevationLevel3, ), dividerColor: scheme.outlineVariant, dividerTheme: DividerThemeData( @@ -71,15 +73,19 @@ ThemeData themeFrom(Color primaryColor, Brightness brightness) { } ColorScheme _colorSchemeFromColor(Color primaryColor, Brightness brightness) { - final scheme = brightness == Brightness.light ? Scheme.light(primaryColor.value) : Scheme.dark(primaryColor.value); + final scheme = SchemeTonalSpot( + sourceColorHct: Hct.fromInt(primaryColor.value), + isDark: brightness == Brightness.dark, + contrastLevel: 0.0, + ); return ColorScheme( brightness: brightness, background: Color(scheme.background), - error: Color(scheme.error), - errorContainer: Color(scheme.errorContainer), onBackground: Color(scheme.onBackground), + error: Color(scheme.error), onError: Color(scheme.onError), + errorContainer: Color(scheme.errorContainer), onErrorContainer: Color(scheme.onErrorContainer), primary: Color(scheme.primary), onPrimary: Color(scheme.onPrimary), @@ -87,17 +93,30 @@ ColorScheme _colorSchemeFromColor(Color primaryColor, Brightness brightness) { onPrimaryContainer: Color(scheme.onPrimaryContainer), secondary: Color(scheme.secondary), onSecondary: Color(scheme.onSecondary), - surface: Color.alphaBlend( - Color(scheme.primary).withOpacity(0.05), - Color(scheme.background), - ), + secondaryContainer: Color(scheme.secondaryContainer), + onSecondaryContainer: Color(scheme.onSecondaryContainer), + tertiary: Color(scheme.tertiary), + onTertiary: Color(scheme.onTertiary), + tertiaryContainer: Color(scheme.tertiaryContainer), + onTertiaryContainer: Color(scheme.onTertiaryContainer), + surface: Color(scheme.surface), onSurface: Color(scheme.onSurface), - surfaceVariant: Color.alphaBlend( - Color(scheme.primary).withOpacity(0.5), - Color(scheme.background), - ), + surfaceVariant: Color(scheme.surfaceVariant), onSurfaceVariant: Color(scheme.onSurfaceVariant), outline: Color(scheme.outline), outlineVariant: Color(scheme.outlineVariant), + surfaceTint: Color(scheme.surfaceTint), + shadow: Color(scheme.shadow), + scrim: Color(scheme.scrim), ); } + +extension ElevatedSurfaceTheme on ColorScheme { + Color _surfaceWithElevation(double elevation) { + return ElevationOverlay.applySurfaceTint(surface, surfaceTint, elevation); + } + + Color get surfaceElevated1 => _surfaceWithElevation(Dimens.elevationLevel1); + Color get surfaceElevated2 => _surfaceWithElevation(Dimens.elevationLevel2); + Color get surfaceElevated3 => _surfaceWithElevation(Dimens.elevationLevel3); +} diff --git a/lib/screens/metering/components/bottom_controls/widget_bottom_controls.dart b/lib/screens/metering/components/bottom_controls/widget_bottom_controls.dart index 957d0b7..24179ba 100644 --- a/lib/screens/metering/components/bottom_controls/widget_bottom_controls.dart +++ b/lib/screens/metering/components/bottom_controls/widget_bottom_controls.dart @@ -28,12 +28,12 @@ class MeteringBottomControls extends StatelessWidget { Widget build(BuildContext context) { return BottomControlsBar( left: onSwitchEvSourceType != null - ? IconButton( + ? IconButton.filledTonal( onPressed: onSwitchEvSourceType, icon: Icon( UserPreferencesProvider.evSourceTypeOf(context) != EvSourceType.camera - ? Icons.camera_rear - : Icons.wb_incandescent, + ? Icons.camera_rear_outlined + : Icons.wb_incandescent_outlined, ), tooltip: UserPreferencesProvider.evSourceTypeOf(context) != EvSourceType.camera ? S.of(context).tooltipUseCamera @@ -46,9 +46,9 @@ class MeteringBottomControls extends StatelessWidget { onPressed: onMeasure, child: ev != null ? _EvValueText(ev: ev!, ev100: ev100!) : null, ), - right: IconButton( + right: IconButton.filledTonal( onPressed: onSettings, - icon: const Icon(Icons.settings), + icon: const Icon(Icons.settings_outlined), tooltip: S.of(context).tooltipOpenSettings, ), ); diff --git a/lib/screens/metering/components/camera_container/components/camera_controls/components/exposure_offset_slider/widget_slider_exposure_offset.dart b/lib/screens/metering/components/camera_container/components/camera_controls/components/exposure_offset_slider/widget_slider_exposure_offset.dart index bd1e71c..0730b53 100644 --- a/lib/screens/metering/components/camera_container/components/camera_controls/components/exposure_offset_slider/widget_slider_exposure_offset.dart +++ b/lib/screens/metering/components/camera_container/components/camera_controls/components/exposure_offset_slider/widget_slider_exposure_offset.dart @@ -21,7 +21,7 @@ class ExposureOffsetSlider extends StatelessWidget { range: range, value: value, onChanged: onChanged, - icon: Icons.light_mode, + icon: Icons.light_mode_outlined, defaultValue: 0, rulerValueAdapter: (value) => value.toStringSignedAsFixed(0), valueAdapter: (value) => S.of(context).evValue(value.toStringSignedAsFixed(1)), diff --git a/lib/screens/metering/components/camera_container/components/camera_controls/components/zoom_slider/widget_slider_zoom.dart b/lib/screens/metering/components/camera_container/components/camera_controls/components/zoom_slider/widget_slider_zoom.dart index 80cd4bd..316d547 100644 --- a/lib/screens/metering/components/camera_container/components/camera_controls/components/zoom_slider/widget_slider_zoom.dart +++ b/lib/screens/metering/components/camera_container/components/camera_controls/components/zoom_slider/widget_slider_zoom.dart @@ -32,7 +32,7 @@ class _ZoomSliderState extends State { range: widget.range, value: widget.value, onChanged: widget.onChanged, - icon: Icons.search, + icon: Icons.search_outlined, defaultValue: EquipmentProfiles.selectedOf(context).lensZoom, rulerValueAdapter: (value) => value.toStringAsFixed(0), valueAdapter: (value) => value.toZoom(), diff --git a/lib/screens/metering/components/camera_container/components/camera_controls_placeholder/widget_placeholder_camera_controls.dart b/lib/screens/metering/components/camera_container/components/camera_controls_placeholder/widget_placeholder_camera_controls.dart index cb4d76b..467a0b1 100644 --- a/lib/screens/metering/components/camera_container/components/camera_controls_placeholder/widget_placeholder_camera_controls.dart +++ b/lib/screens/metering/components/camera_container/components/camera_controls_placeholder/widget_placeholder_camera_controls.dart @@ -20,15 +20,12 @@ class CameraControlsPlaceholder extends StatelessWidget { children: [ IconButton( onPressed: onReset, - icon: Icon(error == CameraErrorType.permissionNotGranted ? Icons.settings : Icons.sync), + icon: Icon(error == CameraErrorType.permissionNotGranted ? Icons.settings_outlined : Icons.sync_outlined), ), const SizedBox(height: Dimens.grid8), Text( error.toStringLocalized(context), - style: Theme.of(context) - .textTheme - .bodyMedium - ?.copyWith(color: Theme.of(context).colorScheme.onBackground), + style: Theme.of(context).textTheme.bodyMedium?.copyWith(color: Theme.of(context).colorScheme.onBackground), textAlign: TextAlign.center, ), ], diff --git a/lib/screens/metering/components/camera_container/components/camera_preview/components/camera_view_placeholder/widget_placeholder_camera_view.dart b/lib/screens/metering/components/camera_container/components/camera_preview/components/camera_view_placeholder/widget_placeholder_camera_view.dart index a2914d2..4da8417 100644 --- a/lib/screens/metering/components/camera_container/components/camera_preview/components/camera_view_placeholder/widget_placeholder_camera_view.dart +++ b/lib/screens/metering/components/camera_container/components/camera_preview/components/camera_view_placeholder/widget_placeholder_camera_view.dart @@ -12,7 +12,7 @@ class CameraViewPlaceholder extends StatelessWidget { return Card( color: error != null ? null : Colors.black, shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(Dimens.borderRadiusM)), - child: Center(child: error != null ? const Icon(Icons.no_photography) : null), + child: Center(child: error != null ? const Icon(Icons.no_photography_outlined) : null), ); } } diff --git a/lib/screens/metering/components/shared/exposure_pairs_list/widget_list_exposure_pairs.dart b/lib/screens/metering/components/shared/exposure_pairs_list/widget_list_exposure_pairs.dart index 2f2188d..415892d 100644 --- a/lib/screens/metering/components/shared/exposure_pairs_list/widget_list_exposure_pairs.dart +++ b/lib/screens/metering/components/shared/exposure_pairs_list/widget_list_exposure_pairs.dart @@ -19,7 +19,7 @@ class ExposurePairsList extends StatelessWidget { duration: Dimens.switchDuration, child: exposurePairs.isEmpty ? IconPlaceholder( - icon: Icons.not_interested, + icon: Icons.not_interested_outlined, text: S.of(context).noExposurePairs, ) : Stack( diff --git a/lib/screens/metering/components/shared/metering_top_bar/widget_top_bar_metering.dart b/lib/screens/metering/components/shared/metering_top_bar/widget_top_bar_metering.dart index f1fef15..adc3c67 100644 --- a/lib/screens/metering/components/shared/metering_top_bar/widget_top_bar_metering.dart +++ b/lib/screens/metering/components/shared/metering_top_bar/widget_top_bar_metering.dart @@ -1,5 +1,6 @@ import 'package:flutter/material.dart'; import 'package:lightmeter/res/dimens.dart'; +import 'package:lightmeter/res/theme.dart'; import 'package:lightmeter/screens/metering/components/shared/metering_top_bar/shape_top_bar_metering.dart'; import 'package:lightmeter/screens/metering/components/shared/readings_container/widget_container_readings.dart'; @@ -19,7 +20,7 @@ class MeteringTopBar extends StatelessWidget { Widget build(BuildContext context) { return CustomPaint( painter: MeteringTopBarShape( - color: Theme.of(context).colorScheme.surface, + color: Theme.of(context).colorScheme.surfaceElevated1, appendixWidth: MediaQuery.of(context).size.width / 2 - Dimens.grid8 / 2 + Dimens.paddingM, appendixHeight: appendixHeight, ), diff --git a/lib/screens/metering/components/shared/readings_container/components/equipment_profile_picker/widget_picker_equipment_profiles.dart b/lib/screens/metering/components/shared/readings_container/components/equipment_profile_picker/widget_picker_equipment_profiles.dart index adbe1d2..d82cdf4 100644 --- a/lib/screens/metering/components/shared/readings_container/components/equipment_profile_picker/widget_picker_equipment_profiles.dart +++ b/lib/screens/metering/components/shared/readings_container/components/equipment_profile_picker/widget_picker_equipment_profiles.dart @@ -11,7 +11,7 @@ class EquipmentProfilePicker extends StatelessWidget { @override Widget build(BuildContext context) { return AnimatedDialogPicker( - icon: Icons.camera, + icon: Icons.camera_outlined, title: S.of(context).equipmentProfile, selectedValue: EquipmentProfiles.selectedOf(context), values: EquipmentProfiles.of(context), diff --git a/lib/screens/metering/components/shared/readings_container/components/film_picker/widget_picker_film.dart b/lib/screens/metering/components/shared/readings_container/components/film_picker/widget_picker_film.dart index 13a9366..6c6ccf1 100644 --- a/lib/screens/metering/components/shared/readings_container/components/film_picker/widget_picker_film.dart +++ b/lib/screens/metering/components/shared/readings_container/components/film_picker/widget_picker_film.dart @@ -13,7 +13,7 @@ class FilmPicker extends StatelessWidget { @override Widget build(BuildContext context) { return AnimatedDialogPicker( - icon: Icons.camera_roll, + icon: Icons.camera_roll_outlined, title: S.of(context).film, subtitle: S.of(context).filmReciprocityHint, selectedValue: Films.selectedOf(context), @@ -23,17 +23,14 @@ class FilmPicker extends StatelessWidget { closedChild: ReadingValueContainer.singleValue( value: ReadingValue( label: _label(context), - value: Films.selectedOf(context).name.isEmpty - ? S.of(context).none - : Films.selectedOf(context).name, + value: Films.selectedOf(context).name.isEmpty ? S.of(context).none : Films.selectedOf(context).name, ), ), ); } String _label(BuildContext context) { - if (Films.selectedOf(context) == const Film.other() || - Films.selectedOf(context).iso == selectedIso.value) { + if (Films.selectedOf(context) == const Film.other() || Films.selectedOf(context).iso == selectedIso.value) { return S.of(context).film; } diff --git a/lib/screens/metering/components/shared/readings_container/components/iso_picker/widget_picker_iso.dart b/lib/screens/metering/components/shared/readings_container/components/iso_picker/widget_picker_iso.dart index a85b30c..4eba200 100644 --- a/lib/screens/metering/components/shared/readings_container/components/iso_picker/widget_picker_iso.dart +++ b/lib/screens/metering/components/shared/readings_container/components/iso_picker/widget_picker_iso.dart @@ -18,16 +18,15 @@ class IsoValuePicker extends StatelessWidget { @override Widget build(BuildContext context) { return AnimatedDialogPicker( - icon: Icons.iso, + icon: Icons.iso_outlined, title: S.of(context).iso, subtitle: S.of(context).filmSpeed, selectedValue: selectedValue, values: values, itemTitleBuilder: (_, value) => Text(value.value.toString()), // using ascending order, because increase in film speed rises EV - itemTrailingBuilder: (selected, value) => value.value != selected.value - ? Text(S.of(context).evValue(selected.toStringDifference(value))) - : null, + itemTrailingBuilder: (selected, value) => + value.value != selected.value ? Text(S.of(context).evValue(selected.toStringDifference(value))) : null, onChanged: onChanged, closedChild: ReadingValueContainer.singleValue( value: ReadingValue( diff --git a/lib/screens/metering/components/shared/readings_container/components/nd_picker/widget_picker_nd.dart b/lib/screens/metering/components/shared/readings_container/components/nd_picker/widget_picker_nd.dart index fdd8f0a..fa28bd4 100644 --- a/lib/screens/metering/components/shared/readings_container/components/nd_picker/widget_picker_nd.dart +++ b/lib/screens/metering/components/shared/readings_container/components/nd_picker/widget_picker_nd.dart @@ -18,7 +18,7 @@ class NdValuePicker extends StatelessWidget { @override Widget build(BuildContext context) { return AnimatedDialogPicker( - icon: Icons.filter_b_and_w, + icon: Icons.filter_b_and_w_outlined, title: S.of(context).nd, subtitle: S.of(context).ndFilterFactor, selectedValue: selectedValue, diff --git a/lib/screens/metering/components/shared/readings_container/components/shared/reading_value_container/widget_container_reading_value.dart b/lib/screens/metering/components/shared/readings_container/components/shared/reading_value_container/widget_container_reading_value.dart index c35d5d6..432213b 100644 --- a/lib/screens/metering/components/shared/readings_container/components/shared/reading_value_container/widget_container_reading_value.dart +++ b/lib/screens/metering/components/shared/readings_container/components/shared/reading_value_container/widget_container_reading_value.dart @@ -47,31 +47,31 @@ class ReadingValueContainer extends StatelessWidget implements AnimatedDialogClo @override Widget build(BuildContext context) { - return ClipRRect( - borderRadius: BorderRadius.circular(Dimens.borderRadiusM), - child: ColoredBox( + return DecoratedBox( + decoration: BoxDecoration( + borderRadius: BorderRadius.circular(Dimens.borderRadiusM), color: backgroundColor(context), - child: Padding( - padding: const EdgeInsets.all(Dimens.paddingM), - child: Stack( - children: [ - if (locked) - Positioned( - top: 0, - right: 0, - child: Icon( - Icons.lock, - size: Theme.of(context).textTheme.labelMedium!.fontSize, - color: Theme.of(context).colorScheme.onPrimaryContainer, - ), + ), + child: Padding( + padding: const EdgeInsets.all(Dimens.paddingM), + child: Stack( + children: [ + if (locked) + Positioned( + top: 0, + right: 0, + child: Icon( + Icons.lock_outlined, + size: Theme.of(context).textTheme.labelMedium!.fontSize, + color: Theme.of(context).colorScheme.onPrimaryContainer, ), - Column( - crossAxisAlignment: CrossAxisAlignment.start, - mainAxisSize: MainAxisSize.min, - children: _items, ), - ], - ), + Column( + crossAxisAlignment: CrossAxisAlignment.start, + mainAxisSize: MainAxisSize.min, + children: _items, + ), + ], ), ), ); diff --git a/lib/screens/metering/screen_metering.dart b/lib/screens/metering/screen_metering.dart index 2cd828c..12f910d 100644 --- a/lib/screens/metering/screen_metering.dart +++ b/lib/screens/metering/screen_metering.dart @@ -24,7 +24,6 @@ class MeteringScreen extends StatelessWidget { Widget build(BuildContext context) { return _InheritedListeners( child: Scaffold( - backgroundColor: Theme.of(context).colorScheme.background, body: Column( children: [ Expanded( diff --git a/lib/screens/settings/components/about/components/report_issue/widget_list_tile_report_issue.dart b/lib/screens/settings/components/about/components/report_issue/widget_list_tile_report_issue.dart index b737791..f7e4d44 100644 --- a/lib/screens/settings/components/about/components/report_issue/widget_list_tile_report_issue.dart +++ b/lib/screens/settings/components/about/components/report_issue/widget_list_tile_report_issue.dart @@ -9,7 +9,7 @@ class ReportIssueListTile extends StatelessWidget { @override Widget build(BuildContext context) { return ListTile( - leading: const Icon(Icons.bug_report), + leading: const Icon(Icons.bug_report_outlined), title: Text(S.of(context).reportIssue), onTap: () { launchUrl( diff --git a/lib/screens/settings/components/about/components/restore_purchases/widget_list_tile_restore_purchases.dart b/lib/screens/settings/components/about/components/restore_purchases/widget_list_tile_restore_purchases.dart index 8bbbdea..b07257e 100644 --- a/lib/screens/settings/components/about/components/restore_purchases/widget_list_tile_restore_purchases.dart +++ b/lib/screens/settings/components/about/components/restore_purchases/widget_list_tile_restore_purchases.dart @@ -8,7 +8,7 @@ class RestorePurchasesListTile extends StatelessWidget { @override Widget build(BuildContext context) { return ListTile( - leading: const Icon(Icons.restore), + leading: const Icon(Icons.restore_outlined), title: Text(S.of(context).restorePurchases), onTap: IAPProductsProvider.of(context).restorePurchases, ); diff --git a/lib/screens/settings/components/about/components/source_code/widget_list_tile_source_code.dart b/lib/screens/settings/components/about/components/source_code/widget_list_tile_source_code.dart index ad40ce5..cc63336 100644 --- a/lib/screens/settings/components/about/components/source_code/widget_list_tile_source_code.dart +++ b/lib/screens/settings/components/about/components/source_code/widget_list_tile_source_code.dart @@ -9,7 +9,7 @@ class SourceCodeListTile extends StatelessWidget { @override Widget build(BuildContext context) { return ListTile( - leading: const Icon(Icons.code), + leading: const Icon(Icons.code_outlined), title: Text(S.of(context).sourceCode), onTap: () { launchUrl( diff --git a/lib/screens/settings/components/about/components/write_email/widget_list_tile_write_email.dart b/lib/screens/settings/components/about/components/write_email/widget_list_tile_write_email.dart index 4d2ee30..aab959f 100644 --- a/lib/screens/settings/components/about/components/write_email/widget_list_tile_write_email.dart +++ b/lib/screens/settings/components/about/components/write_email/widget_list_tile_write_email.dart @@ -10,7 +10,7 @@ class WriteEmailListTile extends StatelessWidget { @override Widget build(BuildContext context) { return ListTile( - leading: const Icon(Icons.email), + leading: const Icon(Icons.email_outlined), title: Text(S.of(context).writeEmail), onTap: () { final mailToUrl = Uri.parse('mailto:$contactEmail?subject=M3 Lightmeter'); diff --git a/lib/screens/settings/components/general/components/caffeine/widget_list_tile_caffeine.dart b/lib/screens/settings/components/general/components/caffeine/widget_list_tile_caffeine.dart index ee0483d..502df78 100644 --- a/lib/screens/settings/components/general/components/caffeine/widget_list_tile_caffeine.dart +++ b/lib/screens/settings/components/general/components/caffeine/widget_list_tile_caffeine.dart @@ -12,7 +12,7 @@ class CaffeineListTile extends StatelessWidget { Widget build(BuildContext context) { return BlocBuilder( builder: (context, state) => SwitchListTile( - secondary: const Icon(Icons.screen_lock_portrait), + secondary: const Icon(Icons.screen_lock_portrait_outlined), title: Text(S.of(context).keepScreenOn), value: state, onChanged: context.read().onCaffeineChanged, diff --git a/lib/screens/settings/components/general/components/haptics/widget_list_tile_haptics.dart b/lib/screens/settings/components/general/components/haptics/widget_list_tile_haptics.dart index ec7ec60..e16d749 100644 --- a/lib/screens/settings/components/general/components/haptics/widget_list_tile_haptics.dart +++ b/lib/screens/settings/components/general/components/haptics/widget_list_tile_haptics.dart @@ -12,7 +12,7 @@ class HapticsListTile extends StatelessWidget { Widget build(BuildContext context) { return BlocBuilder( builder: (context, state) => SwitchListTile( - secondary: const Icon(Icons.vibration), + secondary: const Icon(Icons.vibration_outlined), title: Text(S.of(context).haptics), value: state, onChanged: context.read().onHapticsChanged, diff --git a/lib/screens/settings/components/general/components/language/widget_list_tile_language.dart b/lib/screens/settings/components/general/components/language/widget_list_tile_language.dart index eeaaf4c..14d8971 100644 --- a/lib/screens/settings/components/general/components/language/widget_list_tile_language.dart +++ b/lib/screens/settings/components/general/components/language/widget_list_tile_language.dart @@ -10,14 +10,14 @@ class LanguageListTile extends StatelessWidget { @override Widget build(BuildContext context) { return ListTile( - leading: const Icon(Icons.language), + leading: const Icon(Icons.language_outlined), title: Text(S.of(context).language), trailing: Text(UserPreferencesProvider.localeOf(context).localizedName), onTap: () { showDialog( context: context, builder: (_) => DialogPicker( - icon: Icons.language, + icon: Icons.language_outlined, title: S.of(context).chooseLanguage, selectedValue: UserPreferencesProvider.localeOf(context), values: SupportedLocale.values, diff --git a/lib/screens/settings/components/general/components/volume_actions/widget_list_tile_volume_actions.dart b/lib/screens/settings/components/general/components/volume_actions/widget_list_tile_volume_actions.dart index 0502b60..1bea42e 100644 --- a/lib/screens/settings/components/general/components/volume_actions/widget_list_tile_volume_actions.dart +++ b/lib/screens/settings/components/general/components/volume_actions/widget_list_tile_volume_actions.dart @@ -11,7 +11,7 @@ class VolumeActionsListTile extends StatelessWidget { Widget build(BuildContext context) { return BlocBuilder( builder: (context, state) => SwitchListTile( - secondary: const Icon(Icons.volume_up), + secondary: const Icon(Icons.volume_up_outlined), title: Text(S.of(context).volumeKeysAction), value: state, onChanged: context.read().onVolumeActionChanged, diff --git a/lib/screens/settings/components/general/widget_settings_section_general.dart b/lib/screens/settings/components/general/widget_settings_section_general.dart index 278d342..a1ac95c 100644 --- a/lib/screens/settings/components/general/widget_settings_section_general.dart +++ b/lib/screens/settings/components/general/widget_settings_section_general.dart @@ -17,9 +17,9 @@ class GeneralSettingsSection extends StatelessWidget { return SettingsSection( title: S.of(context).general, children: [ + const TimerListTileProvider(), const CaffeineListTileProvider(), const HapticsListTileProvider(), - const TimerListTileProvider(), if (Platform.isAndroid) const VolumeActionsListTileProvider(), const LanguageListTile(), ], diff --git a/lib/screens/settings/components/lightmeter_pro/components/buy_pro/widget_list_tile_buy_pro.dart b/lib/screens/settings/components/lightmeter_pro/components/buy_pro/widget_list_tile_buy_pro.dart index 4bb401d..cc026a9 100644 --- a/lib/screens/settings/components/lightmeter_pro/components/buy_pro/widget_list_tile_buy_pro.dart +++ b/lib/screens/settings/components/lightmeter_pro/components/buy_pro/widget_list_tile_buy_pro.dart @@ -12,7 +12,7 @@ class BuyProListTile extends StatelessWidget { final status = IAPProducts.productOf(context, IAPProductType.paidFeatures)?.status; final isPending = status == IAPProductStatus.purchased || status == null; return ListTile( - leading: const Icon(Icons.star), + leading: const Icon(Icons.star_outlined), title: Text(S.of(context).unlockProFeatures), onTap: !isPending ? () { diff --git a/lib/screens/settings/components/metering/components/calibration/components/calibration_dialog/widget_dialog_calibration.dart b/lib/screens/settings/components/metering/components/calibration/components/calibration_dialog/widget_dialog_calibration.dart index 57b1316..639b79c 100644 --- a/lib/screens/settings/components/metering/components/calibration/components/calibration_dialog/widget_dialog_calibration.dart +++ b/lib/screens/settings/components/metering/components/calibration/components/calibration_dialog/widget_dialog_calibration.dart @@ -16,7 +16,7 @@ class CalibrationDialog extends StatelessWidget { Widget build(BuildContext context) { final bool hasLightSensor = ServicesProvider.of(context).environment.hasLightSensor; return AlertDialog( - icon: const Icon(Icons.settings_brightness), + icon: const Icon(Icons.settings_brightness_outlined), titlePadding: Dimens.dialogIconTitlePadding, title: Text(S.of(context).calibration), contentPadding: const EdgeInsets.symmetric(horizontal: Dimens.paddingL), @@ -25,38 +25,27 @@ class CalibrationDialog extends StatelessWidget { mainAxisSize: MainAxisSize.min, children: [ Text( - hasLightSensor - ? S.of(context).calibrationMessage - : S.of(context).calibrationMessageCameraOnly, + hasLightSensor ? S.of(context).calibrationMessage : S.of(context).calibrationMessageCameraOnly, ), const SizedBox(height: Dimens.grid16), BlocBuilder( - buildWhen: (previous, current) => - previous.cameraEvCalibration != current.cameraEvCalibration, + buildWhen: (previous, current) => previous.cameraEvCalibration != current.cameraEvCalibration, builder: (context, state) => _CalibrationUnit( title: S.of(context).camera, value: state.cameraEvCalibration, - onChanged: (value) => context - .read() - .add(CameraEvCalibrationChangedEvent(value)), - onReset: () => context - .read() - .add(const CameraEvCalibrationResetEvent()), + onChanged: (value) => context.read().add(CameraEvCalibrationChangedEvent(value)), + onReset: () => context.read().add(const CameraEvCalibrationResetEvent()), ), ), if (hasLightSensor) BlocBuilder( - buildWhen: (previous, current) => - previous.lightSensorEvCalibration != current.lightSensorEvCalibration, + buildWhen: (previous, current) => previous.lightSensorEvCalibration != current.lightSensorEvCalibration, builder: (context, state) => _CalibrationUnit( title: S.of(context).lightSensor, value: state.lightSensorEvCalibration, - onChanged: (value) => context - .read() - .add(LightSensorEvCalibrationChangedEvent(value)), - onReset: () => context - .read() - .add(const LightSensorEvCalibrationResetEvent()), + onChanged: (value) => + context.read().add(LightSensorEvCalibrationChangedEvent(value)), + onReset: () => context.read().add(const LightSensorEvCalibrationResetEvent()), ), ), ], @@ -116,7 +105,7 @@ class _CalibrationUnit extends StatelessWidget { ), IconButton( onPressed: onReset, - icon: const Icon(Icons.sync), + icon: const Icon(Icons.sync_outlined), tooltip: S.of(context).tooltipResetToZero, ), ], diff --git a/lib/screens/settings/components/metering/components/calibration/widget_list_tile_calibration.dart b/lib/screens/settings/components/metering/components/calibration/widget_list_tile_calibration.dart index 61690c4..617d2e5 100644 --- a/lib/screens/settings/components/metering/components/calibration/widget_list_tile_calibration.dart +++ b/lib/screens/settings/components/metering/components/calibration/widget_list_tile_calibration.dart @@ -9,7 +9,7 @@ class CalibrationListTile extends StatelessWidget { @override Widget build(BuildContext context) { return ListTile( - leading: const Icon(Icons.settings_brightness), + leading: const Icon(Icons.settings_brightness_outlined), title: Text(S.of(context).calibration), onTap: () { showDialog( diff --git a/lib/screens/settings/components/metering/components/camera_features/widget_list_tile_camera_features.dart b/lib/screens/settings/components/metering/components/camera_features/widget_list_tile_camera_features.dart index 1446be3..c779912 100644 --- a/lib/screens/settings/components/metering/components/camera_features/widget_list_tile_camera_features.dart +++ b/lib/screens/settings/components/metering/components/camera_features/widget_list_tile_camera_features.dart @@ -11,13 +11,13 @@ class CameraFeaturesListTile extends StatelessWidget { @override Widget build(BuildContext context) { return IAPListTile( - leading: const Icon(Icons.camera_alt), + leading: const Icon(Icons.camera_alt_outlined), title: Text(S.of(context).cameraFeatures), onTap: () { showDialog( context: context, builder: (_) => DialogSwitch( - icon: Icons.layers_outlined, + icon: Icons.camera_alt_outlined, title: S.of(context).cameraFeatures, values: UserPreferencesProvider.cameraConfigOf(context), titleAdapter: (context, feature) { diff --git a/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_container/widget_container_equipment_profile.dart b/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_container/widget_container_equipment_profile.dart index 8504c67..72ba238 100644 --- a/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_container/widget_container_equipment_profile.dart +++ b/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_container/widget_container_equipment_profile.dart @@ -171,7 +171,7 @@ class _AnimatedNameLeading extends AnimatedWidget { return Padding( padding: EdgeInsets.only(right: _progress.value * Dimens.grid8), child: Icon( - Icons.edit, + Icons.edit_outlined, size: _progress.value * Dimens.grid24, ), ); @@ -194,7 +194,7 @@ class _AnimatedArrowButton extends AnimatedWidget { onPressed: onPressed, icon: Transform.rotate( angle: _progress.value * pi, - child: const Icon(Icons.keyboard_arrow_down), + child: const Icon(Icons.keyboard_arrow_down_outlined), ), tooltip: _progress.value == 0 ? S.of(context).tooltipExpand : S.of(context).tooltipCollapse, ); @@ -239,7 +239,7 @@ class _AnimatedEquipmentListTiles extends AnimatedWidget { child: Column( children: [ FilterListTile( - icon: Icons.iso, + icon: Icons.iso_outlined, title: S.of(context).isoValues, description: S.of(context).isoValuesFilterDescription, values: IsoValue.values, @@ -247,7 +247,7 @@ class _AnimatedEquipmentListTiles extends AnimatedWidget { onChanged: onIsoValuesSelecred, ), FilterListTile( - icon: Icons.filter_b_and_w, + icon: Icons.filter_b_and_w_outlined, title: S.of(context).ndFilters, description: S.of(context).ndFiltersFilterDescription, values: NdValue.values, @@ -255,7 +255,7 @@ class _AnimatedEquipmentListTiles extends AnimatedWidget { onChanged: onNdValuesSelected, ), RangePickerListTile( - icon: Icons.camera, + icon: Icons.camera_outlined, title: S.of(context).apertureValues, description: S.of(context).apertureValuesFilterDescription, values: ApertureValue.values, @@ -263,7 +263,7 @@ class _AnimatedEquipmentListTiles extends AnimatedWidget { onChanged: onApertureValuesSelected, ), RangePickerListTile( - icon: Icons.shutter_speed, + icon: Icons.shutter_speed_outlined, title: S.of(context).shutterSpeedValues, description: S.of(context).shutterSpeedValuesFilterDescription, values: ShutterSpeedValue.values, @@ -275,7 +275,7 @@ class _AnimatedEquipmentListTiles extends AnimatedWidget { value.value == 1 ? S.of(context).shutterSpeedManual : value.toString(), ), SliderPickerListTile( - icon: Icons.zoom_in, + icon: Icons.zoom_in_outlined, title: S.of(context).lensZoom, description: S.of(context).lensZoomDescription, value: equipmentData.lensZoom, @@ -291,12 +291,12 @@ class _AnimatedEquipmentListTiles extends AnimatedWidget { children: [ IconButton( onPressed: onCopy, - icon: const Icon(Icons.copy), + icon: const Icon(Icons.copy_outlined), tooltip: S.of(context).tooltipCopy, ), IconButton( onPressed: onDelete, - icon: const Icon(Icons.delete), + icon: const Icon(Icons.delete_outlined), tooltip: S.of(context).tooltipDelete, ), ], diff --git a/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_name_dialog/widget_dialog_equipment_profile_name.dart b/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_name_dialog/widget_dialog_equipment_profile_name.dart index e3014ac..fa6556b 100644 --- a/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_name_dialog/widget_dialog_equipment_profile_name.dart +++ b/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/components/equipment_profile_name_dialog/widget_dialog_equipment_profile_name.dart @@ -23,7 +23,7 @@ class _EquipmentProfileNameDialogState extends State @override Widget build(BuildContext context) { return AlertDialog( - icon: const Icon(Icons.edit), + icon: const Icon(Icons.edit_outlined), titlePadding: Dimens.dialogIconTitlePadding, title: Text(S.of(context).equipmentProfileName), content: TextField( diff --git a/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/screen_equipment_profile.dart b/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/screen_equipment_profile.dart index 9f3cf5a..b07b767 100644 --- a/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/screen_equipment_profile.dart +++ b/lib/screens/settings/components/metering/components/equipment_profiles/components/equipment_profile_screen/screen_equipment_profile.dart @@ -32,7 +32,7 @@ class _EquipmentProfilesScreenState extends State { appBarActions: [ IconButton( onPressed: _addProfile, - icon: const Icon(Icons.add), + icon: const Icon(Icons.add_outlined), tooltip: S.of(context).tooltipAdd, ), ], @@ -150,7 +150,7 @@ class _EquipmentProfilesListPlaceholder extends StatelessWidget { child: Padding( padding: const EdgeInsets.all(Dimens.paddingL), child: IconPlaceholder( - icon: Icons.add, + icon: Icons.add_outlined, text: S.of(context).tapToAdd, ), ), diff --git a/lib/screens/settings/components/metering/components/equipment_profiles/widget_list_tile_equipment_profiles.dart b/lib/screens/settings/components/metering/components/equipment_profiles/widget_list_tile_equipment_profiles.dart index 740918f..610dd59 100644 --- a/lib/screens/settings/components/metering/components/equipment_profiles/widget_list_tile_equipment_profiles.dart +++ b/lib/screens/settings/components/metering/components/equipment_profiles/widget_list_tile_equipment_profiles.dart @@ -10,7 +10,7 @@ class EquipmentProfilesListTile extends StatelessWidget { @override Widget build(BuildContext context) { return IAPListTile( - leading: const Icon(Icons.camera), + leading: const Icon(Icons.camera_outlined), title: Text(S.of(context).equipmentProfiles), onTap: () { Navigator.of(context).push( diff --git a/lib/screens/settings/components/metering/components/films/widget_list_tile_films.dart b/lib/screens/settings/components/metering/components/films/widget_list_tile_films.dart index 72ff433..4eaf017 100644 --- a/lib/screens/settings/components/metering/components/films/widget_list_tile_films.dart +++ b/lib/screens/settings/components/metering/components/films/widget_list_tile_films.dart @@ -11,13 +11,13 @@ class FilmsListTile extends StatelessWidget { @override Widget build(BuildContext context) { return IAPListTile( - leading: const Icon(Icons.camera_roll), + leading: const Icon(Icons.camera_roll_outlined), title: Text(S.of(context).filmsInUse), onTap: () { showDialog>( context: context, builder: (_) => DialogFilter( - icon: const Icon(Icons.camera_roll), + icon: const Icon(Icons.camera_roll_outlined), title: S.of(context).filmsInUse, description: S.of(context).filmsInUseDescription, values: Films.of(context).sublist(1), diff --git a/lib/screens/settings/components/metering/components/fractional_stops/widget_list_tile_fractional_stops.dart b/lib/screens/settings/components/metering/components/fractional_stops/widget_list_tile_fractional_stops.dart index e8b99d1..90ab875 100644 --- a/lib/screens/settings/components/metering/components/fractional_stops/widget_list_tile_fractional_stops.dart +++ b/lib/screens/settings/components/metering/components/fractional_stops/widget_list_tile_fractional_stops.dart @@ -10,14 +10,14 @@ class StopTypeListTile extends StatelessWidget { @override Widget build(BuildContext context) { return ListTile( - leading: const Icon(Icons.straighten), + leading: const Icon(Icons.straighten_outlined), title: Text(S.of(context).fractionalStops), trailing: Text(_typeToString(context, UserPreferencesProvider.stopTypeOf(context))), onTap: () { showDialog( context: context, builder: (_) => DialogPicker( - icon: Icons.straighten, + icon: Icons.straighten_outlined, title: S.of(context).showFractionalStops, selectedValue: UserPreferencesProvider.stopTypeOf(context), values: StopType.values, diff --git a/lib/screens/settings/components/metering/components/show_ev_100/widget_list_tile_show_ev_100.dart b/lib/screens/settings/components/metering/components/show_ev_100/widget_list_tile_show_ev_100.dart index a92623e..e0701f4 100644 --- a/lib/screens/settings/components/metering/components/show_ev_100/widget_list_tile_show_ev_100.dart +++ b/lib/screens/settings/components/metering/components/show_ev_100/widget_list_tile_show_ev_100.dart @@ -13,7 +13,7 @@ class ShowEv100ListTile extends StatelessWidget { return Disable( disable: !context.isPro, child: SwitchListTile( - secondary: const Icon(Icons.adjust), + secondary: const Icon(Icons.adjust_outlined), title: Text(S.of(context).showEv100), value: context.isPro && UserPreferencesProvider.showEv100Of(context), onChanged: (_) => UserPreferencesProvider.of(context).toggleShowEv100(), diff --git a/lib/screens/settings/components/shared/dialog_filter/widget_dialog_filter.dart b/lib/screens/settings/components/shared/dialog_filter/widget_dialog_filter.dart index 374c8d7..7294505 100644 --- a/lib/screens/settings/components/shared/dialog_filter/widget_dialog_filter.dart +++ b/lib/screens/settings/components/shared/dialog_filter/widget_dialog_filter.dart @@ -109,7 +109,7 @@ class _DialogFilterState extends State> { width: 40, child: IconButton( padding: EdgeInsets.zero, - icon: Icon(_hasAnyUnselected ? Icons.select_all : Icons.deselect), + icon: Icon(_hasAnyUnselected ? Icons.select_all_outlined : Icons.deselect_outlined), onPressed: _toggleAll, tooltip: _hasAnyUnselected ? S.of(context).tooltipSelectAll : S.of(context).tooltipDesecelectAll, ), diff --git a/lib/screens/settings/components/theme/components/dynamic_color/widget_list_tile_dynamic_color.dart b/lib/screens/settings/components/theme/components/dynamic_color/widget_list_tile_dynamic_color.dart index 6fc5a6c..3b6ab72 100644 --- a/lib/screens/settings/components/theme/components/dynamic_color/widget_list_tile_dynamic_color.dart +++ b/lib/screens/settings/components/theme/components/dynamic_color/widget_list_tile_dynamic_color.dart @@ -10,7 +10,7 @@ class DynamicColorListTile extends StatelessWidget { @override Widget build(BuildContext context) { return SwitchListTile( - secondary: const Icon(Icons.colorize), + secondary: const Icon(Icons.colorize_outlined), title: Text(S.of(context).dynamicColor), value: UserPreferencesProvider.dynamicColorStateOf(context) == DynamicColorState.enabled, onChanged: UserPreferencesProvider.of(context).enableDynamicColor, diff --git a/lib/screens/settings/components/theme/components/primary_color/components/primary_color_picker_dialog/widget_dialog_picker_primary_color.dart b/lib/screens/settings/components/theme/components/primary_color/components/primary_color_picker_dialog/widget_dialog_picker_primary_color.dart index 380faa1..be9e8ad 100644 --- a/lib/screens/settings/components/theme/components/primary_color/components/primary_color_picker_dialog/widget_dialog_picker_primary_color.dart +++ b/lib/screens/settings/components/theme/components/primary_color/components/primary_color_picker_dialog/widget_dialog_picker_primary_color.dart @@ -24,7 +24,7 @@ class _PrimaryColorDialogPickerState extends State { @override Widget build(BuildContext context) { return AlertDialog( - icon: const Icon(Icons.palette), + icon: const Icon(Icons.palette_outlined), titlePadding: Dimens.dialogIconTitlePadding, title: Text(S.of(context).choosePrimaryColor), content: SizedBox( @@ -124,7 +124,7 @@ class _SelectableColorItemState extends State<_SelectableColorItem> { duration: Dimens.durationS, child: widget.selected ? Icon( - Icons.check, + Icons.check_outlined, color: ThemeData.estimateBrightnessForColor(widget.color) == Brightness.light ? Colors.black : Colors.white, diff --git a/lib/screens/settings/components/theme/components/primary_color/widget_list_tile_primary_color.dart b/lib/screens/settings/components/theme/components/primary_color/widget_list_tile_primary_color.dart index 1cf8762..6560d4c 100644 --- a/lib/screens/settings/components/theme/components/primary_color/widget_list_tile_primary_color.dart +++ b/lib/screens/settings/components/theme/components/primary_color/widget_list_tile_primary_color.dart @@ -13,13 +13,13 @@ class PrimaryColorListTile extends StatelessWidget { if (UserPreferencesProvider.dynamicColorStateOf(context) == DynamicColorState.enabled) { return Disable( child: ListTile( - leading: const Icon(Icons.palette), + leading: const Icon(Icons.palette_outlined), title: Text(S.of(context).primaryColor), ), ); } return ListTile( - leading: const Icon(Icons.palette), + leading: const Icon(Icons.palette_outlined), title: Text(S.of(context).primaryColor), onTap: () { showDialog( diff --git a/lib/screens/settings/components/theme/components/theme_type/widget_list_tile_theme_type.dart b/lib/screens/settings/components/theme/components/theme_type/widget_list_tile_theme_type.dart index cfcd694..a74464c 100644 --- a/lib/screens/settings/components/theme/components/theme_type/widget_list_tile_theme_type.dart +++ b/lib/screens/settings/components/theme/components/theme_type/widget_list_tile_theme_type.dart @@ -10,14 +10,14 @@ class ThemeTypeListTile extends StatelessWidget { @override Widget build(BuildContext context) { return ListTile( - leading: const Icon(Icons.brightness_6), + leading: const Icon(Icons.brightness_6_outlined), title: Text(S.of(context).theme), trailing: Text(_typeToString(context, UserPreferencesProvider.themeTypeOf(context))), onTap: () { showDialog( context: context, builder: (_) => DialogPicker( - icon: Icons.brightness_6, + icon: Icons.brightness_6_outlined, title: S.of(context).chooseTheme, selectedValue: UserPreferencesProvider.themeTypeOf(context), values: ThemeType.values, diff --git a/lib/screens/shared/bottom_controls_bar/widget_bottom_controls_bar.dart b/lib/screens/shared/bottom_controls_bar/widget_bottom_controls_bar.dart index bbe9c98..a55532a 100644 --- a/lib/screens/shared/bottom_controls_bar/widget_bottom_controls_bar.dart +++ b/lib/screens/shared/bottom_controls_bar/widget_bottom_controls_bar.dart @@ -1,5 +1,6 @@ import 'package:flutter/material.dart'; import 'package:lightmeter/res/dimens.dart'; +import 'package:lightmeter/res/theme.dart'; class BottomControlsBar extends StatelessWidget { final Widget center; @@ -15,40 +16,27 @@ class BottomControlsBar extends StatelessWidget { @override Widget build(BuildContext context) { - final scheme = Theme.of(context).colorScheme; - return IconButtonTheme( - data: IconButtonThemeData( - style: ButtonStyle( - backgroundColor: MaterialStatePropertyAll(scheme.surface), - elevation: const MaterialStatePropertyAll(4), - iconColor: MaterialStatePropertyAll(scheme.onSurface), - shadowColor: const MaterialStatePropertyAll(Colors.transparent), - surfaceTintColor: MaterialStatePropertyAll(scheme.surfaceTint), - fixedSize: const MaterialStatePropertyAll(Size(Dimens.grid48, Dimens.grid48)), - ), + return ClipRRect( + borderRadius: const BorderRadius.only( + topLeft: Radius.circular(Dimens.borderRadiusL), + topRight: Radius.circular(Dimens.borderRadiusL), ), - child: ClipRRect( - borderRadius: const BorderRadius.only( - topLeft: Radius.circular(Dimens.borderRadiusL), - topRight: Radius.circular(Dimens.borderRadiusL), - ), - child: ColoredBox( - color: Theme.of(context).colorScheme.surface, - child: SafeArea( - top: false, - child: Padding( - padding: const EdgeInsets.symmetric(vertical: Dimens.paddingL), - child: Row( - mainAxisAlignment: MainAxisAlignment.spaceEvenly, - children: [ - if (left != null) Expanded(child: Center(child: left)) else const Spacer(), - SizedBox.fromSize( - size: const Size.square(Dimens.grid72), - child: center, - ), - if (right != null) Expanded(child: Center(child: right)) else const Spacer(), - ], - ), + child: ColoredBox( + color: Theme.of(context).colorScheme.surfaceElevated1, + child: SafeArea( + top: false, + child: Padding( + padding: const EdgeInsets.symmetric(vertical: Dimens.paddingL), + child: Row( + mainAxisAlignment: MainAxisAlignment.spaceEvenly, + children: [ + if (left != null) Expanded(child: Center(child: left)) else const Spacer(), + SizedBox.fromSize( + size: const Size.square(Dimens.grid72), + child: center, + ), + if (right != null) Expanded(child: Center(child: right)) else const Spacer(), + ], ), ), ), diff --git a/lib/screens/shared/close_button/widget_button_close.dart b/lib/screens/shared/close_button/widget_button_close.dart deleted file mode 100644 index 4d52678..0000000 --- a/lib/screens/shared/close_button/widget_button_close.dart +++ /dev/null @@ -1,15 +0,0 @@ -import 'package:flutter/material.dart'; -import 'package:lightmeter/generated/l10n.dart'; - -class CloseButton extends StatelessWidget { - const CloseButton({super.key}); - - @override - Widget build(BuildContext context) { - return IconButton( - onPressed: Navigator.of(context).pop, - icon: const Icon(Icons.close), - tooltip: S.of(context).tooltipClose, - ); - } -} diff --git a/lib/screens/shared/pro_features_dialog/widget_dialog_pro_features.dart b/lib/screens/shared/pro_features_dialog/widget_dialog_pro_features.dart index 3b8d4f2..cdf76b0 100644 --- a/lib/screens/shared/pro_features_dialog/widget_dialog_pro_features.dart +++ b/lib/screens/shared/pro_features_dialog/widget_dialog_pro_features.dart @@ -24,7 +24,7 @@ class ProFeaturesDialog extends StatelessWidget { @override Widget build(BuildContext context) { return TransparentDialog( - icon: Icons.star, + icon: Icons.star_outlined, title: S.of(context).proFeatures, scrollableContent: false, content: Flexible( diff --git a/lib/screens/shared/ruler_slider/widget_slider_ruler.dart b/lib/screens/shared/ruler_slider/widget_slider_ruler.dart index 537824c..9932ef2 100644 --- a/lib/screens/shared/ruler_slider/widget_slider_ruler.dart +++ b/lib/screens/shared/ruler_slider/widget_slider_ruler.dart @@ -30,7 +30,7 @@ class RulerSlider extends StatelessWidget { children: [ Text( valueAdapter(value), - style: Theme.of(context).textTheme.labelLarge, + style: Theme.of(context).textTheme.labelLarge!.copyWith(color: Theme.of(context).colorScheme.onBackground), ), const SizedBox(height: Dimens.grid4), Expanded( @@ -53,7 +53,7 @@ class RulerSlider extends StatelessWidget { ), ), IconButton( - icon: const Icon(Icons.sync), + icon: const Icon(Icons.sync_outlined), onPressed: value != defaultValue ? () => onChanged(defaultValue) : null, tooltip: S.of(context).tooltipResetToZero, ), @@ -78,6 +78,7 @@ class _Ruler extends StatelessWidget { @override Widget build(BuildContext context) { final mainTicksFontSize = Theme.of(context).textTheme.bodySmall!.fontSize!; + final itemsColor = Theme.of(context).colorScheme.onBackground; return LayoutBuilder( builder: (context, constraints) { final bool showAllMainTicks = @@ -103,11 +104,11 @@ class _Ruler extends StatelessWidget { if (showValue) Text( rulerValueAdapter(index / 2 + min), - style: Theme.of(context).textTheme.bodySmall, + style: Theme.of(context).textTheme.bodySmall!.copyWith(color: itemsColor), ), const SizedBox(width: Dimens.grid4), ColoredBox( - color: Theme.of(context).colorScheme.onBackground, + color: itemsColor, child: SizedBox( height: 1, width: isMainTick ? Dimens.grid8 : Dimens.grid4, diff --git a/lib/screens/shared/sliver_screen/screen_sliver.dart b/lib/screens/shared/sliver_screen/screen_sliver.dart index 97ed830..4c69457 100644 --- a/lib/screens/shared/sliver_screen/screen_sliver.dart +++ b/lib/screens/shared/sliver_screen/screen_sliver.dart @@ -42,7 +42,7 @@ class SliverScreen extends StatelessWidget { if (Navigator.of(context).canPop()) IconButton( onPressed: Navigator.of(context).pop, - icon: const Icon(Icons.close), + icon: const Icon(Icons.close_outlined), tooltip: S.of(context).tooltipClose, ), ], diff --git a/lib/screens/timer/components/metering_config/widget_metering_config_timer.dart b/lib/screens/timer/components/metering_config/widget_metering_config_timer.dart index 8d2e199..8eb91f0 100644 --- a/lib/screens/timer/components/metering_config/widget_metering_config_timer.dart +++ b/lib/screens/timer/components/metering_config/widget_metering_config_timer.dart @@ -2,6 +2,7 @@ import 'package:flutter/material.dart'; import 'package:lightmeter/data/models/exposure_pair.dart'; import 'package:lightmeter/generated/l10n.dart'; import 'package:lightmeter/res/dimens.dart'; +import 'package:lightmeter/res/theme.dart'; import 'package:lightmeter/screens/metering/components/shared/readings_container/components/shared/reading_value_container/widget_container_reading_value.dart'; import 'package:m3_lightmeter_resources/m3_lightmeter_resources.dart'; @@ -21,7 +22,7 @@ class TimerMeteringConfig extends StatelessWidget { Widget build(BuildContext context) { return Container( decoration: BoxDecoration( - color: Theme.of(context).colorScheme.surface, + color: Theme.of(context).colorScheme.surfaceElevated1, borderRadius: const BorderRadius.only( bottomLeft: Radius.circular(Dimens.borderRadiusL), bottomRight: Radius.circular(Dimens.borderRadiusL), diff --git a/lib/screens/timer/components/timeline/widget_timeline_timer.dart b/lib/screens/timer/components/timeline/widget_timeline_timer.dart index 24acff8..ae605fb 100644 --- a/lib/screens/timer/components/timeline/widget_timeline_timer.dart +++ b/lib/screens/timer/components/timeline/widget_timeline_timer.dart @@ -16,7 +16,7 @@ class TimerTimeline extends StatelessWidget { Widget build(BuildContext context) { return CustomPaint( painter: _TimelinePainter( - backgroundColor: Theme.of(context).colorScheme.surface, + backgroundColor: Theme.of(context).colorScheme.primaryContainer, progressColor: Theme.of(context).colorScheme.primary, progress: progress, ), diff --git a/lib/screens/timer/screen_timer.dart b/lib/screens/timer/screen_timer.dart index b76899a..ca42577 100644 --- a/lib/screens/timer/screen_timer.dart +++ b/lib/screens/timer/screen_timer.dart @@ -1,6 +1,7 @@ import 'package:flutter/material.dart'; import 'package:flutter_bloc/flutter_bloc.dart'; import 'package:lightmeter/data/models/exposure_pair.dart'; +import 'package:lightmeter/generated/l10n.dart'; import 'package:lightmeter/res/dimens.dart'; import 'package:lightmeter/screens/shared/animated_circular_button/widget_button_circular_animated.dart'; import 'package:lightmeter/screens/shared/bottom_controls_bar/widget_bottom_controls_bar.dart'; @@ -66,7 +67,6 @@ class TimerScreenState extends State with TickerProviderStateMixin listenWhen: (previous, current) => previous.runtimeType != current.runtimeType, listener: (context, state) => _updateAnimations(state), child: Scaffold( - backgroundColor: Theme.of(context).colorScheme.background, body: Center( child: Column( mainAxisAlignment: MainAxisAlignment.center, @@ -95,7 +95,7 @@ class TimerScreenState extends State with TickerProviderStateMixin ), const Spacer(), BottomControlsBar( - left: IconButton( + left: IconButton.filledTonal( onPressed: () { context.read().add(const ResetTimerEvent()); }, @@ -118,7 +118,11 @@ class TimerScreenState extends State with TickerProviderStateMixin ), ), ), - right: const CloseButton(), + right: IconButton.filledTonal( + onPressed: Navigator.of(context).pop, + icon: const Icon(Icons.close), + tooltip: S.of(context).tooltipClose, + ), ), ], ), diff --git a/pubspec.yaml b/pubspec.yaml index 61d801d..391e525 100644 --- a/pubspec.yaml +++ b/pubspec.yaml @@ -12,7 +12,7 @@ dependencies: camera: 0.10.5+2 camera_android_camerax: 0.6.1+1 clipboard: 0.1.3 - dynamic_color: 1.6.6 + dynamic_color: 1.7.0 exif: 3.1.4 firebase_analytics: 10.6.2 firebase_core: 2.20.0 @@ -61,6 +61,9 @@ dev_dependencies: mocktail: 0.3.0 test: 1.24.3 +dependency_overrides: + material_color_utilities: 0.11.1 + flutter: uses-material-design: true assets: diff --git a/test/screens/metering/components/shared/readings_container/equipment_profile_picker_test.dart b/test/screens/metering/components/shared/readings_container/equipment_profile_picker_test.dart index 0953190..5eec978 100644 --- a/test/screens/metering/components/shared/readings_container/equipment_profile_picker_test.dart +++ b/test/screens/metering/components/shared/readings_container/equipment_profile_picker_test.dart @@ -47,7 +47,7 @@ void main() { await pumpApplication(tester); expectReadingValueContainerText(S.current.equipmentProfile); await tester.openAnimatedPicker(); - expect(find.byIcon(Icons.camera), findsOneWidget); + expect(find.byIcon(Icons.camera_outlined), findsOneWidget); expectDialogPickerText(S.current.equipmentProfile); }, ); diff --git a/test/screens/metering/components/shared/readings_container/iso_picker_test.dart b/test/screens/metering/components/shared/readings_container/iso_picker_test.dart index 9068df4..ae7f88f 100644 --- a/test/screens/metering/components/shared/readings_container/iso_picker_test.dart +++ b/test/screens/metering/components/shared/readings_container/iso_picker_test.dart @@ -38,7 +38,7 @@ void main() { await pumpApplication(tester); expectReadingValueContainerText(S.current.iso); await tester.openAnimatedPicker(); - expect(find.byIcon(Icons.iso), findsOneWidget); + expect(find.byIcon(Icons.iso_outlined), findsOneWidget); expectDialogPickerText(S.current.iso); expectDialogPickerText(S.current.filmSpeed); }, diff --git a/test/screens/metering/components/shared/readings_container/nd_picker_test.dart b/test/screens/metering/components/shared/readings_container/nd_picker_test.dart index 3deb824..783e279 100644 --- a/test/screens/metering/components/shared/readings_container/nd_picker_test.dart +++ b/test/screens/metering/components/shared/readings_container/nd_picker_test.dart @@ -38,7 +38,7 @@ void main() { await pumpApplication(tester); expectReadingValueContainerText(S.current.nd); await tester.openAnimatedPicker(); - expect(find.byIcon(Icons.filter_b_and_w), findsOneWidget); + expect(find.byIcon(Icons.filter_b_and_w_outlined), findsOneWidget); expectDialogPickerText(S.current.nd); expectDialogPickerText(S.current.ndFilterFactor); }, diff --git a/test/screens/metering/goldens/metering_screen.png b/test/screens/metering/goldens/metering_screen.png index 04e2c41..a88dc2c 100644 Binary files a/test/screens/metering/goldens/metering_screen.png and b/test/screens/metering/goldens/metering_screen.png differ diff --git a/test/screens/settings/goldens/settings_screen.png b/test/screens/settings/goldens/settings_screen.png index 0a6016e..1c65c19 100644 Binary files a/test/screens/settings/goldens/settings_screen.png and b/test/screens/settings/goldens/settings_screen.png differ diff --git a/test/screens/timer/goldens/timer_screen.png b/test/screens/timer/goldens/timer_screen.png index f08e7ca..cb3aa24 100644 Binary files a/test/screens/timer/goldens/timer_screen.png and b/test/screens/timer/goldens/timer_screen.png differ