refactor(web): simple actions

refactor: use SemVer classes for version compatability message (#29056 )
* refactor: use SemVer classes for version compatability message * chore: readd major version compatabilty messages * fix: remove 1.106.0 check (we dont support v1 servers anymore)
2026-06-22 22:56:39 -07:00 · 2026-06-22 18:10:51 +02:00 · 2026-06-22 11:28:56 -04:00 · 2026-06-22 16:58:07 +02:00 · 2026-06-22 09:43:15 -05:00
31 changed files with 117 additions and 590 deletions
@@ -1251,22 +1251,6 @@
  "favorite_or_unfavorite_photo": "Favorite or unfavorite photo",
  "favorites": "Favorites",
  "favorites_page_no_favorites": "No favorite assets found",
-  "feature_message_non_destructive_editing_body": "Edit your photos freely — the original is always kept untouched.",
-  "feature_message_non_destructive_editing_title": "Non-destructive editing",
-  "feature_message_ocr_body": "Immich now reads the text inside your photos, so you can search for them by what they say.",
-  "feature_message_ocr_title": "Search text in your photos",
-  "feature_message_open_in_immich_body": "Set Immich as your gallery on Android to open photos straight from other apps.",
-  "feature_message_open_in_immich_title": "Open photos in Immich",
-  "feature_message_recently_added_body": "Jump straight to everything you've added lately on a dedicated page.",
-  "feature_message_recently_added_title": "Recently added",
-  "feature_message_settings_subtitle": "See what's new in version {version}",
-  "feature_message_share_quality_body": "Press and hold the share button to choose the image quality before you share.",
-  "feature_message_share_quality_title": "Choose your share quality",
-  "feature_message_slideshow_body": "Sit back and watch your photos play in a full-screen slideshow.",
-  "feature_message_slideshow_title": "Slideshow",
-  "feature_message_upload_to_album_body": "Add photos directly into an album as you upload them.",
-  "feature_message_upload_to_album_title": "Upload straight to an album",
-  "feature_message_version": "Version {version}",
  "feature_photo_updated": "Feature photo updated",
  "features": "Features",
  "features_in_development": "Features in Development",
@@ -1564,7 +1548,7 @@
  "map_location_picker_page_use_location": "Use this location",
  "map_location_service_disabled_content": "Location service needs to be enabled to display assets from your current location. Do you want to enable it now?",
  "map_location_service_disabled_title": "Location Service disabled",
-  "map_marker_for_images": "Map marker for images taken in {city}, {country}",
+  "map_marker_for_image": "Map marker for image taken in {city}, {country}",
  "map_marker_with_image": "Map marker with image",
  "map_no_location_permission_content": "Location permission is needed to display assets from your current location. Do you want to allow it now?",
  "map_no_location_permission_title": "Location Permission denied",
@@ -2546,7 +2530,6 @@
  "week": "Week",
  "welcome": "Welcome",
  "welcome_to_immich": "Welcome to Immich",
-  "whats_new": "What's new",
  "when": "When",
  "width": "Width",
  "wifi_name": "Wi-Fi Name",
@@ -4,7 +4,6 @@ import 'package:immich_mobile/constants/enums.dart';
 import 'package:immich_mobile/domain/models/config/album_config.dart';
 import 'package:immich_mobile/domain/models/config/backup_config.dart';
 import 'package:immich_mobile/domain/models/config/cleanup_config.dart';
-import 'package:immich_mobile/domain/models/config/feature_message_config.dart';
 import 'package:immich_mobile/domain/models/config/image_config.dart';
 import 'package:immich_mobile/domain/models/config/map_config.dart';
 import 'package:immich_mobile/domain/models/config/network_config.dart';
@@ -33,7 +32,6 @@ class AppConfig {
  final BackupConfig backup;
  final NetworkConfig network;
  final ShareConfig share;
-  final FeatureMessageConfig featureMessage;

  const AppConfig({
    this.logLevel = .info,
@@ -48,7 +46,6 @@ class AppConfig {
    this.backup = const .new(),
    this.network = const .new(),
    this.share = const .new(),
-    this.featureMessage = const .new(),
  });

  AppConfig copyWith({
@@ -64,7 +61,6 @@ class AppConfig {
    BackupConfig? backup,
    NetworkConfig? network,
    ShareConfig? share,
-    FeatureMessageConfig? featureMessage,
  }) => .new(
    logLevel: logLevel ?? this.logLevel,
    theme: theme ?? this.theme,
@@ -78,7 +74,6 @@ class AppConfig {
    backup: backup ?? this.backup,
    network: network ?? this.network,
    share: share ?? this.share,
-    featureMessage: featureMessage ?? this.featureMessage,
  );

  @override
@@ -96,29 +91,15 @@ class AppConfig {
          other.album == album &&
          other.backup == backup &&
          other.network == network &&
-          other.share == share &&
-          other.featureMessage == featureMessage);
+          other.share == share);

  @override
-  int get hashCode => Object.hash(
-    logLevel,
-    theme,
-    cleanup,
-    map,
-    timeline,
-    image,
-    viewer,
-    slideshow,
-    album,
-    backup,
-    network,
-    share,
-    featureMessage,
-  );
+  int get hashCode =>
+      Object.hash(logLevel, theme, cleanup, map, timeline, image, viewer, slideshow, album, backup, network, share);

  @override
  String toString() =>
-      'AppConfig(logLevel: $logLevel, theme: $theme, cleanup: $cleanup, map: $map, timeline: $timeline, image: $image, viewer: $viewer, slideshow: $slideshow, album: $album, backup: $backup, network: $network, share: $share, featureMessage: $featureMessage)';
+      'AppConfig(logLevel: $logLevel, theme: $theme, cleanup: $cleanup, map: $map, timeline: $timeline, image: $image, viewer: $viewer, slideshow: $slideshow, album: $album, backup: $backup, network: $network, share: $share)';

  T read<T>(SettingsKey<T> key) =>
      (switch (key) {
@@ -165,7 +146,6 @@ class AppConfig {
            .slideshowDuration => slideshow.duration,
            .slideshowLook => slideshow.look,
            .slideshowDirection => slideshow.direction,
-            .featureMessageSeenVersion => featureMessage.seenVersion,
          })
          as T;

@@ -219,7 +199,6 @@ class AppConfig {
      .slideshowDuration => copyWith(slideshow: slideshow.copyWith(duration: value as int)),
      .slideshowLook => copyWith(slideshow: slideshow.copyWith(look: value as SlideshowLook)),
      .slideshowDirection => copyWith(slideshow: slideshow.copyWith(direction: value as SlideshowDirection)),
-      .featureMessageSeenVersion => copyWith(featureMessage: featureMessage.copyWith(seenVersion: value as int)),
    };
  }
 }
@@ -1,18 +0,0 @@
-class FeatureMessageConfig {
-  final int seenVersion;
-
-  const FeatureMessageConfig({this.seenVersion = 0});
-
-  FeatureMessageConfig copyWith({int? seenVersion}) =>
-      FeatureMessageConfig(seenVersion: seenVersion ?? this.seenVersion);
-
-  @override
-  bool operator ==(Object other) =>
-      identical(this, other) || (other is FeatureMessageConfig && other.seenVersion == seenVersion);
-
-  @override
-  int get hashCode => seenVersion.hashCode;
-
-  @override
-  String toString() => 'FeatureMessageConfig(seenVersion: $seenVersion)';
-}
@@ -1,49 +0,0 @@
-class FeatureHighlight {
-  final String image;
-  final String titleKey;
-  final String bodyKey;
-
-  const FeatureHighlight({required this.image, required this.titleKey, required this.bodyKey});
-}
-
-const int featureMessageHighlightVersion = 1;
-
-const String featureMessageReleaseLabel = '3.0.0';
-
-const List<FeatureHighlight> featureMessageHighlights = [
-  FeatureHighlight(
-    image: 'assets/feature_message/share_quality.webp',
-    titleKey: 'feature_message_share_quality_title',
-    bodyKey: 'feature_message_share_quality_body',
-  ),
-  FeatureHighlight(
-    image: 'assets/feature_message/slideshow.webp',
-    titleKey: 'feature_message_slideshow_title',
-    bodyKey: 'feature_message_slideshow_body',
-  ),
-  FeatureHighlight(
-    image: 'assets/feature_message/recently_added.webp',
-    titleKey: 'feature_message_recently_added_title',
-    bodyKey: 'feature_message_recently_added_body',
-  ),
-  FeatureHighlight(
-    image: 'assets/feature_message/non_destructive_editing.webp',
-    titleKey: 'feature_message_non_destructive_editing_title',
-    bodyKey: 'feature_message_non_destructive_editing_body',
-  ),
-  FeatureHighlight(
-    image: 'assets/feature_message/ocr.webp',
-    titleKey: 'feature_message_ocr_title',
-    bodyKey: 'feature_message_ocr_body',
-  ),
-  FeatureHighlight(
-    image: 'assets/feature_message/open_in_immich.webp',
-    titleKey: 'feature_message_open_in_immich_title',
-    bodyKey: 'feature_message_open_in_immich_body',
-  ),
-  FeatureHighlight(
-    image: 'assets/feature_message/upload_to_album.webp',
-    titleKey: 'feature_message_upload_to_album_title',
-    bodyKey: 'feature_message_upload_to_album_body',
-  ),
-];
@@ -73,10 +73,7 @@ enum SettingsKey<T> {
  slideshowRepeat<bool>(),
  slideshowDuration<int>(),
  slideshowLook<SlideshowLook>(codec: _EnumCodec(SlideshowLook.values)),
-  slideshowDirection<SlideshowDirection>(codec: _EnumCodec(SlideshowDirection.values)),
-
-  // Feature message
-  featureMessageSeenVersion<int>();
+  slideshowDirection<SlideshowDirection>(codec: _EnumCodec(SlideshowDirection.values));

  final _SettingsCodec<T>? _codecOverride;

@@ -1,17 +0,0 @@
-import 'package:immich_mobile/domain/models/feature_message.model.dart';
-import 'package:immich_mobile/domain/models/settings_key.dart';
-import 'package:immich_mobile/infrastructure/repositories/settings.repository.dart';
-
-class FeatureMessageService {
-  final SettingsRepository _settingsRepository;
-
-  const FeatureMessageService(this._settingsRepository);
-
-  bool shouldShow() {
-    final seen = _settingsRepository.appConfig.read(SettingsKey.featureMessageSeenVersion);
-    return featureMessageHighlights.isNotEmpty && featureMessageHighlightVersion > seen;
-  }
-
-  Future<void> markSeen() =>
-      _settingsRepository.write(SettingsKey.featureMessageSeenVersion, featureMessageHighlightVersion);
-}
@@ -2,9 +2,7 @@ import 'package:auto_route/auto_route.dart';
 import 'package:easy_localization/easy_localization.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_hooks/flutter_hooks.dart' hide Store;
-import 'package:immich_mobile/domain/models/feature_message.model.dart';
 import 'package:immich_mobile/extensions/build_context_extensions.dart';
-import 'package:immich_mobile/presentation/widgets/feature_message/feature_message_dialog.widget.dart';
 import 'package:immich_mobile/routing/router.dart';
 import 'package:immich_mobile/widgets/settings/advanced_settings.dart';
 import 'package:immich_mobile/widgets/settings/asset_list_settings/asset_list_settings.dart';
@@ -89,14 +87,6 @@ class _MobileLayout extends StatelessWidget {
                ],
        )
        .toList();
-    settings.add(
-      SettingsCard(
-        icon: Icons.auto_awesome_outlined,
-        title: 'whats_new'.tr(),
-        subtitle: 'feature_message_settings_subtitle'.tr(namedArgs: {'version': featureMessageReleaseLabel}),
-        onTap: () => showFeatureMessageDialog(context),
-      ),
-    );
    return ListView(padding: const EdgeInsets.only(top: 10.0, bottom: 60), children: [...settings]);
  }
 }
@@ -126,13 +116,6 @@ class _TabletLayout extends HookWidget {
                  ),
                ),
              ),
