[native] Remove progress prop from tooltip buttons components
ClosedPublic
Actions

Authored by angelika on Dec 11 2024, 6:50 PM.

Details

Reviewers

ashoat
tomek

Commits

rCOMMe47bfc45f374: [native] Remove progress prop from tooltip buttons components

Summary

Remove old unecessary progress value from tooltip buttons components. Separate diff because it changes multiple files at once.

Depends on D14126

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

angelika created this revision.Dec 11 2024, 6:50 PM

angelika held this revision as a draft.

angelika edited the summary of this revision. (Show Details)Dec 11 2024, 6:51 PM

angelika added a parent revision: D14126: [native] Migrate RobotextMessageTooltipButton component to Reanimated V2 API.

angelika added a child revision: D14128: [native] Migrate useOverlayPosition, useContentAndHeaderOpacity, useDeliveryIconOpacity hooks to Reanimated V2 API.Dec 11 2024, 7:00 PM

Harbormaster completed remote builds in B33028: Diff 46346.Dec 11 2024, 8:12 PM

angelika published this revision for review.Dec 12 2024, 1:28 AM

ashoat accepted this revision.Dec 12 2024, 3:56 AM

This revision is now accepted and ready to land.Dec 12 2024, 3:56 AM

Rebase and address feedback

Harbormaster completed remote builds in B33099: Diff 46421.Dec 13 2024, 4:47 PM

Rebase

Harbormaster completed remote builds in B33143: Diff 46484.Dec 16 2024, 5:45 PM

Closed by commit rCOMMe47bfc45f374: [native] Remove progress prop from tooltip buttons components. · Explain WhyDec 16 2024, 9:34 PM

This revision was automatically updated to reflect the committed changes.

angelika added a commit: rCOMMe47bfc45f374: [native] Remove progress prop from tooltip buttons components.

Revision Contents
Changeset List

Path

Size

native/

chat/

multimedia-message-tooltip-button.react.js

4 lines

robotext-message-tooltip-button.react.js

4 lines

text-message-tooltip-button.react.js

4 lines

tooltip/

tooltip.react.js

10 lines

Diff 46520

View Options

native/chat/multimedia-message-tooltip-button.react.js

View Options

native/chat/robotext-message-tooltip-button.react.js

View Options

native/chat/text-message-tooltip-button.react.js

View Options

native/tooltip/tooltip.react.js

[native] Remove progress prop from tooltip buttons componentsClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 46520

native/chat/multimedia-message-tooltip-button.react.js

native/chat/robotext-message-tooltip-button.react.js

native/chat/text-message-tooltip-button.react.js

native/tooltip/tooltip.react.js

[native] Remove progress prop from tooltip buttons components
ClosedPublic
Actions

Revision Contents
Changeset List