Skip to content

Commit 0f56763

Browse files
refactor(notifications): use Lumen UI in the prompt drawer view
Address review feedback from tonykhaov: swap the native-ui Flex/Button/ Link used in NotificationsPromptDrawerView for their Lumen equivalents (Box/Button/Link), matching the pattern already used in AnalyticsConsentScreen and ConfirmationErrorView. Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 44a4ec4 commit 0f56763

1 file changed

Lines changed: 20 additions & 8 deletions

File tree

apps/ledger-live-mobile/src/mvvm/features/NotificationsPrompt/screens/NotificationsPromptDrawerView.tsx

Lines changed: 20 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import React from "react";
2-
import { Flex, Link as TextLink, Button } from "@ledgerhq/native-ui";
2+
import { Box, Button, Link } from "@ledgerhq/lumen-ui-rnative";
33
import { useTranslation } from "~/context/Locale";
44
import { NotificationsDrawerIllustration } from "LLM/features/NotificationsPrompt/components/NotificationsDrawerIllustration";
55
import { NotificationsPromptContent } from "LLM/features/NotificationsPrompt/components/NotificationsPromptContent";
@@ -17,17 +17,29 @@ export function NotificationsPromptDrawerView({ promptTarget, onAllow, onLater }
1717
const { allowKey, laterKey } = getNotificationsPromptCopy(promptTarget);
1818

1919
return (
20-
<Flex mb={4}>
21-
<Flex alignItems="center">
20+
<Box lx={{ marginBottom: "s12" }}>
21+
<Box lx={{ alignItems: "center" }}>
2222
<NotificationsDrawerIllustration promptTarget={promptTarget} />
2323
<NotificationsPromptContent promptTarget={promptTarget} />
24-
</Flex>
25-
<Button type="main" mt={8} mb={7} onPress={onAllow} testID="notifications-prompt-allow">
24+
</Box>
25+
<Button
26+
appearance="base"
27+
size="lg"
28+
isFull
29+
lx={{ marginTop: "s32", marginBottom: "s24" }}
30+
onPress={onAllow}
31+
testID="notifications-prompt-allow"
32+
>
2633
{t(allowKey)}
2734
</Button>
28-
<TextLink type="shade" onPress={onLater} testID="notifications-prompt-later">
35+
<Link
36+
appearance="base"
37+
underline={false}
38+
onPress={onLater}
39+
testID="notifications-prompt-later"
40+
>
2941
{t(laterKey)}
30-
</TextLink>
31-
</Flex>
42+
</Link>
43+
</Box>
3244
);
3345
}

0 commit comments

Comments
 (0)