[web] Remove unused styles from `thread-settings-modal.css`
ClosedPublic
Actions

Authored by atul on Apr 15 2022, 1:06 PM.

Details

Reviewers

tomek
• jacek

Commits

rCOMM27c8ac11de98: [web] Remove unused styles from `thread-settings-modal.css`

Summary

These styles were either moved to relevant ThreadSettings[General/Privacy/Delete]Tab component or weren't used to begin with.

Test Plan

Did a quick scan and things look as expected.

Searched for selector in ThreadSettingsModal before deleting from thread-settings-modal.css

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Apr 15 2022, 1:06 PM

Herald added subscribers: • abosh, • benschac, • karol and 2 others. · View Herald TranscriptApr 15 2022, 1:06 PM

atul mentioned this in D3717: [web] Introduce minimally viable `ThreadSettingsPrivacyTab` component.Apr 15 2022, 1:07 PM

Harbormaster completed remote builds in B8239: Diff 11524.Apr 15 2022, 1:11 PM

atul requested review of this revision.Apr 15 2022, 1:11 PM

tomek accepted this revision.Apr 19 2022, 3:06 AM

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

rebase before landing

This revision was landed with ongoing or failed builds.Apr 19 2022, 10:59 AM

Closed by commit rCOMM27c8ac11de98: [web] Remove unused styles from `thread-settings-modal.css` (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM27c8ac11de98: [web] Remove unused styles from `thread-settings-modal.css`.

Harbormaster completed remote builds in B8324: Diff 11629.Apr 19 2022, 11:32 AM

Revision Contents
Changeset List

Path

Size

web/

modals/

threads/

thread-settings-modal.css

95 lines

Diff 11524

View Options

web/modals/threads/thread-settings-modal.css

[web] Remove unused styles from `thread-settings-modal.css`ClosedPublicActions