[web] [refactor] remove unneeded div in chat-thread-list component
ClosedPublic
Actions

Authored by • benschac on Mar 10 2022, 1:55 PM.

Details

Reviewers

atul
• jacek
tomek

Commits

rCOMM8de95514f0a0: [web] [refactor] remove unneeded div in chat-thread-list component

Summary

remove an extra div we don't need. I hard-coded a color but will remove it in future diffs, along with other unneeded styles. I just didn't want to break existing functionality.

before:

Image 2022-03-10 at 4.45.37 PM.jpg (1×1 px, 120 KB)

after:

Image 2022-03-10 at 4.45.12 PM.jpg (1×1 px, 225 KB)

Test Plan

do a side-by-side with production. Make sure the unread pop up doesn't break.

Diff Detail

Repository

rCOMM Comm

Branch

mark-as-unread

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• benschac created this revision.Mar 10 2022, 1:55 PM

Herald added subscribers: • karol, • adrian, tomek, ashoat. · View Herald TranscriptMar 10 2022, 1:55 PM

• benschac edited the summary of this revision. (Show Details)Mar 10 2022, 1:56 PM

• benschac added reviewers: • jacek, tomek.

• benschac edited the summary of this revision. (Show Details)

Harbormaster completed remote builds in B7292: Diff 10282.Mar 10 2022, 1:59 PM

• benschac requested review of this revision.Mar 10 2022, 1:59 PM

I hard-coded a color but will remove it in future diffs, along with other unneeded styles. I just didn't want to break existing functionality.

If removing the div requires adding these colors, then adding them was ok, but I think we should still use variables instead of hardcoded values.

This revision is now accepted and ready to land.Mar 11 2022, 5:44 AM

In D3405#92013, @palys-swm wrote:

I hard-coded a color but will remove it in future diffs, along with other unneeded styles. I just didn't want to break existing functionality.

If removing the div requires adding these colors, then adding them was ok, but I think we should still use variables instead of hardcoded values.

Agreed. I'm going to use variables in the future. Just wanted to get rid of this div.

Closed by commit rCOMM8de95514f0a0: [web] [refactor] remove unneeded div in chat-thread-list component (authored by • benschac). · Explain WhyMar 11 2022, 7:22 AM

This revision was automatically updated to reflect the committed changes.

• benschac added a commit: rCOMM8de95514f0a0: [web] [refactor] remove unneeded div in chat-thread-list component.

• benschac added a reverting change: D3407: Revert "[web] [refactor] remove unneeded div in chat-thread-list component".Mar 11 2022, 10:35 AM

• benschac added a reverting change: rCOMM7d8e81f8dddf: Revert "[web] [refactor] remove unneeded div in chat-thread-list component".Mar 11 2022, 12:48 PM