This is not only a refactor, but it also changes the logic. In the previous version, when item.endsCluster then marginBottom: 5 + clusterEndHeight, but now marginBottom: clusterEndHeight. Is that intentional?

This revision now requires changes to proceed.May 11 2022, 5:46 AM

fix logic

In D3997#112145, @palys-swm wrote:

This is not only a refactor, but it also changes the logic. In the previous version, when item.endsCluster then marginBottom: 5 + clusterEndHeight, but now marginBottom: clusterEndHeight. Is that intentional?

You're right, thanks for catching this.

Harbormaster failed remote builds in B8997: Diff 12535!May 11 2022, 6:06 AM

tomek accepted this revision.May 11 2022, 6:08 AM

This revision is now accepted and ready to land.May 11 2022, 6:08 AM

Harbormaster completed remote builds in B8997: Diff 12535.May 11 2022, 9:25 AM

Closed by commit rCOMM206d81a9a618: [native] [refactor] [ENG-530] refactor container margin bottom if statement (authored by • benschac). · Explain WhyMay 11 2022, 11:42 AM

This revision was automatically updated to reflect the committed changes.

• benschac added a commit: rCOMM206d81a9a618: [native] [refactor] [ENG-530] refactor container margin bottom if statement.

Revision Contents
Changeset List

Path

Size

native/

chat/

composed-message.react.js

8 lines

Diff 12535

View Options

native/chat/composed-message.react.js

	Show First 20 Lines • Show All 130 Lines • ▼ Show 20 Lines
	const styles = StyleSheet.create({			const styles = StyleSheet.create({
	dummyMessage: {			dummyMessage: {
	paddingHorizontal: 12,			paddingHorizontal: 12,
	paddingVertical: 6,			paddingVertical: 6,
	},			},
	message: {			message: {
	overflow: 'hidden',			overflow: 'hidden',
	paddingHorizontal: 12,			paddingHorizontal: 12,
	paddingVertical: 6,			paddingVertical: 8,
	},			},
	text: {			text: {
	fontFamily: 'Arial',			fontFamily: 'Arial',
	fontSize: 18,			fontSize: 14,
				benschacAuthorUnsubmitted Done Inline Actions I actually think this is a bit too small. But that's what's in the design. https://www.figma.com/file/L675ETKDnGaSwlpZAw4MIC/Mobile-App?node-id=351%3A455 @ashoat - I think 16px looks better. benschac: I actually think this is a bit too small. But that's what's in the design. https://www.figma.
				lineHeight: 22,
	},			},
	});			});

	export { InnerTextMessage, dummyNodeForTextMessageHeightMeasurement };			export { InnerTextMessage, dummyNodeForTextMessageHeightMeasurement };

[native] [refactor] [ENG-530] refactor container margin bottom if statementClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 12535

native/chat/composed-message.react.js

[native] [refactor] [ENG-530] refactor container margin bottom if statement
ClosedPublic
Actions

Revision Contents
Changeset List