Page MenuHomePhabricator

[web] [fix] [ENG-731] update thread selected state text
ClosedPublic

Authored by benschac on Apr 11 2022, 11:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 21, 9:57 PM
Unknown Object (File)
Fri, Jun 21, 9:57 PM
Unknown Object (File)
Fri, Jun 21, 9:56 PM
Unknown Object (File)
Fri, Jun 21, 9:56 PM
Unknown Object (File)
Fri, Jun 21, 9:52 PM
Unknown Object (File)
Tue, Jun 18, 4:39 PM
Unknown Object (File)
Mon, Jun 17, 12:11 AM
Unknown Object (File)
Thu, Jun 13, 1:45 PM

Details

Summary
Test Plan

select a chat thread list item on web

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek requested changes to this revision.Apr 12 2022, 9:49 AM

change text color per issue and conversation with ashoat.

It's hard to verify the diff when we don't know what was agreed. It is probably described on linear - the comment says that:

Talked IRL with @ashoat, settled on var(--shades-of-white-60); for the selected text color of the chat thread list item.

But it looks like on current master the value of --thread-color-read is --thread-color-read: var(--shades-black-60). Does the value of --thread-color-read changes in one of your diffs? This diff isn't a part of a stack so it shouldn't depend on anything else.

This revision now requires changes to proceed.Apr 12 2022, 9:49 AM

ty for catching this! you're 100% right:

Image 2022-04-18 at 9.25.19 AM.jpg (386×746 px, 33 KB)

updated the selected text color to var(shades-of-white-60)

This revision is now accepted and ready to land.Apr 19 2022, 2:36 AM