Page MenuHomePhabricator
Differential D12979

[native] Move buttonStyle
ClosedPublic
Actions

Authored by inka on Aug 5 2024, 6:03 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 5, 8:35 AM
Unknown Object (File)
Feb 18 2025, 12:07 AM
Unknown Object (File)
Feb 18 2025, 12:07 AM
Unknown Object (File)
Feb 18 2025, 12:07 AM
Unknown Object (File)
Feb 18 2025, 12:07 AM
Unknown Object (File)
Feb 18 2025, 12:07 AM
Unknown Object (File)
Feb 10 2025, 12:34 PM
Unknown Object (File)
Feb 10 2025, 12:34 PM
View All 80 Files
Subscribers
ashoat

Details

Reviewers
tomek
will
Commits
rCOMMe8fce78e4e69: [native] Move buttonStyle
Summary

issue: ENG-8914
Refactoring from class to function

Test Plan

Tested by creating a nux tooltip and displaying it - still works

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka created this revision.Aug 5 2024, 6:03 AM
Herald added a subscriber: ashoat. · View Herald TranscriptAug 5 2024, 6:03 AM
inka added a parent revision: D12978: [native] Move opacityStyle.Aug 5 2024, 6:11 AM
inka added a child revision: D12980: [native] Move onTipContainerLayout.
Harbormaster completed remote builds in B30926: Diff 43114.Aug 5 2024, 8:14 AM
inka requested review of this revision.Aug 5 2024, 8:14 AM
tomek accepted this revision.Aug 6 2024, 2:20 AM
tomek added inline comments.
native/tooltip/nux-tips-overlay.react.js
309 ↗(On Diff #43114)

We can make this a little more efficient by using more specific dependencies (initialCoordinates and verticalBounds).

This revision is now accepted and ready to land.Aug 6 2024, 2:20 AM
inka updated this revision to Diff 43343.Aug 13 2024, 6:37 AM

Address review

Harbormaster completed remote builds in B31065: Diff 43343.Aug 13 2024, 6:54 AM
Closed by commit rCOMMe8fce78e4e69: [native] Move buttonStyle (authored by inka). · Explain WhyAug 13 2024, 7:46 AM
This revision was automatically updated to reflect the committed changes.
inka added a commit: rCOMMe8fce78e4e69: [native] Move buttonStyle.

Revision Contents
Changeset List

PathSize
native/
tooltip/
29 lines

Diff 43352

View Options

native/tooltip/nux-tips-overlay.react.js

Show First 20 Lines • Show All 48 Lines • ▼ Show 20 Lines
import { BottomSheetProvider } from './bottom-sheet/bottom-sheet-provider.react.js'; import { BottomSheetProvider } from './bottom-sheet/bottom-sheet-provider.react.js';
import ChatContextProvider from './chat/chat-context-provider.react.js'; import ChatContextProvider from './chat/chat-context-provider.react.js';
import MessageEditingContextProvider from './chat/message-editing-context-provider.react.js'; import MessageEditingContextProvider from './chat/message-editing-context-provider.react.js';
import AccessTokenHandler from './components/access-token-handler.react.js'; import AccessTokenHandler from './components/access-token-handler.react.js';
import { AutoJoinCommunityHandler } from './components/auto-join-community-handler.react.js'; import { AutoJoinCommunityHandler } from './components/auto-join-community-handler.react.js';
import BackgroundIdentityLoginHandler from './components/background-identity-login-handler.react.js'; import BackgroundIdentityLoginHandler from './components/background-identity-login-handler.react.js';
import ConnectFarcasterAlertHandler from './components/connect-farcaster-alert-handler.react.js'; import ConnectFarcasterAlertHandler from './components/connect-farcaster-alert-handler.react.js';
import { FeatureFlagsProvider } from './components/feature-flags-provider.react.js'; import { FeatureFlagsProvider } from './components/feature-flags-provider.react.js';
import { NUXTipsContextProvider } from './components/nux-tips-context.react.js';
import PersistedStateGate from './components/persisted-state-gate.js'; import PersistedStateGate from './components/persisted-state-gate.js';
import ReportHandler from './components/report-handler.react.js'; import ReportHandler from './components/report-handler.react.js';
import VersionSupportedChecker from './components/version-supported.react.js'; import VersionSupportedChecker from './components/version-supported.react.js';
import ConnectedStatusBar from './connected-status-bar.react.js'; import ConnectedStatusBar from './connected-status-bar.react.js';
import { SQLiteDataHandler } from './data/sqlite-data-handler.js'; import { SQLiteDataHandler } from './data/sqlite-data-handler.js';
import ErrorBoundary from './error-boundary.react.js'; import ErrorBoundary from './error-boundary.react.js';
import IdentityServiceContextProvider from './identity-service/identity-service-context-provider.react.js'; import IdentityServiceContextProvider from './identity-service/identity-service-context-provider.react.js';
import InputStateContainer from './input/input-state-container.react.js'; import InputStateContainer from './input/input-state-container.react.js';
▲ Show 20 Lines • Show All 233 Lines • ▼ Show 20 Lines navigation = (
onStateChange={onNavigationStateChange} onStateChange={onNavigationStateChange}
theme={theme} theme={theme}
ref={containerRef} ref={containerRef}
> >
<BottomSheetModalProvider> <BottomSheetModalProvider>
<ChatContextProvider> <ChatContextProvider>
<DeepLinksContextProvider> <DeepLinksContextProvider>
<ChatMentionContextProvider> <ChatMentionContextProvider>
<NUXTipsContextProvider>
<RootNavigator /> <RootNavigator />
</NUXTipsContextProvider>
</ChatMentionContextProvider> </ChatMentionContextProvider>
</DeepLinksContextProvider> </DeepLinksContextProvider>
</ChatContextProvider> </ChatContextProvider>
<NavigationHandler /> <NavigationHandler />
<PersistedStateGate> <PersistedStateGate>
<FarcasterDataHandler> <FarcasterDataHandler>
<ConnectFarcasterAlertHandler /> <ConnectFarcasterAlertHandler />
</FarcasterDataHandler> </FarcasterDataHandler>
▲ Show 20 Lines • Show All 106 Lines • Show Last 20 Lines