Manage links modal is complicated so it makes sense to split it into multiple components. Following diffs will introduce new components that are conditionally displayed.
Depends on D8413
Details
Details
- Reviewers
kamil inka bartek - Commits
- rCOMMb3529340de90: [web] Split modal into multiple components
This is only a refactoring. Tested if the modal still works.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| web/invite-links/manage/existing-link-content.react.js | ||
|---|---|---|
| 28 ↗ | (On Diff #28344) | I know this is a refactor, but it seems like we're using spaces for padding |
| web/invite-links/manage/existing-link-content.react.js | ||
|---|---|---|
| 28 ↗ | (On Diff #28344) | If we decide to update the approach, I'll update both diffs. Let's discuss it in https://phab.comm.dev/D8413#inline-54087 |