Page MenuHomePhabricator
Differential D3589

[web] Copy/paste `user-settings-modal.css` into `thread-settings-modal.css`
ClosedPublic
Actions

Authored by atul on Mar 31 2022, 11:19 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 31, 12:58 PM
Unknown Object (File)
Sat, Mar 29, 1:27 PM
Unknown Object (File)
Sun, Mar 16, 4:56 PM
Unknown Object (File)
Mon, Mar 10, 12:25 PM
Unknown Object (File)
Feb 28 2025, 8:40 AM
Unknown Object (File)
Feb 10 2025, 6:08 AM
Unknown Object (File)
Feb 4 2025, 6:28 AM
Unknown Object (File)
Jan 31 2025, 11:41 PM
View All Files
Subscribers
• abosh
• adrian
• karol

Details

Reviewers
• jacek
tomek
• benschac
ashoat
Commits
rCOMM65be6b773f0e: [web] Copy/paste `user-settings-modal.css` into `thread-settings-modal.css`
Summary

noop

Next diffs:

  1. Move the styles used in thread-settings-modal.react from style.css to thread-settings-modal.css.
  2. "Consume" thread-settings-modal.css from thread-settings-modal.react instead of style.css.
  3. Remove extraneous selectors from thread-settings-modal.css that aren't used in thread-settings-modal.react.
  4. Fill in styles that are "used" by thread-settings-modal.react that don't actually exist anywhere in the repo (eg current-tab). Probably by looking through the git history to see what current-tab used to be.
  5. Style the ThreadSettingsModal component so it looks decent.
Test Plan

NA, literally copy/paste and the pasted code isn't even used anywhere

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Mar 31 2022, 11:19 AM
Herald added subscribers: • abosh, • karol, • adrian. · View Herald TranscriptMar 31 2022, 11:19 AM
atul edited the summary of this revision. (Show Details)Mar 31 2022, 11:21 AM

(Added @ashoat as first-pass reviewer since this seems "rubberstamp-able.")

atul published this revision for review.Mar 31 2022, 11:21 AM
Harbormaster completed remote builds in B7779: Diff 10914.Mar 31 2022, 11:25 AM
atul added a child revision: D3591: [web] Move styles in `style.css` used by `ThreadSettingsModal` to `thread-settings-modal.css`.Mar 31 2022, 12:01 PM
ashoat accepted this revision.Mar 31 2022, 8:53 PM
This revision is now accepted and ready to land.Mar 31 2022, 8:53 PM
atul updated this revision to Diff 11014.Apr 4 2022, 12:39 PM

rebase before landing

Harbormaster completed remote builds in B7845: Diff 11014.Apr 4 2022, 12:46 PM
Closed by commit rCOMM65be6b773f0e: [web] Copy/paste `user-settings-modal.css` into `thread-settings-modal.css` (authored by atul). · Explain WhyApr 4 2022, 12:52 PM
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM65be6b773f0e: [web] Copy/paste `user-settings-modal.css` into `thread-settings-modal.css`.

Revision Contents
Changeset List

PathSize
web/
modals/
threads/
143 lines

Diff 11015

View Options

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

Show All 38 Lines.menuContent {
background-color: #eeeeee; background-color: #eeeeee;
border-radius: 5px; border-radius: 5px;
box-shadow: 1px 1px 5px 2px #00000022; box-shadow: 1px 1px 5px 2px #00000022;
} }
.menuContentVisible { .menuContentVisible {
display: block; display: block;
} }
.menuSidebar {
padding-right: 2px;
}
button.menuContent { button.menuContent {
border: none; border: none;
cursor: pointer; cursor: pointer;
padding: 10px; padding: 10px;
font-size: 16px; font-size: 16px;
} }
button.menuContent:hover { button.menuContent:hover {
background-color: #dddddd; background-color: #dddddd;
} }