Paths

Table of Contentst

-
web/
-
modals/threads/
-
threads/
-
new-thread-modal.react.js
-
style.css

Differential D4618

[web] Remove old `NewThreadModal` component
ClosedPublic
Actions

Authored by • jacek on Jul 25 2022, 7:08 AM.

Tags

None

Referenced Files

	F6165465: D4618.id14944.diff
	Tue, Apr 22, 10:58 PM

	F6165464: D4618.id14858.diff
	Tue, Apr 22, 10:58 PM

	Unknown Object (File)
	Mon, Apr 21, 1:14 PM

	Unknown Object (File)
	Sun, Apr 20, 1:38 PM

	Unknown Object (File)
	Sun, Apr 20, 11:37 AM

	Unknown Object (File)
	Sat, Apr 19, 7:59 PM

	Unknown Object (File)
	Fri, Apr 18, 7:51 PM

	Unknown Object (File)
	Thu, Apr 17, 8:22 PM

Subscribers

Details

Reviewers

tomek
atul

Commits

rCOMM633414d38308: [web] Remove old `NewThreadModal` component

Summary

The diff removes old NewThreadModal component that was not used anywhere. Also styles for this modal from style.css has been removed.

Test Plan

Web app build and runs as before

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• jacek created this revision.Jul 25 2022, 7:08 AM

Herald added subscribers: • abosh, • adrian, ashoat. · View Herald TranscriptJul 25 2022, 7:08 AM

• jacek added a parent revision: D4469: [web] Introduce "Create new chat" button.Jul 25 2022, 7:14 AM

Harbormaster completed remote builds in B10805: Diff 14858.Jul 25 2022, 7:22 AM

• jacek requested review of this revision.Jul 25 2022, 7:22 AM

Nice

This revision is now accepted and ready to land.Jul 25 2022, 7:23 AM

atul added inline comments.Jul 25 2022, 7:25 AM

web/style.css
167 ↗	(On Diff #14858)	Confirmed this wasn't used in any other component.
192 ↗	(On Diff #14858)	Confirmed that this wasn't used in any other component.
203 ↗	(On Diff #14858)	Confirmed that this wasn't used in any other component.
207 ↗	(On Diff #14858)	Confirmed that this wasn't used in any other component.

Closed by commit rCOMM633414d38308: [web] Remove old `NewThreadModal` component (authored by • jacek). · Explain WhyJul 26 2022, 3:06 AM

This revision was automatically updated to reflect the committed changes.

• jacek added a commit: rCOMM633414d38308: [web] Remove old `NewThreadModal` component.

Revision Contents
Changeset List

Path

Size

web/

modals/

threads/

new-thread-modal.react.js

38 lines

Diff 14944

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

Loading...

web/style.css

Loading...