chore(deps): update prom/prometheus docker digest to 2b6f734

docker/docker-compose.prod.yml

@@ -83,7 +83,7 @@ services:
     container_name: immich_prometheus
     ports:
       - 9090:9090
-    image: prom/prometheus@sha256:d936808bdea528155c0154a922cd42fd75716b8bb7ba302641350f9f3eaeba09
+    image: prom/prometheus@sha256:2b6f734e372c1b4717008f7d0a0152316aedd4d13ae17ef1e3268dbfaf68041b
     volumes:
       - ./prometheus.yml:/etc/prometheus/prometheus.yml
       - prometheus-data:/prometheus

mobile/lib/presentation/pages/drift_remote_album.page.dart

@@ -171,6 +171,67 @@ class _RemoteAlbumPageState extends ConsumerState<RemoteAlbumPage> {
     unawaited(context.pushRoute(DriftActivitiesRoute(album: _album)));
   }
 
+  Future<void> showOptionSheet(BuildContext context) async {
+    final user = ref.watch(currentUserProvider);
+    final isOwner = user != null ? user.id == _album.ownerId : false;
+    final canAddPhotos =
+        await ref.read(remoteAlbumServiceProvider).getUserRole(_album.id, user?.id ?? '') == AlbumUserRole.editor;
+
+    unawaited(
+      showModalBottomSheet(
+        context: context,
+        backgroundColor: context.colorScheme.surface,
+        isScrollControlled: false,
+        builder: (context) {
+          return DriftRemoteAlbumOption(
+            onDeleteAlbum: isOwner
+                ? () async {
+                    await deleteAlbum(context);
+                    if (context.mounted) {
+                      context.pop();
+                    }
+                  }
+                : null,
+            onAddUsers: isOwner
+                ? () async {
+                    await addUsers(context);
+                    context.pop();
+                  }
+                : null,
+            onAddPhotos: isOwner || canAddPhotos
+                ? () async {
+                    await addAssets(context);
+                    context.pop();
+                  }
+                : null,
+            onToggleAlbumOrder: isOwner
+                ? () async {
+                    await toggleAlbumOrder();
+                    context.pop();
+                  }
+                : null,
+            onEditAlbum: isOwner
+                ? () async {
+                    context.pop();
+                    await showEditTitleAndDescription(context);
+                  }
+                : null,
+            onCreateSharedLink: isOwner
+                ? () async {
+                    context.pop();
+                    unawaited(context.pushRoute(SharedLinkEditRoute(albumId: _album.id)));
+                  }
+                : null,
+            onShowOptions: () {
+              context.pop();
+              context.pushRoute(DriftAlbumOptionsRoute(album: _album));
+            },
+          );
+        },
+      ),
+    );
+  }
+
   @override
   Widget build(BuildContext context) {
     final user = ref.watch(currentUserProvider);
@@ -188,16 +249,8 @@ class _RemoteAlbumPageState extends ConsumerState<RemoteAlbumPage> {
       child: Timeline(
         appBar: RemoteAlbumSliverAppBar(
           icon: Icons.photo_album_outlined,
-          kebabMenu: _AlbumKebabMenu(
-            album: _album,
-            onDeleteAlbum: () => deleteAlbum(context),
-            onAddUsers: () => addUsers(context),
-            onAddPhotos: () => addAssets(context),
-            onToggleAlbumOrder: () => toggleAlbumOrder(),
-            onEditAlbum: () => showEditTitleAndDescription(context),
-            onCreateSharedLink: () => unawaited(context.pushRoute(SharedLinkEditRoute(albumId: _album.id))),
-            onShowOptions: () => context.pushRoute(DriftAlbumOptionsRoute(album: _album)),
-          ),
+          onShowOptions: () => showOptionSheet(context),
+          onToggleAlbumOrder: isOwner ? () => toggleAlbumOrder() : null,
           onEditTitle: isOwner ? () => showEditTitleAndDescription(context) : null,
           onActivity: () => showActivity(context),
         ),
@@ -361,77 +414,3 @@ class _EditAlbumDialogState extends ConsumerState<_EditAlbumDialog> {
     );
   }
 }
-
-class _AlbumKebabMenu extends ConsumerWidget {
-  final RemoteAlbum album;
-  final VoidCallback? onDeleteAlbum;
-  final VoidCallback? onAddUsers;
-  final VoidCallback? onAddPhotos;
-  final VoidCallback? onToggleAlbumOrder;
-  final VoidCallback? onEditAlbum;
-  final VoidCallback? onCreateSharedLink;
-  final VoidCallback? onShowOptions;
-
-  const _AlbumKebabMenu({
-    required this.album,
-    this.onDeleteAlbum,
-    this.onAddUsers,
-    this.onAddPhotos,
-    this.onToggleAlbumOrder,
-    this.onEditAlbum,
-    this.onCreateSharedLink,
-    this.onShowOptions,
-  });
-
-  double _calculateScrollProgress(FlexibleSpaceBarSettings? settings) {
-    if (settings?.maxExtent == null || settings?.minExtent == null) {
-      return 1.0;
-    }
-
-    final deltaExtent = settings!.maxExtent - settings.minExtent;
-    if (deltaExtent <= 0.0) {
-      return 1.0;
-    }
-
-    return (1.0 - (settings.currentExtent - settings.minExtent) / deltaExtent).clamp(0.0, 1.0);
-  }
-
-  @override
-  Widget build(BuildContext context, WidgetRef ref) {
-    final settings = context.dependOnInheritedWidgetOfExactType<FlexibleSpaceBarSettings>();
-    final scrollProgress = _calculateScrollProgress(settings);
-
-    final iconColor = Color.lerp(Colors.white, context.primaryColor, scrollProgress);
-    final iconShadows = [
-      if (scrollProgress < 0.95)
-        Shadow(offset: const Offset(0, 2), blurRadius: 5, color: Colors.black.withValues(alpha: 0.5))
-      else
-        const Shadow(offset: Offset(0, 2), blurRadius: 0, color: Colors.transparent),
-    ];
-
-    final user = ref.watch(currentUserProvider);
-    final isOwner = user != null && user.id == album.ownerId;
-
-    return FutureBuilder<bool>(
-      future: ref
-          .read(remoteAlbumServiceProvider)
-          .getUserRole(album.id, user?.id ?? '')
-          .then((role) => role == AlbumUserRole.editor),
-      builder: (context, snapshot) {
-        final canAddPhotos = snapshot.data ?? false;
-
-        return DriftRemoteAlbumOption(
-          iconColor: iconColor,
-          iconShadows: iconShadows,
-          onDeleteAlbum: isOwner ? onDeleteAlbum : null,
-          onAddUsers: isOwner ? onAddUsers : null,
-          onAddPhotos: isOwner || canAddPhotos ? onAddPhotos : null,
-          onToggleAlbumOrder: isOwner ? onToggleAlbumOrder : null,
-          onEditAlbum: isOwner ? onEditAlbum : null,
-          onCreateSharedLink: isOwner ? onCreateSharedLink : null,
-          onShowOptions: onShowOptions,
-        );
-      },
-    );
-  }
-}

mobile/lib/presentation/widgets/action_buttons/base_action_button.widget.dart

@@ -53,7 +53,7 @@ class BaseActionButton extends ConsumerWidget {
         style: MenuItemButton.styleFrom(alignment: Alignment.centerLeft, padding: const EdgeInsets.all(16)),
         leadingIcon: Icon(iconData, color: effectiveIconColor),
         onPressed: onPressed,
-        child: Text(label, style: theme.textTheme.labelLarge?.copyWith(fontSize: 16, color: iconColor)),
+        child: Text(label, style: theme.textTheme.labelLarge?.copyWith(fontSize: 16)),
       );
     }
 

