💄 Optimize message actions

This commit is contained in:
2025-10-09 00:38:45 +08:00
parent 77bae4d6fd
commit 3bf815ac61
2 changed files with 164 additions and 30 deletions

View File

@@ -1058,6 +1058,7 @@
"iframeCodeHint": "<iframe src=\"...\" width=\"...\" height=\"...\">", "iframeCodeHint": "<iframe src=\"...\" width=\"...\" height=\"...\">",
"parseIframe": "Parse Iframe", "parseIframe": "Parse Iframe",
"messageActions": "Message Actions", "messageActions": "Message Actions",
"messageContent": "Message Content",
"viewEmbedLoadHint": "Tap to load", "viewEmbedLoadHint": "Tap to load",
"levelingStage1": "Novice", "levelingStage1": "Novice",
"levelingStage2": "Apprentice", "levelingStage2": "Apprentice",

View File

@@ -179,7 +179,7 @@ class MessageItem extends HookConsumerWidget {
onJump(message.meta['message_id']); onJump(message.meta['message_id']);
} }
}, },
child: Container( child: SizedBox(
width: double.infinity, width: double.infinity,
child: MouseRegion( child: MouseRegion(
onEnter: (_) => isHovered.value = true, onEnter: (_) => isHovered.value = true,
@@ -244,7 +244,7 @@ class MessageItem extends HookConsumerWidget {
} }
} }
class MessageActionSheet extends StatelessWidget { class MessageActionSheet extends StatefulWidget {
final bool isCurrentUser; final bool isCurrentUser;
final Function(String action)? onAction; final Function(String action)? onAction;
final bool translatableLanguage; final bool translatableLanguage;
@@ -266,79 +266,178 @@ class MessageActionSheet extends StatelessWidget {
required this.remoteMessage, required this.remoteMessage,
}); });
@override
State<MessageActionSheet> createState() => _MessageActionSheetState();
}
class _MessageActionSheetState extends State<MessageActionSheet> {
bool _isExpanded = false;
static const int _maxPreviewLines = 3;
String get _displayContent {
return widget.translatedText ?? widget.remoteMessage.content ?? '';
}
bool get _shouldShowExpandButton {
// Simple check: show expand button if content is not empty
// The actual line limiting is handled by maxLines in SelectableText
return (widget.translatedText ?? widget.remoteMessage.content ?? '')
.isNotEmpty;
}
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
return SheetScaffold( return SheetScaffold(
titleText: 'messageActions'.tr(), titleText: 'messageActions'.tr(),
child: SingleChildScrollView( child: SingleChildScrollView(
child: Column( child: Column(
crossAxisAlignment: CrossAxisAlignment.start,
mainAxisSize: MainAxisSize.min,
children: [ children: [
// Message content preview section
if (widget.remoteMessage.content?.isNotEmpty ?? false) ...[
Container(
margin: const EdgeInsets.fromLTRB(16, 16, 16, 0),
padding: const EdgeInsets.all(12),
decoration: BoxDecoration(
color: Theme.of(
context,
).colorScheme.surfaceContainerHighest.withOpacity(0.3),
borderRadius: BorderRadius.circular(12),
border: Border.all(
color: Theme.of(
context,
).colorScheme.outlineVariant.withOpacity(0.5),
width: 1,
),
),
child: Column(
crossAxisAlignment: CrossAxisAlignment.start,
mainAxisSize: MainAxisSize.min,
children: [
// Header
Row(
children: [
Icon(
Symbols.article,
size: 16,
color: Theme.of(context).colorScheme.primary,
),
const Gap(6),
Text(
'messageContent'.tr(),
style: TextStyle(
fontSize: 12,
fontWeight: FontWeight.w500,
color: Theme.of(context).colorScheme.primary,
),
),
const Spacer(),
if (_shouldShowExpandButton)
IconButton(
icon: Icon(
_isExpanded
? Symbols.expand_less
: Symbols.expand_more,
size: 16,
),
onPressed: () {
setState(() {
_isExpanded = !_isExpanded;
});
},
padding: EdgeInsets.zero,
constraints: const BoxConstraints(
minWidth: 24,
minHeight: 24,
),
),
],
),
const Gap(8),
// Selectable content
SelectableText(
_displayContent,
style: TextStyle(
fontSize: 14,
height: 1.4,
color: Theme.of(context).colorScheme.onSurfaceVariant,
),
minLines: 1,
maxLines: _isExpanded ? null : _maxPreviewLines,
textAlign: TextAlign.start,
),
],
),
),
const Gap(4), const Gap(4),
if (isCurrentUser) ],
ListTile(
const Gap(8),
// Action buttons
if (widget.isCurrentUser)
_ActionListTile(
leading: Icon(Symbols.edit), leading: Icon(Symbols.edit),
title: Text('edit'.tr()), title: Text('edit'.tr()),
minTileHeight: 48,
onTap: () { onTap: () {
onAction!.call(MessageItemAction.edit); widget.onAction!.call(MessageItemAction.edit);
Navigator.pop(context); Navigator.pop(context);
}, },
), ),
if (isCurrentUser) if (widget.isCurrentUser)
ListTile( _ActionListTile(
leading: Icon(Symbols.delete), leading: Icon(Symbols.delete),
title: Text('delete'.tr()), title: Text('delete'.tr()),
minTileHeight: 48,
onTap: () { onTap: () {
onAction!.call(MessageItemAction.delete); widget.onAction!.call(MessageItemAction.delete);
Navigator.pop(context); Navigator.pop(context);
}, },
), ),
if (isCurrentUser) const Divider(height: 8), if (widget.isCurrentUser) const Divider(height: 8),
ListTile(
_ActionListTile(
leading: Icon(Symbols.reply), leading: Icon(Symbols.reply),
title: Text('reply'.tr()), title: Text('reply'.tr()),
minTileHeight: 48,
onTap: () { onTap: () {
onAction!.call(MessageItemAction.reply); widget.onAction!.call(MessageItemAction.reply);
Navigator.pop(context); Navigator.pop(context);
}, },
), ),
ListTile( _ActionListTile(
leading: Icon(Symbols.forward), leading: Icon(Symbols.forward),
title: Text('forward'.tr()), title: Text('forward'.tr()),
minTileHeight: 48,
onTap: () { onTap: () {
onAction!.call(MessageItemAction.forward); widget.onAction!.call(MessageItemAction.forward);
Navigator.pop(context); Navigator.pop(context);
}, },
), ),
if (translatableLanguage) const Divider(height: 8),
if (translatableLanguage) if (widget.translatableLanguage) const Divider(height: 8),
ListTile( if (widget.translatableLanguage)
_ActionListTile(
leading: Icon(Symbols.translate), leading: Icon(Symbols.translate),
minTileHeight: 48,
title: Text( title: Text(
translatedText == null widget.translatedText == null
? 'translate'.tr() ? 'translate'.tr()
: translating : widget.translating
? 'translating'.tr() ? 'translating'.tr()
: 'translated'.tr(), : 'translated'.tr(),
), ),
onTap: () { onTap: () {
translate(); widget.translate();
Navigator.pop(context); Navigator.pop(context);
}, },
), ),
if (isMobile) const Divider(height: 8),
if (isMobile) if (widget.isMobile) const Divider(height: 8),
ListTile( if (widget.isMobile)
_ActionListTile(
leading: Icon(Symbols.copy_all), leading: Icon(Symbols.copy_all),
title: Text('copyMessage'.tr()), title: Text('copyMessage'.tr()),
minTileHeight: 48,
onTap: () { onTap: () {
Clipboard.setData( Clipboard.setData(
ClipboardData(text: remoteMessage.content ?? ''), ClipboardData(text: widget.remoteMessage.content ?? ''),
); );
Navigator.pop(context); Navigator.pop(context);
}, },
@@ -350,6 +449,40 @@ class MessageActionSheet extends StatelessWidget {
} }
} }
class _ActionListTile extends StatelessWidget {
final Widget leading;
final Widget title;
final VoidCallback onTap;
const _ActionListTile({
required this.leading,
required this.title,
required this.onTap,
});
@override
Widget build(BuildContext context) {
return InkWell(
onTap: onTap,
child: Padding(
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 8),
child: Row(
children: [
SizedBox(width: 24, height: 24, child: leading),
const Gap(12),
Expanded(child: title),
Icon(
Symbols.chevron_right,
size: 16,
color: Theme.of(context).colorScheme.outline,
),
],
),
),
);
}
}
class MessageHoverActionMenu extends StatelessWidget { class MessageHoverActionMenu extends StatelessWidget {
final bool isCurrentUser; final bool isCurrentUser;
final Function(String action)? onAction; final Function(String action)? onAction;