[web] Move declarations closer to where they're used in ComposedMessage
ClosedPublic
Actions

Authored by ashoat on Wed, Oct 9, 8:22 PM.

Tags

None

Referenced Files

	F3102349: D13674.diff
	Wed, Oct 30, 10:33 AM

	F3096325: D13674.id45114.diff
	Tue, Oct 29, 1:04 PM

	Unknown Object (File)
	Sun, Oct 20, 11:58 AM

	Unknown Object (File)
	Sat, Oct 19, 4:19 PM

	Unknown Object (File)
	Fri, Oct 18, 4:50 PM

	Unknown Object (File)
	Fri, Oct 18, 4:04 PM

	Unknown Object (File)
	Fri, Oct 18, 1:18 PM

	Unknown Object (File)
	Thu, Oct 17, 8:09 PM

View All 20 Files

Subscribers

None

Details

Reviewers

tomek

Commits

rCOMMf367427d3b5c: [web] Move declarations closer to where they're used in ComposedMessage

Summary

Now that we no longer have a separate function and class component, we can localize some of these declarations.

Depends on D13673

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Wed, Oct 9, 8:22 PM

ashoat added a child revision: D13675: [lib][web] Narrow ChatMessageInfoItem to ComposableChatMessageInfoItem where appropriate.

Harbormaster completed remote builds in B32158: Diff 45033.Wed, Oct 9, 8:53 PM

ashoat requested review of this revision.Wed, Oct 9, 8:53 PM

tomek accepted this revision.Thu, Oct 10, 8:25 AM

This revision is now accepted and ready to land.Thu, Oct 10, 8:25 AM

Closed by commit rCOMMf367427d3b5c: [web] Move declarations closer to where they're used in ComposedMessage (authored by ashoat). · Explain WhySun, Oct 13, 6:43 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMf367427d3b5c: [web] Move declarations closer to where they're used in ComposedMessage.

Revision Contents
Changeset List

Path

Size

web/

chat/

composed-message.react.js

116 lines

Diff 45114

View Options

web/chat/composed-message.react.js

[web] Move declarations closer to where they're used in ComposedMessageClosedPublicActions