mobile/lib/presentation/widgets/asset_viewer/asset_viewer.page.dart

@@ -92,8 +92,6 @@ class AssetViewer extends ConsumerStatefulWidget {
     if (asset.isVideo || asset.isMotionPhoto) {
       ref.read(videoPlaybackValueProvider.notifier).reset();
       ref.read(videoPlayerControlsProvider.notifier).pause();
-      // Hide controls by default for videos and motion photos
-      ref.read(assetViewerProvider.notifier).setControls(false);
     }
   }
 }
@@ -527,13 +525,7 @@ class _AssetViewerState extends ConsumerState<AssetViewer> {
 
   void _onScaleStateChanged(PhotoViewScaleState scaleState) {
     if (scaleState != PhotoViewScaleState.initial) {
-      ref.read(assetViewerProvider.notifier).setControls(false);
       ref.read(videoPlayerControlsProvider.notifier).pause();
-      return;
-    }
-
-    if (!showingBottomSheet) {
-      ref.read(assetViewerProvider.notifier).setControls(true);
     }
   }
 

mobile/lib/presentation/widgets/remote_album/drift_album_option.widget.dart

@@ -2,7 +2,6 @@ import 'package:flutter/material.dart';
 import 'package:hooks_riverpod/hooks_riverpod.dart';
 import 'package:immich_mobile/extensions/build_context_extensions.dart';
 import 'package:immich_mobile/extensions/translate_extensions.dart';
-import 'package:immich_mobile/presentation/widgets/action_buttons/base_action_button.widget.dart';
 
 class DriftRemoteAlbumOption extends ConsumerWidget {
   const DriftRemoteAlbumOption({
@@ -15,8 +14,6 @@ class DriftRemoteAlbumOption extends ConsumerWidget {
     this.onToggleAlbumOrder,
     this.onEditAlbum,
     this.onShowOptions,
-    this.iconColor,
-    this.iconShadows,
   });
 
   final VoidCallback? onAddPhotos;
@@ -27,131 +24,73 @@ class DriftRemoteAlbumOption extends ConsumerWidget {
   final VoidCallback? onToggleAlbumOrder;
   final VoidCallback? onEditAlbum;
   final VoidCallback? onShowOptions;
-  final Color? iconColor;
-  final List<Shadow>? iconShadows;
 
   @override
   Widget build(BuildContext context, WidgetRef ref) {
-    final theme = context.themeData;
-    final menuChildren = <Widget>[];
+    TextStyle textStyle = Theme.of(context).textTheme.bodyLarge!.copyWith(fontWeight: FontWeight.w600);
 
-    if (onEditAlbum != null) {
-      menuChildren.add(
-        BaseActionButton(
-          label: 'edit_album'.t(context: context),
-          iconData: Icons.edit,
-          onPressed: onEditAlbum,
-          menuItem: true,
+    return SafeArea(
+      child: Padding(
+        padding: const EdgeInsets.symmetric(vertical: 24.0),
+        child: ListView(
+          shrinkWrap: true,
+          children: [
+            if (onEditAlbum != null)
+              ListTile(
+                leading: const Icon(Icons.edit),
+                title: Text('edit_album'.t(context: context), style: textStyle),
+                onTap: onEditAlbum,
+              ),
+            if (onAddPhotos != null)
+              ListTile(
+                leading: const Icon(Icons.add_a_photo),
+                title: Text('add_photos'.t(context: context), style: textStyle),
+                onTap: onAddPhotos,
+              ),
+            if (onAddUsers != null)
+              ListTile(
+                leading: const Icon(Icons.group_add),
+                title: Text('album_viewer_page_share_add_users'.t(context: context), style: textStyle),
+                onTap: onAddUsers,
+              ),
+            if (onLeaveAlbum != null)
+              ListTile(
+                leading: const Icon(Icons.person_remove_rounded),
+                title: Text('leave_album'.t(context: context), style: textStyle),
+                onTap: onLeaveAlbum,
+              ),
+            if (onToggleAlbumOrder != null)
+              ListTile(
+                leading: const Icon(Icons.swap_vert_rounded),
+                title: Text('change_display_order'.t(context: context), style: textStyle),
+                onTap: onToggleAlbumOrder,
+              ),
+            if (onCreateSharedLink != null)
+              ListTile(
+                leading: const Icon(Icons.link),
+                title: Text('create_shared_link'.t(context: context), style: textStyle),
+                onTap: onCreateSharedLink,
+              ),
+            if (onShowOptions != null)
+              ListTile(
+                leading: const Icon(Icons.settings),
+                title: Text('options'.t(context: context), style: textStyle),
+                onTap: onShowOptions,
+              ),
+            if (onDeleteAlbum != null) ...[
+              const Divider(indent: 16, endIndent: 16),
+              ListTile(
+                leading: Icon(Icons.delete, color: context.isDarkTheme ? Colors.red[400] : Colors.red[800]),
+                title: Text(
+                  'delete_album'.t(context: context),
+                  style: textStyle.copyWith(color: context.isDarkTheme ? Colors.red[400] : Colors.red[800]),
+                ),
+                onTap: onDeleteAlbum,
+              ),
+            ],
+          ],
         ),
-      );
-    }
-
-    if (onAddPhotos != null) {
-      menuChildren.add(
-        BaseActionButton(
-          label: 'add_photos'.t(context: context),
-          iconData: Icons.add_a_photo,
-          onPressed: onAddPhotos,
-          menuItem: true,
-        ),
-      );
-    }
-
-    if (onAddUsers != null) {
-      menuChildren.add(
-        BaseActionButton(
-          label: 'album_viewer_page_share_add_users'.t(context: context),
-          iconData: Icons.group_add,
-          onPressed: onAddUsers,
-          menuItem: true,
-        ),
-      );
-    }
-
-    if (onLeaveAlbum != null) {
-      menuChildren.add(
-        BaseActionButton(
-          label: 'leave_album'.t(context: context),
-          iconData: Icons.person_remove_rounded,
-          onPressed: onLeaveAlbum,
-          menuItem: true,
-        ),
-      );
-    }
-
-    if (onToggleAlbumOrder != null) {
-      menuChildren.add(
-        BaseActionButton(
-          label: 'change_display_order'.t(context: context),
-          iconData: Icons.swap_vert_rounded,
-          onPressed: onToggleAlbumOrder,
-          menuItem: true,
-        ),
-      );
-    }
-
-    if (onCreateSharedLink != null) {
-      menuChildren.add(
-        BaseActionButton(
-          label: 'create_shared_link'.t(context: context),
-          iconData: Icons.link,
-          onPressed: onCreateSharedLink,
-          menuItem: true,
-        ),
-      );
-    }
-
-    if (onShowOptions != null) {
-      menuChildren.add(
-        BaseActionButton(
-          label: 'options'.t(context: context),
-          iconData: Icons.settings,
-          onPressed: onShowOptions,
-          menuItem: true,
-        ),
-      );
-    }
-
-    if (onDeleteAlbum != null) {
-      menuChildren.add(const Divider(height: 1));
-      menuChildren.add(
-        BaseActionButton(
-          label: 'delete_album'.t(context: context),
-          iconData: Icons.delete,
-          iconColor: context.isDarkTheme ? Colors.red[400] : Colors.red[800],
-          onPressed: onDeleteAlbum,
-          menuItem: true,
-        ),
-      );
-    }
-
-    return MenuAnchor(
-      consumeOutsideTap: true,
-      style: MenuStyle(
-        backgroundColor: WidgetStatePropertyAll(theme.scaffoldBackgroundColor),
-        surfaceTintColor: const WidgetStatePropertyAll(Colors.grey),
-        elevation: const WidgetStatePropertyAll(4),
-        shape: const WidgetStatePropertyAll(
-          RoundedRectangleBorder(borderRadius: BorderRadius.all(Radius.circular(12))),
-        ),
-        padding: const WidgetStatePropertyAll(EdgeInsets.symmetric(vertical: 6)),
       ),
-      menuChildren: [
-        ConstrainedBox(
-          constraints: const BoxConstraints(minWidth: 150),
-          child: Column(
-            mainAxisSize: MainAxisSize.min,
-            crossAxisAlignment: CrossAxisAlignment.stretch,
-            children: menuChildren,
-          ),
-        ),
-      ],
-      builder: (context, controller, child) {
-        return IconButton(
-          icon: Icon(Icons.more_vert_rounded, color: iconColor ?? Colors.white, shadows: iconShadows),
-          onPressed: () => controller.isOpen ? controller.close() : controller.open(),
-        );
-      },
     );
   }
 }

mobile/lib/providers/background_sync.provider.dart

@@ -5,21 +5,16 @@ import 'package:immich_mobile/providers/sync_status.provider.dart';
 
 final backgroundSyncProvider = Provider<BackgroundSyncManager>((ref) {
   final syncStatusNotifier = ref.read(syncStatusProvider.notifier);
+  final backupProvider = ref.read(driftBackupProvider.notifier);
 
   final manager = BackgroundSyncManager(
     onRemoteSyncStart: () {
       syncStatusNotifier.startRemoteSync();
-      final backupProvider = ref.read(driftBackupProvider.notifier);
-      if (backupProvider.mounted) {
-        backupProvider.updateError(BackupError.none);
-      }
+      backupProvider.updateError(BackupError.none);
     },
     onRemoteSyncComplete: (isSuccess) {
       syncStatusNotifier.completeRemoteSync();
-      final backupProvider = ref.read(driftBackupProvider.notifier);
-      if (backupProvider.mounted) {
-        backupProvider.updateError(isSuccess == true ? BackupError.none : BackupError.syncFailed);
-      }
+      backupProvider.updateError(isSuccess == true ? BackupError.none : BackupError.syncFailed);
     },
     onRemoteSyncError: syncStatusNotifier.errorRemoteSync,
     onLocalSyncStart: syncStatusNotifier.startLocalSync,

mobile/lib/providers/backup/drift_backup.provider.dart

@@ -212,8 +212,8 @@ class DriftBackupNotifier extends StateNotifier<DriftBackupState> {
         ),
       ) {
     {
-      _statusSubscription = _uploadService.taskStatusStream.listen(_handleTaskStatusUpdate);
-      _progressSubscription = _uploadService.taskProgressStream.listen(_handleTaskProgressUpdate);
+      _uploadService.taskStatusStream.listen(_handleTaskStatusUpdate);
+      _uploadService.taskProgressStream.listen(_handleTaskProgressUpdate);
     }
   }
 
@@ -224,10 +224,6 @@ class DriftBackupNotifier extends StateNotifier<DriftBackupState> {
 
   /// Remove upload item from state
   void _removeUploadItem(String taskId) {
-    if (!mounted) {
-      _logger.warning("Skip _removeUploadItem: notifier disposed");
-      return;
-    }
     if (state.uploadItems.containsKey(taskId)) {
       final updatedItems = Map<String, DriftUploadStatus>.from(state.uploadItems);
       updatedItems.remove(taskId);
@@ -236,10 +232,6 @@ class DriftBackupNotifier extends StateNotifier<DriftBackupState> {
   }
 
   void _handleTaskStatusUpdate(TaskStatusUpdate update) {
-    if (!mounted) {
-      _logger.warning("Skip _handleTaskStatusUpdate: notifier disposed");
-      return;
-    }
     final taskId = update.task.taskId;
 
     switch (update.status) {
@@ -299,10 +291,6 @@ class DriftBackupNotifier extends StateNotifier<DriftBackupState> {
   }
 
   void _handleTaskProgressUpdate(TaskProgressUpdate update) {
-    if (!mounted) {
-      _logger.warning("Skip _handleTaskProgressUpdate: notifier disposed");
-      return;
-    }
     final taskId = update.task.taskId;
     final filename = update.task.displayName;
     final progress = update.progress;
@@ -344,15 +332,7 @@ class DriftBackupNotifier extends StateNotifier<DriftBackupState> {
   }
 
   Future<void> getBackupStatus(String userId) async {
-    if (!mounted) {
-      _logger.warning("Skip getBackupStatus (pre-call): notifier disposed");
-      return;
-    }
     final counts = await _uploadService.getBackupCounts(userId);
-    if (!mounted) {
-      _logger.warning("Skip getBackupStatus (post-call): notifier disposed");
-      return;
-    }
 
     state = state.copyWith(
       totalCount: counts.total,
@@ -363,10 +343,6 @@ class DriftBackupNotifier extends StateNotifier<DriftBackupState> {
   }
 
   void updateError(BackupError error) async {
-    if (!mounted) {
-      _logger.warning("Skip updateError: notifier disposed");
-      return;
-    }
     state = state.copyWith(error: error);
   }
 
@@ -384,18 +360,10 @@ class DriftBackupNotifier extends StateNotifier<DriftBackupState> {
   }
 
   Future<void> cancel() async {
-    if (!mounted) {
-      _logger.warning("Skip cancel (pre-call): notifier disposed");
-      return;
-    }
     dPrint(() => "Canceling backup tasks...");
     state = state.copyWith(enqueueCount: 0, enqueueTotalCount: 0, isCanceling: true, error: BackupError.none);
 
     final activeTaskCount = await _uploadService.cancelBackup();
-    if (!mounted) {
-      _logger.warning("Skip cancel (post-call): notifier disposed");
-      return;
-    }
 
     if (activeTaskCount > 0) {
       dPrint(() => "$activeTaskCount tasks left, continuing to cancel...");
@@ -408,17 +376,9 @@ class DriftBackupNotifier extends StateNotifier<DriftBackupState> {
   }
 
   Future<void> handleBackupResume(String userId) async {
-    if (!mounted) {
-      _logger.warning("Skip handleBackupResume (pre-call): notifier disposed");
-      return;
-    }
     _logger.info("Resuming backup tasks...");
     state = state.copyWith(error: BackupError.none);
     final tasks = await _uploadService.getActiveTasks(kBackupGroup);
-    if (!mounted) {
-      _logger.warning("Skip handleBackupResume (post-call): notifier disposed");
-      return;
-    }
     _logger.info("Found ${tasks.length} tasks");
 
     if (tasks.isEmpty) {

mobile/lib/widgets/backup/backup_info_card.dart

@@ -53,7 +53,6 @@ class BackupInfoCard extends StatelessWidget {
                       info,
                       style: context.textTheme.titleLarge?.copyWith(
                         color: context.colorScheme.onSurface.withAlpha(isLoading ? 50 : 255),
-                        fontFeatures: const [FontFeature.tabularFigures()],
                       ),
                     ),
                     if (isLoading)

mobile/lib/widgets/common/remote_album_sliver_app_bar.dart

@@ -24,13 +24,15 @@ class RemoteAlbumSliverAppBar extends ConsumerStatefulWidget {
   const RemoteAlbumSliverAppBar({
     super.key,
     this.icon = Icons.camera,
-    required this.kebabMenu,
+    this.onShowOptions,
+    this.onToggleAlbumOrder,
     this.onEditTitle,
     this.onActivity,
   });
 
   final IconData icon;
-  final Widget kebabMenu;
+  final void Function()? onShowOptions;
+  final void Function()? onToggleAlbumOrder;
   final void Function()? onEditTitle;
   final void Function()? onActivity;
 
@@ -89,12 +91,21 @@ class _MesmerizingSliverAppBarState extends ConsumerState<RemoteAlbumSliverAppBa
               onPressed: () => context.maybePop(),
             ),
       actions: [
+        if (widget.onToggleAlbumOrder != null)
+          IconButton(
+            icon: Icon(Icons.swap_vert_rounded, color: actionIconColor, shadows: actionIconShadows),
+            onPressed: widget.onToggleAlbumOrder,
+          ),
         if (currentAlbum.isActivityEnabled && currentAlbum.isShared)
           IconButton(
             icon: Icon(Icons.chat_outlined, color: actionIconColor, shadows: actionIconShadows),
             onPressed: widget.onActivity,
           ),
-        widget.kebabMenu,
+        if (widget.onShowOptions != null)
+          IconButton(
+            icon: Icon(Icons.more_vert, color: actionIconColor, shadows: actionIconShadows),
+            onPressed: widget.onShowOptions,
+          ),
       ],
       title: Builder(
         builder: (context) {

mobile/test/presentation/widgets/remote_album/drift_album_option_widget_test.dart

@@ -1,500 +0,0 @@
-import 'package:flutter/material.dart';
-import 'package:flutter_test/flutter_test.dart';
-import 'package:immich_mobile/presentation/widgets/remote_album/drift_album_option.widget.dart';
-
-import '../../../widget_tester_extensions.dart';
-
-void main() {
-  group('DriftRemoteAlbumOption', () {
-    testWidgets('shows kebab menu icon button', (tester) async {
-      await tester.pumpConsumerWidget(
-        const DriftRemoteAlbumOption(),
-      );
-
-      expect(find.byIcon(Icons.more_vert_rounded), findsOneWidget);
-    });
-
-    testWidgets('opens menu when icon button is tapped', (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.edit), findsOneWidget);
-    });
-
-    testWidgets('shows edit album option when onEditAlbum is provided',
-        (tester) async {
-      bool editCalled = false;
-
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () => editCalled = true,
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.edit), findsOneWidget);
-
-      await tester.tap(find.byIcon(Icons.edit));
-      await tester.pumpAndSettle();
-
-      expect(editCalled, isTrue);
-    });
-
-    testWidgets('hides edit album option when onEditAlbum is null',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onAddPhotos: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.edit), findsNothing);
-    });
-
-    testWidgets('shows add photos option when onAddPhotos is provided',
-        (tester) async {
-      bool addPhotosCalled = false;
-
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onAddPhotos: () => addPhotosCalled = true,
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.add_a_photo), findsOneWidget);
-
-      await tester.tap(find.byIcon(Icons.add_a_photo));
-      await tester.pumpAndSettle();
-
-      expect(addPhotosCalled, isTrue);
-    });
-
-    testWidgets('hides add photos option when onAddPhotos is null',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.add_a_photo), findsNothing);
-    });
-
-    testWidgets('shows add users option when onAddUsers is provided',
-        (tester) async {
-      bool addUsersCalled = false;
-
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onAddUsers: () => addUsersCalled = true,
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.group_add), findsOneWidget);
-
-      await tester.tap(find.byIcon(Icons.group_add));
-      await tester.pumpAndSettle();
-
-      expect(addUsersCalled, isTrue);
-    });
-
-    testWidgets('hides add users option when onAddUsers is null',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.group_add), findsNothing);
-    });
-
-    testWidgets('shows leave album option when onLeaveAlbum is provided',
-        (tester) async {
-      bool leaveAlbumCalled = false;
-
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onLeaveAlbum: () => leaveAlbumCalled = true,
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.person_remove_rounded), findsOneWidget);
-
-      await tester.tap(find.byIcon(Icons.person_remove_rounded));
-      await tester.pumpAndSettle();
-
-      expect(leaveAlbumCalled, isTrue);
-    });
-
-    testWidgets('hides leave album option when onLeaveAlbum is null',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.person_remove_rounded), findsNothing);
-    });
-
-    testWidgets(
-        'shows toggle album order option when onToggleAlbumOrder is provided',
-        (tester) async {
-      bool toggleOrderCalled = false;
-
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onToggleAlbumOrder: () => toggleOrderCalled = true,
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.swap_vert_rounded), findsOneWidget);
-
-      await tester.tap(find.byIcon(Icons.swap_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(toggleOrderCalled, isTrue);
-    });
-
-    testWidgets('hides toggle album order option when onToggleAlbumOrder is null',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.swap_vert_rounded), findsNothing);
-    });
-
-    testWidgets(
-        'shows create shared link option when onCreateSharedLink is provided',
-        (tester) async {
-      bool createSharedLinkCalled = false;
-
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onCreateSharedLink: () => createSharedLinkCalled = true,
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.link), findsOneWidget);
-
-      await tester.tap(find.byIcon(Icons.link));
-      await tester.pumpAndSettle();
-
-      expect(createSharedLinkCalled, isTrue);
-    });
-
-    testWidgets('hides create shared link option when onCreateSharedLink is null',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.link), findsNothing);
-    });
-
-    testWidgets('shows options option when onShowOptions is provided',
-        (tester) async {
-      bool showOptionsCalled = false;
-
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onShowOptions: () => showOptionsCalled = true,
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.settings), findsOneWidget);
-
-      await tester.tap(find.byIcon(Icons.settings));
-      await tester.pumpAndSettle();
-
-      expect(showOptionsCalled, isTrue);
-    });
-
-    testWidgets('hides options option when onShowOptions is null',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.settings), findsNothing);
-    });
-
-    testWidgets('shows delete album option when onDeleteAlbum is provided',
-        (tester) async {
-      bool deleteAlbumCalled = false;
-
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onDeleteAlbum: () => deleteAlbumCalled = true,
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.delete), findsOneWidget);
-
-      await tester.tap(find.byIcon(Icons.delete));
-      await tester.pumpAndSettle();
-
-      expect(deleteAlbumCalled, isTrue);
-    });
-
-    testWidgets('hides delete album option when onDeleteAlbum is null',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.delete), findsNothing);
-    });
-
-    testWidgets('shows divider before delete album option', (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-          onDeleteAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byType(Divider), findsOneWidget);
-    });
-
-    testWidgets('shows all options when all callbacks are provided',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          onEditAlbum: () {},
-          onAddPhotos: () {},
-          onAddUsers: () {},
-          onLeaveAlbum: () {},
-          onToggleAlbumOrder: () {},
-          onCreateSharedLink: () {},
-          onShowOptions: () {},
-          onDeleteAlbum: () {},
-        ),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.edit), findsOneWidget);
-      expect(find.byIcon(Icons.add_a_photo), findsOneWidget);
-      expect(find.byIcon(Icons.group_add), findsOneWidget);
-      expect(find.byIcon(Icons.person_remove_rounded), findsOneWidget);
-      expect(find.byIcon(Icons.swap_vert_rounded), findsOneWidget);
-      expect(find.byIcon(Icons.link), findsOneWidget);
-      expect(find.byIcon(Icons.settings), findsOneWidget);
-      expect(find.byIcon(Icons.delete), findsOneWidget);
-      expect(find.byType(Divider), findsOneWidget);
-    });
-
-    testWidgets('shows no options when all callbacks are null', (tester) async {
-      await tester.pumpConsumerWidget(
-        const DriftRemoteAlbumOption(),
-      );
-
-      await tester.tap(find.byIcon(Icons.more_vert_rounded));
-      await tester.pumpAndSettle();
-
-      expect(find.byIcon(Icons.edit), findsNothing);
-      expect(find.byIcon(Icons.add_a_photo), findsNothing);
-      expect(find.byIcon(Icons.group_add), findsNothing);
-      expect(find.byIcon(Icons.person_remove_rounded), findsNothing);
-      expect(find.byIcon(Icons.swap_vert_rounded), findsNothing);
-      expect(find.byIcon(Icons.link), findsNothing);
-      expect(find.byIcon(Icons.settings), findsNothing);
-      expect(find.byIcon(Icons.delete), findsNothing);
-    });
-
-    testWidgets('uses custom icon color when provided', (tester) async {
-      const customColor = Colors.red;
-
-      await tester.pumpConsumerWidget(
-        const DriftRemoteAlbumOption(
-          iconColor: customColor,
-        ),
-      );
-
-      final iconButton = tester.widget<IconButton>(find.byType(IconButton));
-      final icon = iconButton.icon as Icon;
-
-      expect(icon.color, equals(customColor));
-    });
-
-    testWidgets('uses default white color when iconColor is null',
-        (tester) async {
-      await tester.pumpConsumerWidget(
-        const DriftRemoteAlbumOption(),
-      );
-
-      final iconButton = tester.widget<IconButton>(find.byType(IconButton));
-      final icon = iconButton.icon as Icon;
-
-      expect(icon.color, equals(Colors.white));
-    });
-
-    testWidgets('applies icon shadows when provided', (tester) async {
-      final shadows = [
-        const Shadow(offset: Offset(0, 2), blurRadius: 5, color: Colors.black),
-      ];
-
-      await tester.pumpConsumerWidget(
-        DriftRemoteAlbumOption(
-          iconShadows: shadows,
-        ),
-      );
-
-      final iconButton = tester.widget<IconButton>(find.byType(IconButton));
-      final icon = iconButton.icon as Icon;
-
-      expect(icon.shadows, equals(shadows));
-    });
-
-    group('owner vs non-owner scenarios', () {
-      testWidgets('owner sees all management options', (tester) async {
-        // Simulating owner scenario - all callbacks provided
-        await tester.pumpConsumerWidget(
-          DriftRemoteAlbumOption(
-            onEditAlbum: () {},
-            onAddPhotos: () {},
-            onAddUsers: () {},
-            onToggleAlbumOrder: () {},
-            onCreateSharedLink: () {},
-            onShowOptions: () {},
-            onDeleteAlbum: () {},
-          ),
-        );
-
-        await tester.tap(find.byIcon(Icons.more_vert_rounded));
-        await tester.pumpAndSettle();
-
-        // Owner should see all management options
-        expect(find.byIcon(Icons.edit), findsOneWidget);
-        expect(find.byIcon(Icons.add_a_photo), findsOneWidget);
-        expect(find.byIcon(Icons.group_add), findsOneWidget);
-        expect(find.byIcon(Icons.swap_vert_rounded), findsOneWidget);
-        expect(find.byIcon(Icons.link), findsOneWidget);
-        expect(find.byIcon(Icons.delete), findsOneWidget);
-        // Owner should NOT see leave album
-        expect(find.byIcon(Icons.person_remove_rounded), findsNothing);
-      });
-
-      testWidgets('non-owner with editor role sees limited options',
-          (tester) async {
-        // Simulating non-owner with editor role - can add photos, show options, leave
-        await tester.pumpConsumerWidget(
-          DriftRemoteAlbumOption(
-            onAddPhotos: () {},
-            onShowOptions: () {},
-            onLeaveAlbum: () {},
-          ),
-        );
-
-        await tester.tap(find.byIcon(Icons.more_vert_rounded));
-        await tester.pumpAndSettle();
-
-        // Editor can add photos
-        expect(find.byIcon(Icons.add_a_photo), findsOneWidget);
-        // Can see options
-        expect(find.byIcon(Icons.settings), findsOneWidget);
-        // Can leave album
-        expect(find.byIcon(Icons.person_remove_rounded), findsOneWidget);
-        // Cannot see owner-only options
-        expect(find.byIcon(Icons.edit), findsNothing);
-        expect(find.byIcon(Icons.group_add), findsNothing);
-        expect(find.byIcon(Icons.swap_vert_rounded), findsNothing);
-        expect(find.byIcon(Icons.link), findsNothing);
-        expect(find.byIcon(Icons.delete), findsNothing);
-      });
-
-      testWidgets('non-owner viewer sees minimal options', (tester) async {
-        // Simulating viewer - can only show options and leave
-        await tester.pumpConsumerWidget(
-          DriftRemoteAlbumOption(
-            onShowOptions: () {},
-            onLeaveAlbum: () {},
-          ),
-        );
-
-        await tester.tap(find.byIcon(Icons.more_vert_rounded));
-        await tester.pumpAndSettle();
-
-        // Can see options
-        expect(find.byIcon(Icons.settings), findsOneWidget);
-        // Can leave album
-        expect(find.byIcon(Icons.person_remove_rounded), findsOneWidget);
-        // Cannot see any other options
-        expect(find.byIcon(Icons.edit), findsNothing);
-        expect(find.byIcon(Icons.add_a_photo), findsNothing);
-        expect(find.byIcon(Icons.group_add), findsNothing);
-        expect(find.byIcon(Icons.swap_vert_rounded), findsNothing);
-        expect(find.byIcon(Icons.link), findsNothing);
-        expect(find.byIcon(Icons.delete), findsNothing);
-      });
-    });
-  });
-}

