-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
welcome card: update style, copy, and behavior #3137
Conversation
ui/src/components/WelcomeCard.tsx
Outdated
</p> | ||
<p className="leading-5"> | ||
Visit the{' '} | ||
<span className="font-semibold dark:font-bold">Tlon Studio</span>{' '} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might make sense to have Tlon Studio
be a link?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alecananian during first time boots, this card will exist above the two pre-invited groups "Tlon Locals" and "Tlon Studio", so they'll technically be right where they need to be! That being said, I think it'd a good idea to duplicate the ability to jump to Tlon Studio in case curious people tap the bolded word.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linking to Tlon Studio for duplicity's sake is probably a good idea — after this it's g2g!
Added that affordance for any curious clickers :~) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 nice
This brings the welcome card up to date with the copy and styling in Figma. Adapts it to be shown on desktop views as well, positioned above the activity feed on the home screen.
Closed status is no longer stored locally on the frontend, instead its tracked in the settings agent. This means closing the card once will hide it on all clients.
Fixes LAND-1399