Skip to content

Commit 07fd507

Browse files
committed
msglist: Make date-separator text color match recipient-header dates
I removed the TODO(design), because now this element's color matches the corresponding element in the web app, and that seems like a fine way to settle the question. I didn't find a date-separator example in the Figma (and Greg didn't either, when implementing them in zulip#469.)
1 parent bf872e7 commit 07fd507

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

lib/widgets/message_list.dart

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -519,9 +519,6 @@ class DateSeparator extends StatelessWidget {
519519

520520
final Message message;
521521

522-
// TODO(design) do we want this color to match recipient headers?
523-
static final _textColor = Colors.black.withOpacity(0.4);
524-
525522
@override
526523
Widget build(BuildContext context) {
527524
// This makes the small-caps text vertically centered,
@@ -542,7 +539,7 @@ class DateSeparator extends StatelessWidget {
542539
color: Colors.black)))))),
543540
Padding(padding: const EdgeInsets.fromLTRB(2, 0, 2, textBottomPadding),
544541
child: DateText(
545-
color: _textColor,
542+
color: const HSLColor.fromAHSL(0.75, 0, 0, 0.15).toColor(),
546543
fontSize: 16,
547544
height: (16 / 16),
548545
timestamp: message.timestamp)),

0 commit comments

Comments
 (0)