Paths

Table of Contentst

Differential D8844

[native] Refactor markdown-link component
ClosedPublic
Actions

Authored by • patryk on Aug 17 2023, 1:55 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Oct 27, 2:33 PM

	Unknown Object (File)
	Oct 2 2024, 8:38 AM

	Unknown Object (File)
	Sep 27 2024, 8:53 PM

	Unknown Object (File)
	Sep 27 2024, 8:53 PM

	Unknown Object (File)
	Sep 27 2024, 8:53 PM

	Unknown Object (File)
	Sep 27 2024, 8:53 PM

	Unknown Object (File)
	Sep 27 2024, 8:53 PM

	Unknown Object (File)
	Sep 27 2024, 8:53 PM

Subscribers

Details

Reviewers

tomek
inka
rohan

Commits

rCOMM1430543adb94: [native] Refactor markdown-link component

Summary

This diff extracts markdown-link.react.js code to markdown-utils.js. This code is going to be reused in markdown-chat-mention component.

Depends on D8834.

Test Plan

Tested later in the stack.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• patryk created this revision.Aug 17 2023, 1:55 AM

• patryk held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 17 2023, 1:55 AM

Harbormaster completed remote builds in B21827: Diff 29999.Aug 17 2023, 2:12 AM

Change shouldBePressable logic

Harbormaster completed remote builds in B21836: Diff 30008.Aug 17 2023, 7:14 AM

• patryk edited the summary of this revision. (Show Details)Aug 18 2023, 6:08 AM

• patryk edited the test plan for this revision. (Show Details)

• patryk added reviewers: tomek, inka, rohan.

• patryk edited the summary of this revision. (Show Details)Aug 19 2023, 9:52 AM

• patryk added a child revision: D8845: [native] Introduce markdown component for chat mentions.

• patryk added a child revision: D8871: [web][native] Refactor getDefaultTextMessageRules.Aug 19 2023, 3:22 PM

• patryk removed a child revision: D8871: [web][native] Refactor getDefaultTextMessageRules.Aug 19 2023, 3:25 PM

• patryk edited the summary of this revision. (Show Details)Aug 20 2023, 6:28 AM

• patryk added a parent revision: D8834: [lib] Introduce chat mention match and parse utilities.

• patryk published this revision for review.Aug 20 2023, 11:11 PM

We have an agreement in our team that if we are moving some code, we should first move it and change it in next diff, becuse moving and changing in one diff makes it hard to review. You can see in this diff that it looks on phabricator as if you removed useHandleLinkClick function, and you didn't do that

native/markdown/markdown-utils.js
20 ↗	(On Diff #30008)	I'm assuming this argument will be used somewhere
60 ↗	(On Diff #30008)	I think this should be `() => void`

This revision is now accepted and ready to land.Aug 22 2023, 7:08 AM

Change mixed to void

Harbormaster completed remote builds in B22010: Diff 30221.Aug 22 2023, 7:28 AM

Remove blockOnPress variable from useMarkdownOnPressUtils

Rename shouldBePressable to isRevealed

Harbormaster completed remote builds in B22011: Diff 30222.Aug 22 2023, 7:53 AM

Harbormaster completed remote builds in B22012: Diff 30223.Aug 22 2023, 7:57 AM

Rebase

Harbormaster completed remote builds in B22706: Diff 31242.Sep 19 2023, 1:47 AM

Closed by commit rCOMM1430543adb94: [native] Refactor markdown-link component (authored by • patryk). · Explain WhySep 28 2023, 1:16 AM

This revision was automatically updated to reflect the committed changes.

• patryk added a commit: rCOMM1430543adb94: [native] Refactor markdown-link component.

Revision Contents
Changeset List

Path

Size

native/

markdown/

markdown-link.react.js

97 lines

	markdown-utils.js
	markdown-link.react.js

80 lines

Diff 31486

native/markdown/markdown-link.react.js

Loading...

native/markdown/markdown-utils.js

Loading...