open-api/immich-openapi-specs.json

@@ -8020,6 +8020,55 @@
         "x-immich-state": "Alpha"
       }
     },
+    "/plugins/triggers": {
+      "get": {
+        "description": "Retrieve a list of all available plugin triggers.",
+        "operationId": "getPluginTriggers",
+        "parameters": [],
+        "responses": {
+          "200": {
+            "content": {
+              "application/json": {
+                "schema": {
+                  "items": {
+                    "$ref": "#/components/schemas/PluginTriggerResponseDto"
+                  },
+                  "type": "array"
+                }
+              }
+            },
+            "description": ""
+          }
+        },
+        "security": [
+          {
+            "bearer": []
+          },
+          {
+            "cookie": []
+          },
+          {
+            "api_key": []
+          }
+        ],
+        "summary": "List all plugin triggers",
+        "tags": [
+          "Plugins"
+        ],
+        "x-immich-history": [
+          {
+            "version": "v2.3.0",
+            "state": "Added"
+          },
+          {
+            "version": "v2.3.0",
+            "state": "Alpha"
+          }
+        ],
+        "x-immich-permission": "plugin.read",
+        "x-immich-state": "Alpha"
+      }
+    },
     "/plugins/{id}": {
       "get": {
         "description": "Retrieve information about a specific plugin by its ID.",
@@ -10332,6 +10381,21 @@
               "format": "uuid",
               "type": "string"
             }
+          },
+          {
+            "name": "id",
+            "required": false,
+            "in": "query",
+            "x-immich-history": [
+              {
+                "version": "v2.5.0",
+                "state": "Added"
+              }
+            ],
+            "schema": {
+              "format": "uuid",
+              "type": "string"
+            }
           }
         ],
         "responses": {
@@ -18282,7 +18346,7 @@
           },
           "supportedContexts": {
             "items": {
-              "$ref": "#/components/schemas/PluginContext"
+              "$ref": "#/components/schemas/PluginContextType"
             },
             "type": "array"
           },
