Keep Pin primary CTA (#7061)

This commit is contained in:
dan
2024-12-11 22:32:57 +00:00
committed by GitHub
parent 329b254de1
commit 3ee94f3620
@@ -250,6 +250,7 @@ export function ProfileFeedHeader({info}: {info: FeedSourceFeedInfo}) {
{hasSession && ( {hasSession && (
<Layout.Header.Slot> <Layout.Header.Slot>
{isPinned ? (
<Menu.Root> <Menu.Root>
<Menu.Trigger label={_(msg`Open feed options menu`)}> <Menu.Trigger label={_(msg`Open feed options menu`)}>
{({props}) => { {({props}) => {
@@ -274,14 +275,13 @@ export function ProfileFeedHeader({info}: {info: FeedSourceFeedInfo}) {
<Menu.Outer> <Menu.Outer>
<Menu.Item <Menu.Item
disabled={isFeedStateChangePending} disabled={isFeedStateChangePending}
label={ label={_(msg`Unpin from Home`)}
isPinned ? _(msg`Unpin from home`) : _(msg`Pin to home`)
}
onPress={onTogglePinned}> onPress={onTogglePinned}>
<Menu.ItemText> <Menu.ItemText>{_(msg`Unpin from home`)}</Menu.ItemText>
{isPinned ? _(msg`Unpin from home`) : _(msg`Pin to home`)} <Menu.ItemIcon
</Menu.ItemText> icon={isPinned ? X : Pin}
<Menu.ItemIcon icon={isPinned ? X : Pin} position="right" /> position="right"
/>
</Menu.Item> </Menu.Item>
<Menu.Item <Menu.Item
disabled={isFeedStateChangePending} disabled={isFeedStateChangePending}
@@ -303,6 +303,17 @@ export function ProfileFeedHeader({info}: {info: FeedSourceFeedInfo}) {
</Menu.Item> </Menu.Item>
</Menu.Outer> </Menu.Outer>
</Menu.Root> </Menu.Root>
) : (
<Button
label={_(msg`Pin to Home`)}
size="small"
variant="ghost"
shape="square"
color="secondary"
onPress={onTogglePinned}>
<ButtonIcon icon={Pin} size="lg" />
</Button>
)}
</Layout.Header.Slot> </Layout.Header.Slot>
)} )}
</Layout.Header.Outer> </Layout.Header.Outer>