From 1fdd7c343bb0e596e819e89eb9d9965404a91498 Mon Sep 17 00:00:00 2001 From: Richard Shiue <71320345+richardshiue@users.noreply.github.com> Date: Fri, 14 Mar 2025 11:31:05 +0800 Subject: [PATCH] chore: use tooltip instead of multi line for related questions (#7533) --- .../presentation/chat_related_question.dart | 38 ++++++++++--------- 1 file changed, 21 insertions(+), 17 deletions(-) diff --git a/frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_related_question.dart b/frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_related_question.dart index cc21cfc80e..34427479fe 100644 --- a/frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_related_question.dart +++ b/frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_related_question.dart @@ -3,6 +3,7 @@ import 'package:appflowy/generated/locale_keys.g.dart'; import 'package:easy_localization/easy_localization.dart'; import 'package:flowy_infra_ui/style_widget/button.dart'; import 'package:flowy_infra_ui/style_widget/text.dart'; +import 'package:flowy_infra_ui/widget/flowy_tooltip.dart'; import 'package:flowy_infra_ui/widget/spacing.dart'; import 'package:flutter/material.dart'; import 'package:universal_platform/universal_platform.dart'; @@ -66,25 +67,28 @@ class RelatedQuestionItem extends StatelessWidget { @override Widget build(BuildContext context) { - return FlowyButton( - mainAxisAlignment: MainAxisAlignment.start, - text: Flexible( - child: FlowyText( - question, - lineHeight: 1.4, - maxLines: null, + return FlowyTooltip( + message: question, + child: FlowyButton( + mainAxisAlignment: MainAxisAlignment.start, + text: Flexible( + child: FlowyText( + question, + lineHeight: 1.4, + overflow: TextOverflow.ellipsis, + ), ), + expandText: false, + margin: UniversalPlatform.isMobile + ? const EdgeInsets.symmetric(horizontal: 8.0, vertical: 4.0) + : const EdgeInsets.all(8.0), + leftIcon: FlowySvg( + FlowySvgs.ai_chat_outlined_s, + color: Theme.of(context).colorScheme.primary, + size: const Size.square(16.0), + ), + onTap: () => onQuestionSelected(question), ), - expandText: false, - margin: UniversalPlatform.isMobile - ? const EdgeInsets.symmetric(horizontal: 8.0, vertical: 4.0) - : const EdgeInsets.all(8.0), - leftIcon: FlowySvg( - FlowySvgs.ai_chat_outlined_s, - color: Theme.of(context).colorScheme.primary, - size: const Size.square(16.0), - ), - onTap: () => onQuestionSelected(question), ); } }