Page MenuHomePhabricator
Differential D3928

[web] Push `<form>` tag down from `ThreadSettingsModal` to child "Tab" components
ClosedPublic
Actions

Authored by atul on May 5 2022, 6:39 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 10, 3:44 AM
Unknown Object (File)
Thu, Apr 10, 3:44 AM
Unknown Object (File)
Tue, Apr 8, 6:33 PM
Unknown Object (File)
Tue, Apr 8, 6:17 PM
Unknown Object (File)
Tue, Apr 8, 4:22 PM
Unknown Object (File)
Fri, Apr 4, 5:26 AM
Unknown Object (File)
Tue, Apr 1, 8:59 PM
Unknown Object (File)
Mon, Mar 31, 3:14 PM
View All Files
Subscribers
• abosh
• adrian
• karol
tomek

Details

Reviewers
ashoat
tomek
Commits
rCOMMcb2b044b0ed6: [web] Push `<form>` tag down from `ThreadSettingsModal` to child "Tab"…
Summary

Addresses feedback from @palys-swm in a previous stack. Makes ThreadSettingsModal cleaner.

Test Plan

Things continue to look/work as expected:

6869.png (1×736 px, 77 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

Diff 12294

View Options

web/modals/threads/thread-settings-delete-tab.react.js

Show First 20 Lines • Show All 275 Lines • ▼ Show 20 Lines:root {
/* Link */ /* Link */
--link-background-primary-default: var(--violet-dark-100); --link-background-primary-default: var(--violet-dark-100);
/* Separator */ /* Separator */
--separator-background-primary-default: var(--shades-black-75); --separator-background-primary-default: var(--shades-black-75);
/* Modal */ /* Modal */
--modal-background-primary-default: var(--shades-black-85); --modal-background-primary-default: var(--shades-black-85);
--modal-background-secondary-default: var(--shades-black-90);
--modal-header-primary-default: var(--shades-white-100); --modal-header-primary-default: var(--shades-white-100);
--modal-close-primary-default: var(--shades-black-50); --modal-close-primary-default: var(--shades-black-50);
--modal-close-primary-hover: var(--shades-white-100); --modal-close-primary-hover: var(--shades-white-100);
--modal-listItem-primary-hover: var(--shades-black-90); --modal-listItem-primary-hover: var(--shades-black-90);
/* Input Field */ /* Input Field */
--inputField-background-primary-default: var(--shades-black-85); --inputField-background-primary-default: var(--shades-black-85);
--inputField-background-secondary-default: var(--shades-black-75); --inputField-background-secondary-default: var(--shades-black-75);
Show All 32 Lines[data-theme='light'] {
/* Link */ /* Link */
--link-background-primary-default: var(--violet-dark-100); --link-background-primary-default: var(--violet-dark-100);
/* Separator */ /* Separator */
--separator-background-primary-default: var(--shades-white-60); --separator-background-primary-default: var(--shades-white-60);
/* Modal */ /* Modal */
--modal-background-primary-default: var(--shades-white-100); --modal-background-primary-default: var(--shades-white-100);
--modal-background-secondary-default: var(--shades-white-80);
--modal-header-primary-default: var(--shades-black-95); --modal-header-primary-default: var(--shades-black-95);
--modal-close-primary-default: var(--shades-black-50); --modal-close-primary-default: var(--shades-black-50);
--modal-close-primary-hover: var(--shades-black-95); --modal-close-primary-hover: var(--shades-black-95);
--modal-listItem-primary-hover: var(--shades-white-70); --modal-listItem-primary-hover: var(--shades-white-70);
/* Input Field */ /* Input Field */
--inputField-background-primary-default: var(--shades-white-100); --inputField-background-primary-default: var(--shades-white-100);
--inputField-background-secondary-default: var(--shades-white-60); --inputField-background-secondary-default: var(--shades-white-60);
Show All 12 Lines
View Options

web/modals/threads/thread-settings-general-tab.react.js

Show First 20 Lines • Show All 275 Lines • ▼ Show 20 Lines:root {
/* Link */ /* Link */
--link-background-primary-default: var(--violet-dark-100); --link-background-primary-default: var(--violet-dark-100);
/* Separator */ /* Separator */
--separator-background-primary-default: var(--shades-black-75); --separator-background-primary-default: var(--shades-black-75);
/* Modal */ /* Modal */
--modal-background-primary-default: var(--shades-black-85); --modal-background-primary-default: var(--shades-black-85);
--modal-background-secondary-default: var(--shades-black-90);
--modal-header-primary-default: var(--shades-white-100); --modal-header-primary-default: var(--shades-white-100);
--modal-close-primary-default: var(--shades-black-50); --modal-close-primary-default: var(--shades-black-50);
--modal-close-primary-hover: var(--shades-white-100); --modal-close-primary-hover: var(--shades-white-100);
--modal-listItem-primary-hover: var(--shades-black-90); --modal-listItem-primary-hover: var(--shades-black-90);
/* Input Field */ /* Input Field */
--inputField-background-primary-default: var(--shades-black-85); --inputField-background-primary-default: var(--shades-black-85);
--inputField-background-secondary-default: var(--shades-black-75); --inputField-background-secondary-default: var(--shades-black-75);
Show All 32 Lines[data-theme='light'] {
/* Link */ /* Link */
--link-background-primary-default: var(--violet-dark-100); --link-background-primary-default: var(--violet-dark-100);
/* Separator */ /* Separator */
--separator-background-primary-default: var(--shades-white-60); --separator-background-primary-default: var(--shades-white-60);
/* Modal */ /* Modal */
--modal-background-primary-default: var(--shades-white-100); --modal-background-primary-default: var(--shades-white-100);
--modal-background-secondary-default: var(--shades-white-80);
--modal-header-primary-default: var(--shades-black-95); --modal-header-primary-default: var(--shades-black-95);
--modal-close-primary-default: var(--shades-black-50); --modal-close-primary-default: var(--shades-black-50);
--modal-close-primary-hover: var(--shades-black-95); --modal-close-primary-hover: var(--shades-black-95);
--modal-listItem-primary-hover: var(--shades-white-70); --modal-listItem-primary-hover: var(--shades-white-70);
/* Input Field */ /* Input Field */
--inputField-background-primary-default: var(--shades-white-100); --inputField-background-primary-default: var(--shades-white-100);
--inputField-background-secondary-default: var(--shades-white-60); --inputField-background-secondary-default: var(--shades-white-60);
Show All 12 Lines
View Options

web/modals/threads/thread-settings-modal.react.js

Show First 20 Lines • Show All 275 Lines • ▼ Show 20 Lines:root {
/* Link */ /* Link */
--link-background-primary-default: var(--violet-dark-100); --link-background-primary-default: var(--violet-dark-100);
/* Separator */ /* Separator */
--separator-background-primary-default: var(--shades-black-75); --separator-background-primary-default: var(--shades-black-75);
/* Modal */ /* Modal */
--modal-background-primary-default: var(--shades-black-85); --modal-background-primary-default: var(--shades-black-85);
--modal-background-secondary-default: var(--shades-black-90);
--modal-header-primary-default: var(--shades-white-100); --modal-header-primary-default: var(--shades-white-100);
--modal-close-primary-default: var(--shades-black-50); --modal-close-primary-default: var(--shades-black-50);
--modal-close-primary-hover: var(--shades-white-100); --modal-close-primary-hover: var(--shades-white-100);
--modal-listItem-primary-hover: var(--shades-black-90); --modal-listItem-primary-hover: var(--shades-black-90);
/* Input Field */ /* Input Field */
--inputField-background-primary-default: var(--shades-black-85); --inputField-background-primary-default: var(--shades-black-85);
--inputField-background-secondary-default: var(--shades-black-75); --inputField-background-secondary-default: var(--shades-black-75);
Show All 32 Lines[data-theme='light'] {
/* Link */ /* Link */
--link-background-primary-default: var(--violet-dark-100); --link-background-primary-default: var(--violet-dark-100);
/* Separator */ /* Separator */
--separator-background-primary-default: var(--shades-white-60); --separator-background-primary-default: var(--shades-white-60);
/* Modal */ /* Modal */
--modal-background-primary-default: var(--shades-white-100); --modal-background-primary-default: var(--shades-white-100);
--modal-background-secondary-default: var(--shades-white-80);
--modal-header-primary-default: var(--shades-black-95); --modal-header-primary-default: var(--shades-black-95);
--modal-close-primary-default: var(--shades-black-50); --modal-close-primary-default: var(--shades-black-50);
--modal-close-primary-hover: var(--shades-black-95); --modal-close-primary-hover: var(--shades-black-95);
--modal-listItem-primary-hover: var(--shades-white-70); --modal-listItem-primary-hover: var(--shades-white-70);
/* Input Field */ /* Input Field */
--inputField-background-primary-default: var(--shades-white-100); --inputField-background-primary-default: var(--shades-white-100);
--inputField-background-secondary-default: var(--shades-white-60); --inputField-background-secondary-default: var(--shades-white-60);
Show All 12 Lines
View Options

web/modals/threads/thread-settings-privacy-tab.react.js

Show First 20 Lines • Show All 275 Lines • ▼ Show 20 Lines:root {
/* Link */ /* Link */
--link-background-primary-default: var(--violet-dark-100); --link-background-primary-default: var(--violet-dark-100);
/* Separator */ /* Separator */
--separator-background-primary-default: var(--shades-black-75); --separator-background-primary-default: var(--shades-black-75);
/* Modal */ /* Modal */
--modal-background-primary-default: var(--shades-black-85); --modal-background-primary-default: var(--shades-black-85);
--modal-background-secondary-default: var(--shades-black-90);
--modal-header-primary-default: var(--shades-white-100); --modal-header-primary-default: var(--shades-white-100);
--modal-close-primary-default: var(--shades-black-50); --modal-close-primary-default: var(--shades-black-50);
--modal-close-primary-hover: var(--shades-white-100); --modal-close-primary-hover: var(--shades-white-100);
--modal-listItem-primary-hover: var(--shades-black-90); --modal-listItem-primary-hover: var(--shades-black-90);
/* Input Field */ /* Input Field */
--inputField-background-primary-default: var(--shades-black-85); --inputField-background-primary-default: var(--shades-black-85);
--inputField-background-secondary-default: var(--shades-black-75); --inputField-background-secondary-default: var(--shades-black-75);
Show All 32 Lines[data-theme='light'] {
/* Link */ /* Link */
--link-background-primary-default: var(--violet-dark-100); --link-background-primary-default: var(--violet-dark-100);
/* Separator */ /* Separator */
--separator-background-primary-default: var(--shades-white-60); --separator-background-primary-default: var(--shades-white-60);
/* Modal */ /* Modal */
--modal-background-primary-default: var(--shades-white-100); --modal-background-primary-default: var(--shades-white-100);
--modal-background-secondary-default: var(--shades-white-80);
--modal-header-primary-default: var(--shades-black-95); --modal-header-primary-default: var(--shades-black-95);
--modal-close-primary-default: var(--shades-black-50); --modal-close-primary-default: var(--shades-black-50);
--modal-close-primary-hover: var(--shades-black-95); --modal-close-primary-hover: var(--shades-black-95);
--modal-listItem-primary-hover: var(--shades-white-70); --modal-listItem-primary-hover: var(--shades-white-70);
/* Input Field */ /* Input Field */
--inputField-background-primary-default: var(--shades-white-100); --inputField-background-primary-default: var(--shades-white-100);
--inputField-background-secondary-default: var(--shades-white-60); --inputField-background-secondary-default: var(--shades-white-60);
Show All 12 Lines