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
F3178345: D3928.id12275.diff
Fri, Nov 8, 1:34 AM
F3177493: D3928.diff
Thu, Nov 7, 11:32 PM
Unknown Object (File)
Thu, Oct 10, 2:09 AM
Unknown Object (File)
Sep 24 2024, 1:42 AM
Unknown Object (File)
Sep 24 2024, 1:41 AM
Unknown Object (File)
Sep 24 2024, 1:41 AM
Unknown Object (File)
Sep 23 2024, 9:28 PM
Unknown Object (File)
Sep 23 2024, 12:40 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

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

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

Loading...