Skip to content

Commit baef2c7

Browse files
theme [nfc]: Replace dateSeparator with foreground
The variable giving color to divider lines in the Message List UI i.e. `dateSeparator` has been replaced with the `foreground` variable of `DesignVariables`, according to the Figma design. There isn't any change in the values of color from the previous one i.e. `dateSeparator`. Figma: https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=4993-20811&t=vu2wWOBiVb0BB6Vz-0 discussion: #1213 (comment) Fixes: #973
1 parent 4a4c9ff commit baef2c7

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

lib/widgets/message_list.dart

+2-8
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ import 'theme.dart';
3030
class MessageListTheme extends ThemeExtension<MessageListTheme> {
3131
static final light = MessageListTheme._(
3232
bgMessageRegular: const HSLColor.fromAHSL(1, 0, 0, 1).toColor(),
33-
dateSeparator: Colors.black,
3433
dmRecipientHeaderBg: const HSLColor.fromAHSL(1, 46, 0.35, 0.93).toColor(),
3534
labelTime: const HSLColor.fromAHSL(0.49, 0, 0, 0).toColor(),
3635
senderBotIcon: const HSLColor.fromAHSL(1, 180, 0.08, 0.65).toColor(),
@@ -52,7 +51,6 @@ class MessageListTheme extends ThemeExtension<MessageListTheme> {
5251

5352
static final dark = MessageListTheme._(
5453
bgMessageRegular: const HSLColor.fromAHSL(1, 0, 0, 0.11).toColor(),
55-
dateSeparator: Colors.white,
5654
dmRecipientHeaderBg: const HSLColor.fromAHSL(1, 46, 0.15, 0.2).toColor(),
5755
labelTime: const HSLColor.fromAHSL(0.5, 0, 0, 1).toColor(),
5856
senderBotIcon: const HSLColor.fromAHSL(1, 180, 0.05, 0.5).toColor(),
@@ -73,7 +71,6 @@ class MessageListTheme extends ThemeExtension<MessageListTheme> {
7371

7472
MessageListTheme._({
7573
required this.bgMessageRegular,
76-
required this.dateSeparator,
7774
required this.dmRecipientHeaderBg,
7875
required this.labelTime,
7976
required this.senderBotIcon,
@@ -94,7 +91,6 @@ class MessageListTheme extends ThemeExtension<MessageListTheme> {
9491
}
9592

9693
final Color bgMessageRegular;
97-
final Color dateSeparator;
9894
final Color dmRecipientHeaderBg;
9995
final Color labelTime;
10096
final Color senderBotIcon;
@@ -106,7 +102,6 @@ class MessageListTheme extends ThemeExtension<MessageListTheme> {
106102
@override
107103
MessageListTheme copyWith({
108104
Color? bgMessageRegular,
109-
Color? dateSeparator,
110105
Color? dmRecipientHeaderBg,
111106
Color? labelTime,
112107
Color? senderBotIcon,
@@ -117,7 +112,6 @@ class MessageListTheme extends ThemeExtension<MessageListTheme> {
117112
}) {
118113
return MessageListTheme._(
119114
bgMessageRegular: bgMessageRegular ?? this.bgMessageRegular,
120-
dateSeparator: dateSeparator ?? this.dateSeparator,
121115
dmRecipientHeaderBg: dmRecipientHeaderBg ?? this.dmRecipientHeaderBg,
122116
labelTime: labelTime ?? this.labelTime,
123117
senderBotIcon: senderBotIcon ?? this.senderBotIcon,
@@ -135,7 +129,6 @@ class MessageListTheme extends ThemeExtension<MessageListTheme> {
135129
}
136130
return MessageListTheme._(
137131
bgMessageRegular: Color.lerp(bgMessageRegular, other.bgMessageRegular, t)!,
138-
dateSeparator: Color.lerp(dateSeparator, other.dateSeparator, t)!,
139132
dmRecipientHeaderBg: Color.lerp(dmRecipientHeaderBg, other.dmRecipientHeaderBg, t)!,
140133
labelTime: Color.lerp(labelTime, other.labelTime, t)!,
141134
senderBotIcon: Color.lerp(senderBotIcon, other.senderBotIcon, t)!,
@@ -909,8 +902,9 @@ class DateSeparator extends StatelessWidget {
909902
const textBottomPadding = 2.0;
910903

911904
final messageListTheme = MessageListTheme.of(context);
905+
final designVariables = DesignVariables.of(context);
912906

913-
final line = BorderSide(width: 0, color: messageListTheme.dateSeparator);
907+
final line = BorderSide(width: 0, color: designVariables.foreground);
914908

915909
// TODO(#681) use different color for DM messages
916910
return ColoredBox(color: messageListTheme.bgMessageRegular,

0 commit comments

Comments
 (0)