- Notifications
You must be signed in to change notification settings - Fork1k
feat: add tooltip field to workspace app that renders as markdown#19651
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Changes from1 commit
f24a1f4
14d3699
52ddebf
2255f73
4afab8d
551a0ec
ae1dee5
191fa45
c2d6191
e3d79f1
bacf93b
921b155
4c14ebd
4755bb3
061c7ba
0e08cc6
cae0ec7
760b65c
c1926bd
adbd52d
9b34b75
File filter
Filter by extension
Conversations
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
- Loading branch information
Uh oh!
There was an error while loading.Please reload this page.
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -135,9 +135,7 @@ export const AppLink: FC<AppLinkProps> = ({ | ||
</AgentButton> | ||
); | ||
if (primaryTooltip || app.tooltip) { | ||
return ( | ||
<TooltipProvider> | ||
<Tooltip> | ||
@@ -146,7 +144,7 @@ export const AppLink: FC<AppLinkProps> = ({ | ||
{primaryTooltip ? ( | ||
primaryTooltip | ||
) : app.tooltip ? ( | ||
<Markdown className="text-content-secondary prose-sm font-medium wrap-anywhere"> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Oh, neat. Didn't realize that we had Tailwind's typography plugin installed. Being able to use | ||
{app.tooltip} | ||
</Markdown> | ||
) : null} | ||
Uh oh!
There was an error while loading.Please reload this page.