Skip to content

Standardize tooltip for chat section #5195

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Jazzcort
Copy link
Contributor

Description

Before this fix, tooltips were showing for some items and not for the others. The inconsistency fells like a bug. In addidtion, the format of the tooltips doesn't match the standard pattern in VS Code.

This fix provide tooltips for all the items in the Continue UI and formalize the text format of each tooltip.
#4819

Screenshots

Screen.Recording.2025-04-16.at.9.24.18.AM.mov

Before this fix, tooltips were showing for some items and not for the others.
The inconsistency fells like a bug. In addidtion, the format of the tooltips
doesn't match the standard pattern in VS Code.

This fix provide tooltips for all the items in the Continue UI and formalize
the text format of each tooltip.
continuedev#4819
@Jazzcort Jazzcort requested a review from a team as a code owner April 16, 2025 15:35
@Jazzcort Jazzcort requested review from RomneyDa and removed request for a team April 16, 2025 15:35
Copy link

netlify bot commented Apr 16, 2025

Deploy Preview for continuedev ready!

Name Link
🔨 Latest commit b212c8c
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67ffce441cec20000801cf85
😎 Deploy Preview https://deploy-preview-5195--continuedev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@RomneyDa
Copy link
Collaborator

RomneyDa commented Apr 25, 2025

This is duplicate of #4971, closing for now or let me know if I missed something!

At the time we decided to close that one but realized it would nice to have status information in the tooltips so I redid it here #4971 but same idea

EDIT: ignore this message it's wrong

@RomneyDa RomneyDa closed this Apr 25, 2025
@RomneyDa RomneyDa reopened this Apr 25, 2025
@RomneyDa
Copy link
Collaborator

Noticed there were additional tooltips here, taking a closer look

@RomneyDa
Copy link
Collaborator

@Jazzcort looks like I don't have permissions for this branch, if you can fix merge conflicts (basically take out the notch section tooltips I think) then this is good to go!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants