Skip to content

Commit d5a25e6

Browse files
committed
msglist: Move star icon 2px away from the edge of the screen
Fixes: #1247
1 parent a055486 commit d5a25e6

File tree

1 file changed

+18
-5
lines changed

1 file changed

+18
-5
lines changed

lib/widgets/message_list.dart

+18-5
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import 'dart:math';
33
import 'package:collection/collection.dart';
44
import 'package:flutter/material.dart';
55
import 'package:flutter_color_models/flutter_color_models.dart';
6-
import 'package:intl/intl.dart';
6+
import 'package:intl/intl.dart' hide TextDirection;
77

88
import '../api/model/model.dart';
99
import '../generated/l10n/zulip_localizations.dart';
@@ -1387,6 +1387,19 @@ class MessageWithPossibleSender extends StatelessWidget {
13871387
case MessageEditState.none:
13881388
}
13891389

1390+
final starOffset = switch (Directionality.of(context)) {
1391+
TextDirection.ltr => -2.0,
1392+
TextDirection.rtl => 2.0,
1393+
};
1394+
1395+
Widget? star;
1396+
if (message.flags.contains(MessageFlag.starred)) {
1397+
star = Transform.translate(
1398+
offset: Offset(starOffset, 0),
1399+
child: Icon(ZulipIcons.star_filled, size: 16, color: designVariables.star)
1400+
);
1401+
}
1402+
13901403
return GestureDetector(
13911404
behavior: HitTestBehavior.translucent,
13921405
onLongPress: () => showMessageActionSheet(context: context, message: message),
@@ -1417,10 +1430,10 @@ class MessageWithPossibleSender extends StatelessWidget {
14171430
letterSpacing: proportionalLetterSpacing(
14181431
context, 0.05, baseFontSize: 12))),
14191432
])),
1420-
SizedBox(width: 16,
1421-
child: message.flags.contains(MessageFlag.starred)
1422-
? Icon(ZulipIcons.star_filled, size: 16, color: designVariables.star)
1423-
: null),
1433+
SizedBox(
1434+
width: 16,
1435+
child: star,
1436+
),
14241437
]),
14251438
])));
14261439
}

0 commit comments

Comments
 (0)