@@ -18301,7 +18365,7 @@
         ],
         "type": "object"
       },
-      "PluginContext": {
+      "PluginContextType": {
         "enum": [
           "asset",
           "album",
@@ -18329,7 +18393,7 @@
           },
           "supportedContexts": {
             "items": {
-              "$ref": "#/components/schemas/PluginContext"
+              "$ref": "#/components/schemas/PluginContextType"
             },
             "type": "array"
           },
@@ -18401,6 +18465,29 @@
         ],
         "type": "object"
       },
+      "PluginTriggerResponseDto": {
+        "properties": {
+          "contextType": {
+            "allOf": [
+              {
+                "$ref": "#/components/schemas/PluginContextType"
+              }
+            ]
+          },
+          "type": {
+            "allOf": [
+              {
+                "$ref": "#/components/schemas/PluginTriggerType"
+              }
+            ]
+          }
+        },
+        "required": [
+          "contextType",
+          "type"
+        ],
+        "type": "object"
+      },
       "PluginTriggerType": {
         "enum": [
           "AssetCreate",
@@ -23316,11 +23403,11 @@
             "type": "string"
           },
           "triggerType": {
-            "enum": [
-              "AssetCreate",
-              "PersonRecognized"
-            ],
-            "type": "string"
+            "allOf": [
+              {
+                "$ref": "#/components/schemas/PluginTriggerType"
+              }
+            ]
           }
         },
         "required": [
@@ -23358,6 +23445,13 @@
           },
           "name": {
             "type": "string"
+          },
+          "triggerType": {
+            "allOf": [
+              {
+                "$ref": "#/components/schemas/PluginTriggerType"
+              }
+            ]
           }
         },
         "type": "object"

