Skip to content

Commit 9275940

Browse files
committed
msglist: Follow web in recipient-header-date color
The old color, which we started using for stream headers in e97e6fd and DM headers in 81f7038, doesn't match the Figma: https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=538-20849&t=ubXJTWC5kuz5awbN-0 #666666 or the web app: --color-date, which is hsl(0deg 0% 15% / 75%) I wonder if some confusion about contrastingColor (see a recent commit where we removed a variable with that name) might have led to the old color being used for stream headers, and perhaps the DM-header change was just done to align with stream headers. Anyway, might as well use the web app's color, which we were doing before e97e6fd. We can get a dark-theme variant from the web app too, which is nice. The Figma doesn't yet offer a dark-theme variant.
1 parent 4c88148 commit 9275940

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/widgets/message_list.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -519,7 +519,7 @@ class DateSeparator extends StatelessWidget {
519519

520520
final Message message;
521521

522-
// This color matches recipient headers. TODO(design) is that what we want?
522+
// TODO(design) do we want this color to match recipient headers?
523523
static final _textColor = Colors.black.withOpacity(0.4);
524524

525525
@override
@@ -801,7 +801,7 @@ class RecipientHeaderDate extends StatelessWidget {
801801
return Padding(
802802
padding: const EdgeInsets.fromLTRB(10, 0, 16, 0),
803803
child: DateText(
804-
color: Colors.black.withOpacity(0.4),
804+
color: const HSLColor.fromAHSL(0.75, 0, 0, 0.15).toColor(),
805805
fontSize: 16,
806806
// In Figma this has a line-height of 19, but using 18
807807
// here to match the stream/topic text widgets helps

0 commit comments

Comments
 (0)