yujunjun/AppFlowy
1
0
Fork 0
mirror of https://github.com/AppFlowy-IO/AppFlowy.git synced 2025-11-04 03:54:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1317 from alemoreau/grid-incomplete_display_of_the_property_names

Browse Source 
fix: add overflow ellipsis in FieldCellButton
This commit is contained in:
Lucas.Xu 2022-10-19 08:13:43 +07:00 committed by GitHub
commit 31ed88364a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
frontend/app_flowy/lib/plugins/grid/presentation/widgets/header/field_cell.dart
View File

@ -161,15 +161,22 @@ class FieldCellButton extends StatelessWidget {
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
final theme = context.watch<AppTheme>(); final theme = context.watch<AppTheme>();
// Using this technique to have proper text ellipsis
// https://github.com/flutter/flutter/issues/18761#issuecomment-812390920
final text = Characters(field.name)
.replaceAll(Characters(''), Characters('\u{200B}'))
.toString();
return FlowyButton( return FlowyButton(
radius: BorderRadius.zero, radius: BorderRadius.zero,
hoverColor: theme.shader6, hoverColor: theme.shader6,
onTap: onTap, onTap: onTap,
leftIcon: svgWidget(field.fieldType.iconName(), color: theme.iconColor), leftIcon: svgWidget(field.fieldType.iconName(), color: theme.iconColor),
text: FlowyText.medium( text: FlowyText.medium(
field.name, text,
fontSize: 12, fontSize: 12,
maxLines: maxLines, maxLines: maxLines,
overflow: TextOverflow.ellipsis,
), ),
margin: GridSize.cellContentInsets, margin: GridSize.cellContentInsets,
); );