web/src/lib/actions/autogrow.ts

@@ -0,0 +1,19 @@
+import { tick } from 'svelte';
+import type { Action } from 'svelte/action';
+
+type Parameters = {
+  height?: string;
+  value: string; // added to enable reactivity
+};
+
+export const autoGrowHeight: Action<HTMLTextAreaElement, Parameters> = (textarea, { height = 'auto' }) => {
+  const update = () => {
+    void tick().then(() => {
+      textarea.style.height = height;
+      textarea.style.height = `${textarea.scrollHeight}px`;
+    });
+  };
+
+  update();
+  return { update };
+};

web/src/lib/components/album-page/album-description.svelte

@@ -1,10 +1,8 @@
 <script lang="ts">
-  import { shortcut } from '$lib/actions/shortcut';
-  import { handleError } from '$lib/utils/handle-error';
   import { updateAlbumInfo } from '@immich/sdk';
-  import { Textarea } from '@immich/ui';
+  import { handleError } from '$lib/utils/handle-error';
+  import AutogrowTextarea from '$lib/components/shared-components/autogrow-textarea.svelte';
   import { t } from 'svelte-i18n';
-  import { fromAction } from 'svelte/attachments';
 
   interface Props {
     id: string;
@@ -14,34 +12,27 @@
 
   let { id, description = $bindable(), isOwned }: Props = $props();
 
-  const handleFocusOut = async () => {
+  const handleUpdateDescription = async (newDescription: string) => {
     try {
       await updateAlbumInfo({
         id,
         updateAlbumDto: {
-          description,
+          description: newDescription,
         },
       });
     } catch (error) {
       handleError(error, $t('errors.unable_to_save_album'));
     }
+    description = newDescription;
   };
 </script>
 
 {#if isOwned}
-  <Textarea
-    bind:value={description}
-    class="outline-none border-b border-gray-500 bg-transparent ring-0 focus:ring-0 resize-none focus:border-b-2 focus:border-immich-primary dark:focus:border-immich-dark-primary dark:bg-transparent"
-    rows={1}
-    grow
-    shape="rectangle"
-    onfocusout={handleFocusOut}
+  <AutogrowTextarea
+    content={description}
+    class="w-full mt-2 text-black dark:text-white border-b-2 border-transparent border-gray-500 bg-transparent text-base outline-none transition-all focus:border-b-2 focus:border-immich-primary disabled:border-none dark:focus:border-immich-dark-primary hover:border-gray-400"
+    onContentUpdate={handleUpdateDescription}
     placeholder={$t('add_a_description')}
-    data-testid="autogrow-textarea"
-    {@attach fromAction(shortcut, () => ({
-      shortcut: { key: 'Enter', ctrl: true },
-      onShortcut: (e) => e.currentTarget.blur(),
-    }))}
   />
 {:else if description}
   <p class="break-words whitespace-pre-line w-full text-black dark:text-white text-base">

web/src/lib/components/asset-viewer/activity-viewer.svelte

@@ -1,5 +1,6 @@
 <script lang="ts">
   import { resolve } from '$app/paths';
+  import { autoGrowHeight } from '$lib/actions/autogrow';
   import { shortcut } from '$lib/actions/shortcut';
   import ButtonContextMenu from '$lib/components/shared-components/context-menu/button-context-menu.svelte';
   import MenuOption from '$lib/components/shared-components/context-menu/menu-option.svelte';
@@ -11,11 +12,10 @@
   import { handleError } from '$lib/utils/handle-error';
   import { isTenMinutesApart } from '$lib/utils/timesince';
   import { ReactionType, type ActivityResponseDto, type AssetTypeEnum, type UserResponseDto } from '@immich/sdk';
-  import { Icon, IconButton, LoadingSpinner, Textarea, toastManager } from '@immich/ui';
+  import { Icon, IconButton, LoadingSpinner, toastManager } from '@immich/ui';
   import { mdiClose, mdiDeleteOutline, mdiDotsVertical, mdiSend, mdiThumbUp } from '@mdi/js';
   import * as luxon from 'luxon';
   import { t } from 'svelte-i18n';
-  import { fromAction } from 'svelte/attachments';
   import UserAvatar from '../shared-components/user-avatar.svelte';
 
   const units: Intl.RelativeTimeFormatUnit[] = ['year', 'month', 'week', 'day', 'hour', 'minute', 'second'];
@@ -245,20 +245,19 @@
         </div>
         <form class="flex w-full max-h-56 gap-1" {onsubmit}>
           <div class="flex w-full items-center gap-4">
-            <Textarea
+            <textarea
               {disabled}
               bind:value={message}
-              rows={1}
-              grow
+              use:autoGrowHeight={{ height: '5px', value: message }}
               placeholder={disabled ? $t('comments_are_disabled') : $t('say_something')}
-              {@attach fromAction(shortcut, () => ({
+              use:shortcut={{
                 shortcut: { key: 'Enter' },
                 onShortcut: () => handleSendComment(),
-              }))}
+              }}
               class="h-4.5 {disabled
                 ? 'cursor-not-allowed'
-                : ''} w-full max-h-56 pe-2 items-center overflow-y-auto leading-4 outline-none resize-none bg-gray-200 dark:bg-gray-200"
-            ></Textarea>
+                : ''} w-full max-h-56 pe-2 items-center overflow-y-auto leading-4 outline-none resize-none bg-gray-200"
+            ></textarea>
           </div>
           {#if isSendingMessage}
             <div class="flex items-end place-items-center pb-2 ms-0">

web/src/lib/components/asset-viewer/detail-panel-description.svelte

@@ -1,10 +1,9 @@
 <script lang="ts">
-  import { shortcut } from '$lib/actions/shortcut';
+  import AutogrowTextarea from '$lib/components/shared-components/autogrow-textarea.svelte';
   import { handleError } from '$lib/utils/handle-error';
   import { updateAsset, type AssetResponseDto } from '@immich/sdk';
-  import { Textarea, toastManager } from '@immich/ui';
+  import { toastManager } from '@immich/ui';
   import { t } from 'svelte-i18n';
-  import { fromAction } from 'svelte/attachments';
 
   interface Props {
     asset: AssetResponseDto;
@@ -13,17 +12,15 @@
 
   let { asset, isOwner }: Props = $props();
 
-  let currentDescription = asset.exifInfo?.description ?? '';
-  let draftDescription = $state(currentDescription);
+  let description = $derived(asset.exifInfo?.description || '');
 
-  const handleFocusOut = async () => {
-    if (draftDescription === currentDescription) {
-      return;
-    }
+  const handleFocusOut = async (newDescription: string) => {
     try {
-      await updateAsset({ id: asset.id, updateAssetDto: { description: draftDescription } });
+      await updateAsset({ id: asset.id, updateAssetDto: { description: newDescription } });
+
+      asset.exifInfo = { ...asset.exifInfo, description: newDescription };
+
       toastManager.success($t('asset_description_updated'));
-      currentDescription = draftDescription;
     } catch (error) {
       handleError(error, $t('cannot_update_the_description'));
     }
@@ -32,23 +29,15 @@
 
 {#if isOwner}
   <section class="px-4 mt-10">
-    <Textarea
-      bind:value={draftDescription}
-      class="max-h-40 outline-none border-b border-gray-500 bg-transparent ring-0 focus:ring-0 resize-none focus:border-b-2 focus:border-immich-primary dark:focus:border-immich-dark-primary dark:bg-transparent"
-      rows={1}
-      grow
-      shape="rectangle"
-      onfocusout={handleFocusOut}
+    <AutogrowTextarea
+      content={description}
+      class="max-h-125 w-full border-b border-gray-500 bg-transparent text-base text-black outline-none transition-all focus:border-b-2 focus:border-immich-primary disabled:border-none dark:text-white dark:focus:border-immich-dark-primary immich-scrollbar"
+      onContentUpdate={handleFocusOut}
       placeholder={$t('add_a_description')}
-      data-testid="autogrow-textarea"
-      {@attach fromAction(shortcut, () => ({
-        shortcut: { key: 'Enter', ctrl: true },
-        onShortcut: (e) => e.currentTarget.blur(),
-      }))}
     />
   </section>
-{:else if draftDescription}
+{:else if description}
   <section class="px-4 mt-6">
-    <p class="wrap-break-word whitespace-pre-line w-full text-black dark:text-white text-base">{draftDescription}</p>
+    <p class="wrap-break-word whitespace-pre-line w-full text-black dark:text-white text-base">{description}</p>
   </section>
 {/if}

web/src/lib/components/shared-components/autogrow-textarea.spec.ts

@@ -0,0 +1,60 @@
+import AutogrowTextarea from '$lib/components/shared-components/autogrow-textarea.svelte';
+import { render, screen, waitFor } from '@testing-library/svelte';
+import userEvent from '@testing-library/user-event';
+
+describe('AutogrowTextarea component', () => {
+  const getTextarea = () => screen.getByTestId('autogrow-textarea') as HTMLTextAreaElement;
+
+  it('should render correctly', () => {
+    render(AutogrowTextarea);
+    const textarea = getTextarea();
+    expect(textarea).toBeInTheDocument();
+  });
+
+  it('should show the content passed to the component', () => {
+    render(AutogrowTextarea, { content: 'stuff' });
+    const textarea = getTextarea();
+    expect(textarea.value).toBe('stuff');
+  });
+
+  it('should show the placeholder passed to the component', () => {
+    render(AutogrowTextarea, { placeholder: 'asdf' });
+    const textarea = getTextarea();
+    expect(textarea.placeholder).toBe('asdf');
+  });
+
+  it('should execute the passed callback on blur', async () => {
+    const user = userEvent.setup();
+    const update = vi.fn();
+    render(AutogrowTextarea, { content: 'existing', onContentUpdate: update });
+    const textarea = getTextarea();
+    await user.click(textarea);
+    await user.keyboard('extra');
+    textarea.blur();
+    await waitFor(() => expect(update).toHaveBeenCalledWith('existingextra'));
+  });
+
+  it('should execute the passed callback when pressing ctrl+enter in the textarea', async () => {
+    const user = userEvent.setup();
+    const update = vi.fn();
+    render(AutogrowTextarea, { onContentUpdate: update });
+    const textarea = getTextarea();
+    await user.click(textarea);
+    const string = 'content';
+    await user.keyboard(string);
+    await user.keyboard('{Control>}{Enter}{/Control}');
+    await waitFor(() => expect(update).toHaveBeenCalledWith(string));
+  });
+
+  it('should not execute the passed callback if the text has not changed', async () => {
+    const user = userEvent.setup();
+    const update = vi.fn();
+    render(AutogrowTextarea, { content: 'initial', onContentUpdate: update });
+    const textarea = getTextarea();
+    await user.click(textarea);
+    await user.clear(textarea);
+    await user.keyboard('initial');
+    await user.keyboard('{Control>}{Enter}{/Control}');
+    await waitFor(() => expect(update).not.toHaveBeenCalled());
+  });
+});

web/src/lib/components/shared-components/autogrow-textarea.svelte

@@ -0,0 +1,35 @@
+<script lang="ts">
+  import { autoGrowHeight } from '$lib/actions/autogrow';
+  import { shortcut } from '$lib/actions/shortcut';
+
+  interface Props {
+    content?: string;
+    class?: string;
+    onContentUpdate?: (newContent: string) => void;
+    placeholder?: string;
+  }
+
+  let { content = '', class: className = '', onContentUpdate = () => null, placeholder = '' }: Props = $props();
+
+  let newContent = $derived(content);
+
+  const updateContent = () => {
+    if (content === newContent) {
+      return;
+    }
+    onContentUpdate(newContent);
+  };
+</script>
+
+<textarea
+  bind:value={newContent}
+  class="resize-none {className}"
+  onfocusout={updateContent}
+  {placeholder}
+  use:shortcut={{
+    shortcut: { key: 'Enter', ctrl: true },
+    onShortcut: (e) => e.currentTarget.blur(),
+  }}
+  use:autoGrowHeight={{ value: newContent }}
+  data-testid="autogrow-textarea">{content}</textarea
+>

web/src/lib/managers/timeline-manager/timeline-manager.svelte.ts

@@ -346,7 +346,7 @@ export class TimelineManager extends VirtualScrollManager {
 
   async findMonthGroupForAsset(asset: AssetDescriptor | AssetResponseDto) {
     if (!this.isInitialized) {
-      await this.initTask.waitUntilExecution();
+      await this.initTask.waitUntilCompletion();
     }
 
     const { id } = asset;

web/src/lib/utils/date-time.ts

@@ -27,9 +27,6 @@ export const getShortDateRange = (startDate: string | Date, endDate: string | Da
   const endDateLocalized = endDate.toLocaleString(userLocale, {
     month: 'short',
     year: 'numeric',
-    // The API returns the date in UTC. If the earliest asset was taken on Jan 1st at 1am,
-    // we expect the album to start in January, even if the local timezone is UTC-5 for instance.
-    timeZone: 'UTC',
   });
 
   if (startDate.getFullYear() === endDate.getFullYear()) {