-              SliverToBoxAdapter(
-                child: ListTile(
-                  title: Text('whats_new'.tr()),
-                  leading: const Icon(Icons.auto_awesome_outlined),
-                  onTap: () => showFeatureMessageDialog(context),
-                ),
-              ),
            ],
          ),
        ),
@@ -3,37 +3,14 @@ import 'package:flutter/material.dart';
 import 'package:hooks_riverpod/hooks_riverpod.dart';
 import 'package:immich_mobile/presentation/widgets/memory/memory_lane.widget.dart';
 import 'package:immich_mobile/presentation/widgets/timeline/timeline.widget.dart';
-import 'package:immich_mobile/presentation/widgets/feature_message/feature_message_dialog.widget.dart';
-import 'package:immich_mobile/providers/feature_message.provider.dart';
 import 'package:immich_mobile/providers/infrastructure/memory.provider.dart';

@RoutePage()
-class MainTimelinePage extends ConsumerStatefulWidget {
+class MainTimelinePage extends ConsumerWidget {
  const MainTimelinePage({super.key});

  @override
-  ConsumerState<MainTimelinePage> createState() => _MainTimelinePageState();
-}
-
-class _MainTimelinePageState extends ConsumerState<MainTimelinePage> {
-  @override
-  void initState() {
-    super.initState();
-    WidgetsBinding.instance.addPostFrameCallback((_) {
-      if (!mounted || ref.read(featureMessageCheckedProvider)) {
-        return;
-      }
-      ref.read(featureMessageCheckedProvider.notifier).state = true;
-
-      final service = ref.read(featureMessageServiceProvider);
-      // if (service.shouldShow()) {
-      showFeatureMessageDialog(context).then((_) => service.markSeen());
-      // }
-    });
-  }
-
-  @override
-  Widget build(BuildContext context) {
+  Widget build(BuildContext context, WidgetRef ref) {
    final hasMemories = ref.watch(driftMemoryFutureProvider.select((state) => state.value?.isNotEmpty ?? false));
    return Timeline(
      topSliverWidget: const SliverToBoxAdapter(child: DriftMemoryLane()),
@@ -1,201 +0,0 @@
-import 'package:easy_localization/easy_localization.dart';
-import 'package:flutter/material.dart';
-import 'package:immich_mobile/domain/models/feature_message.model.dart';
-import 'package:immich_mobile/extensions/build_context_extensions.dart';
-
-Future<void> showFeatureMessageDialog(BuildContext context) {
-  return showGeneralDialog<void>(
-    context: context,
-    useRootNavigator: true,
-    barrierDismissible: true,
-    barrierLabel: 'whats_new'.tr(),
-    barrierColor: Colors.black.withValues(alpha: 0.55),
-    transitionDuration: const Duration(milliseconds: 280),
-    pageBuilder: (_, __, ___) => const _FeatureMessageDialog(),
-    transitionBuilder: (_, animation, __, child) {
-      final curved = CurvedAnimation(parent: animation, curve: Curves.easeOutCubic, reverseCurve: Curves.easeInCubic);
-      return FadeTransition(
-        opacity: animation,
-        child: ScaleTransition(scale: Tween<double>(begin: 0.94, end: 1.0).animate(curved), child: child),
-      );
-    },
-  );
-}
-
-class _FeatureMessageDialog extends StatefulWidget {
-  const _FeatureMessageDialog();
-
-  @override
-  State<_FeatureMessageDialog> createState() => _FeatureMessageDialogState();
-}
-
-class _FeatureMessageDialogState extends State<_FeatureMessageDialog> {
-  final PageController _controller = PageController();
-  int _index = 0;
-
-  bool get _isLast => _index >= featureMessageHighlights.length - 1;
-
-  @override
-  void dispose() {
-    _controller.dispose();
-    super.dispose();
-  }
-
-  void _advance() {
-    if (_isLast) {
-      Navigator.of(context).pop();
-      return;
-    }
-    _controller.nextPage(duration: const Duration(milliseconds: 320), curve: Curves.easeOutCubic);
-  }
-
-  @override
-  Widget build(BuildContext context) {
-    return Dialog(
-      insetPadding: const EdgeInsets.symmetric(horizontal: 8, vertical: 64),
-      clipBehavior: Clip.antiAlias,
-      backgroundColor: context.colorScheme.surface,
-      shape: const RoundedRectangleBorder(borderRadius: BorderRadius.all(Radius.circular(24))),
-      child: ConstrainedBox(
-        constraints: BoxConstraints(maxHeight: context.height * 0.9, maxWidth: 480),
-        child: Column(
-          mainAxisSize: MainAxisSize.min,
-          mainAxisAlignment: MainAxisAlignment.start,
-          crossAxisAlignment: CrossAxisAlignment.start,
-          children: [
-            Padding(
-              padding: const EdgeInsets.fromLTRB(24, 20, 24, 4),
-              child: Column(
-                crossAxisAlignment: CrossAxisAlignment.start,
-                mainAxisSize: MainAxisSize.min,
-                children: [
-                  Text('whats_new'.tr(), style: context.textTheme.titleLarge?.copyWith(fontWeight: FontWeight.w700)),
-                  const SizedBox(height: 2),
-                  Text(
-                    'feature_message_version'.tr(namedArgs: {'version': featureMessageReleaseLabel}),
-                    style: context.textTheme.bodyMedium?.copyWith(color: context.colorScheme.onSurfaceVariant),
-                  ),
-                ],
-              ),
-            ),
-            Expanded(
-              child: PageView.builder(
-                controller: _controller,
-                itemCount: featureMessageHighlights.length,
-                onPageChanged: (i) => setState(() => _index = i),
-                itemBuilder: (_, index) => _FeaturePage(highlight: featureMessageHighlights[index]),
-              ),
-            ),
-            const SizedBox(height: 8),
-            _PageDots(controller: _controller, index: _index, count: featureMessageHighlights.length),
-            Padding(
-              padding: const EdgeInsets.fromLTRB(20, 16, 20, 16),
-              child: SizedBox(
-                width: double.infinity,
-                child: FilledButton(
-                  onPressed: _advance,
-                  style: FilledButton.styleFrom(padding: const EdgeInsets.symmetric(vertical: 14)),
-                  child: AnimatedSwitcher(
-                    duration: const Duration(milliseconds: 200),
-                    child: Text(_isLast ? 'ok'.tr() : 'next'.tr(), key: ValueKey(_isLast)),
-                  ),
-                ),
-              ),
-            ),
-          ],
-        ),
-      ),
-    );
-  }
-}
-
-class _FeaturePage extends StatelessWidget {
-  final FeatureHighlight highlight;
-
-  const _FeaturePage({required this.highlight});
-
-  @override
-  Widget build(BuildContext context) {
-    final scheme = context.colorScheme;
-
-    return SingleChildScrollView(
-      child: Column(
-        children: [
-          Padding(
-            padding: const EdgeInsets.fromLTRB(20, 8, 20, 0),
-            child: ClipRRect(
-              borderRadius: BorderRadius.circular(18),
-              child: ColoredBox(
-                color: scheme.surfaceContainerHighest,
-                child: SizedBox(
-                  width: double.infinity,
-                  height: 300,
-                  child: Image.asset(
-                    highlight.image,
-                    fit: BoxFit.contain,
-                    errorBuilder: (context, _, __) =>
-                        Center(child: Icon(Icons.auto_awesome_outlined, color: scheme.onSurfaceVariant, size: 56)),
-                  ),
-                ),
-              ),
-            ),
-          ),
-          Padding(
-            padding: const EdgeInsets.fromLTRB(24, 18, 24, 8),
-            child: Column(
-              crossAxisAlignment: CrossAxisAlignment.start,
-              mainAxisSize: MainAxisSize.min,
-              children: [
-                Text(
-                  highlight.titleKey.tr(),
-                  style: context.textTheme.titleLarge?.copyWith(fontWeight: FontWeight.w700),
-                ),
-                const SizedBox(height: 8),
-                Text(
-                  highlight.bodyKey.tr(),
-                  style: context.textTheme.bodyMedium?.copyWith(color: scheme.onSurfaceVariant, height: 1.4),
-                ),
-              ],
-            ),
-          ),
-        ],
-      ),
-    );
-  }
-}
-
-class _PageDots extends StatelessWidget {
-  final PageController controller;
-  final int index;
-  final int count;
-
-  const _PageDots({required this.controller, required this.index, required this.count});
-
-  @override
-  Widget build(BuildContext context) {
-    final primary = context.primaryColor;
-
-    return AnimatedBuilder(
-      animation: controller,
-      builder: (context, _) {
-        final page = controller.hasClients ? (controller.page ?? index.toDouble()) : index.toDouble();
-        return Row(
-          mainAxisAlignment: MainAxisAlignment.center,
-          children: List.generate(count, (i) {
-            final activeness = (1 - (page - i).abs()).clamp(0.0, 1.0);
-            return AnimatedContainer(
-              duration: const Duration(milliseconds: 150),
-              margin: const EdgeInsets.symmetric(horizontal: 3),
-              height: 7,
-              width: 7 + 16 * activeness,
-              decoration: BoxDecoration(
-                color: Color.lerp(context.colorScheme.surfaceContainerHighest, primary, activeness),
-                borderRadius: BorderRadius.circular(8),
-              ),
-            );
-          }),
-        );
-      },
-    );
-  }
-}
@@ -1,101 +0,0 @@
-import 'package:easy_localization/easy_localization.dart';
-import 'package:flutter/material.dart';
-import 'package:immich_mobile/domain/models/feature_message.model.dart';
-import 'package:immich_mobile/extensions/build_context_extensions.dart';
-import 'package:immich_mobile/presentation/widgets/bottom_sheet/base_bottom_sheet.widget.dart';
-
-Future<void> showFeatureMessageSheet(BuildContext context) {
-  return showModalBottomSheet(
-    context: context,
-    isScrollControlled: true,
-    backgroundColor: Colors.transparent,
-    useRootNavigator: true,
-    builder: (_) => const _FeatureMessageSheet(),
-  );
-}
-
-class _FeatureMessageSheet extends StatelessWidget {
-  const _FeatureMessageSheet();
-
-  @override
-  Widget build(BuildContext context) {
-    return BaseBottomSheet(
-      actions: const [],
-      resizeOnScroll: false,
-      expand: false,
-      initialChildSize: 0.6,
-      minChildSize: 0.4,
-      maxChildSize: 0.9,
-      slivers: [
-        SliverToBoxAdapter(
-          child: Padding(
-            padding: const EdgeInsets.fromLTRB(24, 4, 24, 16),
-            child: Text(
-              'whats_new'.tr(),
-              style: context.textTheme.headlineSmall?.copyWith(fontWeight: FontWeight.w600),
-            ),
-          ),
-        ),
-        SliverList.separated(
-          itemCount: featureMessageHighlights.length,
-          separatorBuilder: (_, __) => const SizedBox(height: 8),
-          itemBuilder: (_, index) => _HighlightCard(highlight: featureMessageHighlights[index]),
-        ),
-        const SliverToBoxAdapter(child: SizedBox(height: 16)),
-      ],
-      footer: SafeArea(
-        top: false,
-        child: Padding(
-          padding: const EdgeInsets.fromLTRB(24, 8, 24, 16),
-          child: SizedBox(
-            width: double.infinity,
-            child: FilledButton(
-              onPressed: () => Navigator.of(context).pop(),
-              style: FilledButton.styleFrom(padding: const EdgeInsets.symmetric(vertical: 14)),
-              child: Text('feature_message_got_it'.tr()),
-            ),
-          ),
-        ),
-      ),
-    );
-  }
-}
-
-class _HighlightCard extends StatelessWidget {
-  final FeatureHighlight highlight;
-
-  const _HighlightCard({required this.highlight});
-
-  @override
-  Widget build(BuildContext context) {
-    return Padding(
-      padding: const EdgeInsets.symmetric(horizontal: 16),
-      child: Column(
-        crossAxisAlignment: CrossAxisAlignment.start,
-        children: [
-          ClipRRect(
-            borderRadius: BorderRadius.circular(16),
-            child: AspectRatio(
-              aspectRatio: 16 / 9,
-              child: Image.asset(
-                highlight.image,
-                fit: BoxFit.cover,
-                errorBuilder: (context, _, __) => ColoredBox(
-                  color: context.colorScheme.surfaceContainerHighest,
-                  child: Icon(Icons.image_outlined, color: context.colorScheme.onSurfaceVariant, size: 48),
-                ),
-              ),
-            ),
-          ),
-          const SizedBox(height: 12),
-          Text(highlight.titleKey.tr(), style: context.textTheme.titleMedium?.copyWith(fontWeight: FontWeight.w600)),
-          const SizedBox(height: 4),
-          Text(
-            highlight.bodyKey.tr(),
-            style: context.textTheme.bodyMedium?.copyWith(color: context.colorScheme.onSurfaceVariant),
-          ),
-        ],
-      ),
-    );
-  }
-}
@@ -1,9 +0,0 @@
-import 'package:hooks_riverpod/hooks_riverpod.dart';
-import 'package:immich_mobile/domain/services/feature_message.service.dart';
-import 'package:immich_mobile/providers/infrastructure/settings.provider.dart';
-
-final featureMessageServiceProvider = Provider<FeatureMessageService>(
-  (ref) => FeatureMessageService(ref.read(settingsProvider)),
-);
-
-final featureMessageCheckedProvider = StateProvider<bool>((ref) => false);
@@ -1,7 +1,16 @@
-String? getVersionCompatibilityMessage(int _, int appMinor, int _, int serverMinor) {
+import 'package:immich_mobile/utils/semver.dart';
+
+String? getVersionCompatibilityMessage(SemVer serverVersion, SemVer appVersion) {
  // Add latest compat info up top
-  if (serverMinor < 106 && appMinor >= 106) {
-    return 'Your app minor version is not compatible with the server! Please update your server to version v1.106.0 or newer to login';
+
+  // ensure mobile app major version is not behind server major version
+  if (appVersion.major < serverVersion.major) {
+    return 'Your mobile app version is not compatible with the server! Please update your mobile app to the latest version.';
+  }
+
+  // ensure mobile app major version is not ahead of server major version by more than 1 major version
+  if (appVersion.major > serverVersion.major + 1) {
+    return 'Your server version is not compatible with the mobile app! Please update your server to the latest version.';
  }

  return null;
@@ -18,7 +18,6 @@ import 'package:immich_mobile/extensions/translate_extensions.dart';
 import 'package:immich_mobile/infrastructure/repositories/settings.repository.dart';
 import 'package:immich_mobile/providers/auth.provider.dart';
 import 'package:immich_mobile/providers/background_sync.provider.dart';
-import 'package:immich_mobile/providers/feature_message.provider.dart';
 import 'package:immich_mobile/providers/gallery_permission.provider.dart';
 import 'package:immich_mobile/providers/oauth.provider.dart';
 import 'package:immich_mobile/providers/server_info.provider.dart';
@@ -27,6 +26,7 @@ import 'package:immich_mobile/providers/websocket.provider.dart';
 import 'package:immich_mobile/repositories/permission.repository.dart';
 import 'package:immich_mobile/routing/router.dart';
 import 'package:immich_mobile/utils/provider_utils.dart';
+import 'package:immich_mobile/utils/semver.dart';
 import 'package:immich_mobile/utils/url_helper.dart';
 import 'package:immich_mobile/utils/version_compatibility.dart';
 import 'package:immich_mobile/widgets/common/immich_logo.dart';
@@ -89,18 +89,9 @@ class LoginForm extends HookConsumerWidget {
    checkVersionMismatch() async {
      try {
        final packageInfo = await PackageInfo.fromPlatform();
-        final appVersion = packageInfo.version;
-        final appMajorVersion = int.parse(appVersion.split('.')[0]);
-        final appMinorVersion = int.parse(appVersion.split('.')[1]);
-        final serverMajorVersion = serverInfo.serverVersion.major;
-        final serverMinorVersion = serverInfo.serverVersion.minor;
-
-        warningMessage.value = getVersionCompatibilityMessage(
-          appMajorVersion,
-          appMinorVersion,
-          serverMajorVersion,
-          serverMinorVersion,
-        );
+        final appSemVer = SemVer.fromString(packageInfo.version);
+        final serverSemVer = serverInfo.serverVersion;
+        warningMessage.value = getVersionCompatibilityMessage(appSemVer, serverSemVer);
      } catch (error) {
        warningMessage.value = 'Error checking version compatibility';
      }
@@ -263,7 +254,6 @@ class LoginForm extends HookConsumerWidget {
          }
          unawaited(handleSyncFlow());
          ref.read(websocketProvider.notifier).connect();
-          unawaited(ref.read(featureMessageServiceProvider).markSeen());
          unawaited(context.router.replaceAll([const TabShellRoute()]));
          return;
        }
@@ -351,7 +341,6 @@ class LoginForm extends HookConsumerWidget {
              await getManageMediaPermission();
            }
            unawaited(handleSyncFlow());
-            unawaited(ref.read(featureMessageServiceProvider).markSeen());
            unawaited(context.router.replaceAll([const TabShellRoute()]));
            return;
          }
@@ -8,15 +8,13 @@ class SettingsCard extends StatelessWidget {
    required this.icon,
    required this.title,
    required this.subtitle,
-    this.settingRoute,
-    this.onTap,
+    required this.settingRoute,
  });

  final IconData icon;
  final String title;
  final String subtitle;
-  final PageRouteInfo? settingRoute;
-  final VoidCallback? onTap;
+  final PageRouteInfo settingRoute;

  @override
  Widget build(BuildContext context) {
@@ -40,7 +38,7 @@ class SettingsCard extends StatelessWidget {
          ),
          title: Text(title, style: context.textTheme.titleMedium!.copyWith(color: context.primaryColor)),
          subtitle: Text(subtitle, style: context.textTheme.bodyMedium),
-          onTap: onTap ?? (settingRoute != null ? () => context.pushRoute(settingRoute!) : null),
+          onTap: () => context.pushRoute(settingRoute),
        ),
      ),
    );
@@ -119,7 +119,6 @@ flutter:
  uses-material-design: true
  assets:
    - assets/
-    - assets/feature_message/
  fonts:
    - family: GoogleSans
      fonts:
@@ -1,29 +1,47 @@
 import 'package:flutter_test/flutter_test.dart';
+import 'package:immich_mobile/utils/semver.dart';
 import 'package:immich_mobile/utils/version_compatibility.dart';

 void main() {
-  test('getVersionCompatibilityMessage', () {
-    String? result;
+  group('app major version behind server', () {
+    const message =
+        'Your mobile app version is not compatible with the server! Please update your mobile app to the latest version.';

-    result = getVersionCompatibilityMessage(1, 106, 1, 105);
-    expect(
-      result,
-      'Your app minor version is not compatible with the server! Please update your server to version v1.106.0 or newer to login',
-    );
+    test('returns message when app major is behind server major', () {
+      final result = getVersionCompatibilityMessage(
+        const SemVer(major: 2, minor: 0, patch: 0),
+        const SemVer(major: 1, minor: 200, patch: 0),
+      );
+      expect(result, message);
+    });

-    result = getVersionCompatibilityMessage(1, 107, 1, 105);
-    expect(
-      result,
-      'Your app minor version is not compatible with the server! Please update your server to version v1.106.0 or newer to login',
-    );
+    test('returns null when app major matches server major', () {
+      final result = getVersionCompatibilityMessage(
+        const SemVer(major: 2, minor: 0, patch: 0),
+        const SemVer(major: 2, minor: 0, patch: 0),
+      );
+      expect(result, null);
+    });
+  });

-    result = getVersionCompatibilityMessage(1, 106, 1, 106);
-    expect(result, null);
+  group('app major version too far ahead of server', () {
+    const message =
+        'Your server version is not compatible with the mobile app! Please update your server to the latest version.';

-    result = getVersionCompatibilityMessage(1, 107, 1, 106);
-    expect(result, null);
+    test('returns message when app major is more than one ahead of server', () {
+      final result = getVersionCompatibilityMessage(
+        const SemVer(major: 1, minor: 200, patch: 0),
+        const SemVer(major: 3, minor: 0, patch: 0),
+      );
+      expect(result, message);
+    });

-    result = getVersionCompatibilityMessage(1, 107, 1, 108);
-    expect(result, null);
+    test('returns null when app major is exactly one ahead of server', () {
+      final result = getVersionCompatibilityMessage(
+        const SemVer(major: 1, minor: 200, patch: 0),
+        const SemVer(major: 2, minor: 0, patch: 0),
+      );
+      expect(result, null);
+    });
  });
 }
@@ -159,7 +159,9 @@
  }

  .text-white-shadow {
-    text-shadow: 0 1px 2px rgba(0, 0, 0, 0.8);
+    text-shadow:
+      0 0 4px rgba(0, 0, 0, 0.9),
+      0 1px 3px rgba(0, 0, 0, 0.8);
  }

  .icon-white-drop-shadow {
@@ -10,13 +10,11 @@
  import RemoveAssetFromStack from '$lib/components/asset-viewer/actions/RemoveAssetFromStack.svelte';
  import RestoreAction from '$lib/components/asset-viewer/actions/RestoreAction.svelte';
  import SetFeaturedPhotoAction from '$lib/components/asset-viewer/actions/SetPersonFeaturedAction.svelte';
-  import SetProfilePictureAction from '$lib/components/asset-viewer/actions/SetProfilePictureAction.svelte';
  import SetStackPrimaryAsset from '$lib/components/asset-viewer/actions/SetStackPrimaryAsset.svelte';
  import SetVisibilityAction from '$lib/components/asset-viewer/actions/SetVisibilityAction.svelte';
  import UnstackAction from '$lib/components/asset-viewer/actions/UnstackAction.svelte';
  import LoadingDots from '$lib/components/LoadingDots.svelte';
  import ButtonContextMenu from '$lib/components/shared-components/context-menu/ButtonContextMenu.svelte';
-  import MenuOption from '$lib/components/shared-components/context-menu/MenuOption.svelte';
  import RemoveFromAlbumAction from '$lib/components/timeline/actions/RemoveFromAlbumAction.svelte';
  import { assetViewerManager } from '$lib/managers/asset-viewer-manager.svelte';
  import { authManager } from '$lib/managers/auth-manager.svelte';
@@ -84,6 +82,27 @@
    shortcuts: [{ key: 'Escape' }],
  });

+  const PlayOriginalVideo: ActionItem = $derived({
+    title: playOriginalVideo ? $t('play_transcoded_video') : $t('play_original_video'),
+    icon: mdiVideoOutline,
+    $if: () => asset.type === AssetTypeEnum.Video,
+    onAction: () => setPlayOriginalVideo(!playOriginalVideo),
+  });
+
+  const ViewInTimeline: ActionItem = $derived({
+    title: $t('view_in_timeline'),
+    icon: mdiImageSearch,
+    $if: () => isOwner && !isLocked && !asset.isArchived && !asset.isTrashed,
+    onAction: () => goto(Route.photos({ at: stack?.primaryAssetId ?? asset.id })),
+  });
+
+  const ViewSimilar: ActionItem = $derived({
+    title: $t('view_similar_photos'),
+    icon: mdiCompare,
+    $if: () => !isLocked && !asset.isArchived && !asset.isTrashed && smartSearchEnabled,
+    onAction: () => goto(Route.search({ queryAssetId: stack?.primaryAssetId ?? asset.id })),
+  });
+
  const Actions = $derived(getAssetActions($t, asset));
  const sharedLink = getSharedLink();
 </script>
@@ -169,41 +188,21 @@
        {#if person}
          <SetFeaturedPhotoAction {asset} {person} {onAction} />
        {/if}
-        {#if asset.type === AssetTypeEnum.Image && !isLocked}
-          <SetProfilePictureAction {asset} />
-        {/if}

-        {#if !isLocked}
-          {#if isOwner}
-            <ArchiveAction {asset} {onAction} {preAction} />
-            {#if !asset.isArchived && !asset.isTrashed}
-              <MenuOption
-                icon={mdiImageSearch}
-                onClick={() => goto(Route.photos({ at: stack?.primaryAssetId ?? asset.id }))}
-                text={$t('view_in_timeline')}
-              />
-            {/if}
-          {/if}
-          {#if !asset.isArchived && !asset.isTrashed && smartSearchEnabled}
-            <MenuOption
-              icon={mdiCompare}
-              onClick={() => goto(Route.search({ queryAssetId: stack?.primaryAssetId ?? asset.id }))}
-              text={$t('view_similar_photos')}
-            />
-          {/if}
+        <ActionMenuItem action={Actions.SetProfilePicture} />
+
+        {#if isOwner && !isLocked}
+          <ArchiveAction {asset} {onAction} {preAction} />
        {/if}
+        <ActionMenuItem action={ViewInTimeline} />
+        <ActionMenuItem action={ViewSimilar} />

        {#if !asset.isTrashed && isOwner}
          <SetVisibilityAction asset={toTimelineAsset(asset)} {onAction} {preAction} />
        {/if}

-        {#if asset.type === AssetTypeEnum.Video}
-          <MenuOption
-            icon={mdiVideoOutline}
-            onClick={() => setPlayOriginalVideo(!playOriginalVideo)}
-            text={playOriginalVideo ? $t('play_transcoded_video') : $t('play_original_video')}
-          />
-        {/if}
+        <ActionMenuItem action={PlayOriginalVideo} />
+
        {#if isOwner}
          <hr />
          <ActionMenuItem action={Actions.RefreshFacesJob} />
@@ -23,7 +23,7 @@
    type AlbumResponseDto,
    type AssetResponseDto,
  } from '@immich/sdk';
-  import { Icon, IconButton, LoadingSpinner, Text } from '@immich/ui';
+  import { Icon, IconButton, Link, LoadingSpinner, Text } from '@immich/ui';
  import { mdiCamera, mdiCameraIris, mdiClose, mdiImageOutline, mdiInformationOutline } from '@mdi/js';
  import { onDestroy } from 'svelte';
  import { t } from 'svelte-i18n';
@@ -310,14 +310,13 @@
          {#snippet popup({ marker })}
            {@const { lat, lon } = marker}
            <div class="flex flex-col items-center gap-1">
-              <p class="font-bold">{lat.toPrecision(6)}, {lon.toPrecision(6)}</p>
-              <a
+              <Text fontWeight="bold">{lat.toPrecision(6)}, {lon.toPrecision(6)}</Text>
+              <Link
                href="https://www.openstreetmap.org/?mlat={lat}&mlon={lon}&zoom=13#map=15/{lat}/{lon}"
-                target="_blank"
-                class="font-medium text-primary underline focus:outline-none"
+                class="text-primary"
              >
                {$t('open_in_openstreetmap')}
-              </a>
+              </Link>
            </div>
          {/snippet}
        </Map>
@@ -1,20 +0,0 @@
-<script lang="ts">
-  import MenuOption from '$lib/components/shared-components/context-menu/MenuOption.svelte';
-  import ProfileImageCropperModal from '$lib/modals/ProfileImageCropperModal.svelte';
-  import type { AssetResponseDto } from '@immich/sdk';
-  import { modalManager } from '@immich/ui';
-  import { mdiAccountCircleOutline } from '@mdi/js';
-  import { t } from 'svelte-i18n';
-
-  interface Props {
-    asset: AssetResponseDto;
-  }
-
-  let { asset }: Props = $props();
-</script>
-
-<MenuOption
-  icon={mdiAccountCircleOutline}
-  onClick={() => modalManager.show(ProfileImageCropperModal, { asset })}
-  text={$t('set_as_profile_picture')}
-/>
@@ -342,7 +342,7 @@

        {#if !!assetOwner}
          <div class="absolute inset-e-2 bottom-1 z-2 max-w-[50%]">
-            <p class="max-w-full truncate text-xs font-medium text-white drop-shadow-lg">
+            <p class="text-white-shadow max-w-full truncate p-1 text-xs font-medium text-white">
              {assetOwner.name}
            </p>
          </div>
@@ -38,7 +38,6 @@
    Control,
    ControlButton,
    ControlGroup,
-    FullscreenControl,
    GeoJSON,
    GeolocateControl,
    MapLibre,
@@ -343,7 +342,6 @@

      {#if !simplified}
        <GeolocateControl position="top-left" />
-        <FullscreenControl position="top-left" />
        <ScaleControl />
        <AttributionControl compact={false} />
      {/if}
@@ -401,13 +399,13 @@
      >
        {#snippet children({ feature }: { feature: Feature })}
          {#if useLocationPin}
-            <Icon icon={mdiMapMarker} size="50px" class="translate-y-[-50%] text-primary" />
+            <Icon icon={mdiMapMarker} size="50px" class="translate-y-[calc(5px-50%)] text-primary" />
          {:else}
            <img
              src={getAssetMediaUrl({ id: feature.properties?.id })}
              class="size-15 rounded-full border-2 border-immich-primary bg-immich-primary object-cover shadow-lg transition-all duration-200 hover:scale-150 hover:border-immich-dark-primary"
              alt={feature.properties?.city && feature.properties.country
-                ? $t('map_marker_for_images', {
+                ? $t('map_marker_for_image', {
                    values: { city: feature.properties.city, country: feature.properties.country },
                  })
                : $t('map_marker_with_image')}
@@ -415,7 +413,7 @@
          {/if}
          {#if popup}
            <Popup offset={[0, -30]} openOn="click" closeOnClickOutside>
-              {@render popup?.({ marker: asMarker(feature) })}
+              {@render popup({ marker: asMarker(feature) })}
            </Popup>
          {/if}
        {/snippet}
@@ -11,6 +11,7 @@ import {
 } from '@immich/sdk';
 import { modalManager, toastManager, type ActionItem } from '@immich/ui';
 import {
+  mdiAccountCircleOutline,
  mdiAlertOutline,
  mdiCogRefreshOutline,
  mdiContentCopy,
@@ -41,6 +42,7 @@ import { authManager } from '$lib/managers/auth-manager.svelte';
 import { eventManager } from '$lib/managers/event-manager.svelte';
 import AssetAddToAlbumModal from '$lib/modals/AssetAddToAlbumModal.svelte';
 import AssetTagModal from '$lib/modals/AssetTagModal.svelte';
+import ProfileImageCropperModal from '$lib/modals/ProfileImageCropperModal.svelte';
 import SharedLinkCreateModal from '$lib/modals/SharedLinkCreateModal.svelte';
 import { SlideshowState, slideshowStore } from '$lib/stores/slideshow.store';
 import { getAssetMediaUrl, getSharedLink, sleep } from '$lib/utils';
@@ -242,6 +244,13 @@ export const getAssetActions = ($t: MessageFormatter, asset: AssetResponseDto) =
    shortcuts: [{ key: 'e' }],
  };

+  const SetProfilePicture: ActionItem = {
+    title: $t('set_as_profile_picture'),
+    icon: mdiAccountCircleOutline,
+    $if: () => asset.type === AssetTypeEnum.Image && asset.visibility !== AssetVisibility.Locked,
+    onAction: () => modalManager.show(ProfileImageCropperModal, { asset }),
+  };
+
  const RefreshFacesJob: ActionItem = {
    title: $t('refresh_faces'),
    icon: mdiHeadSyncOutline,
@@ -286,6 +295,7 @@ export const getAssetActions = ($t: MessageFormatter, asset: AssetResponseDto) =
    Tag,
    TagPeople,
    Edit,
+    SetProfilePicture,
    RefreshFacesJob,
    RefreshMetadataJob,
    RegenerateThumbnailJob,
@@ -169,7 +169,9 @@
            preload={false}
          />
          {#if person.name}
-            <span class="absolute inset-s-0 bottom-2 w-full px-1 text-center font-medium text-white select-text">
+            <span
+              class="text-white-shadow absolute inset-s-0 bottom-2 w-full px-1 text-center font-medium text-white select-text"
+            >
              {person.name}
            </span>
          {/if}
Author	SHA1	Message	Date
Mees Frensel	3fe3e0960c	refactor(web): simple actions	2026-06-22 18:10:51 +02:00
Brandon Wees	a5198e23a8	refactor: use SemVer classes for version compatability message (#29056 ) * refactor: use SemVer classes for version compatability message * chore: readd major version compatabilty messages * fix: remove 1.106.0 check (we dont support v1 servers anymore)	2026-06-22 11:28:56 -04:00
Mees Frensel	51f2905fcc	fix(web): remove map's fullscreen button (#29192 )	2026-06-22 16:58:07 +02:00
Vogeluff	3b7d75c18a	feat(web): Add text-white-shadow to elements and increase the shadows effect (#29165 ) * fix(web): increase text shadow strength for white text on thumbnails * fix(web): fix class order for text-white-shadow * fixup: format fix	2026-06-22 09:43:15 -05:00