-
Notifications
You must be signed in to change notification settings - Fork 617
Add delete control for custom Ask AI questions #11053
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
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: jwunderl <[email protected]>
Co-authored-by: jwunderl <[email protected]>
|
@copilot icon looks bad, it's got no vertical padding / too much horizontal padding so it looks like a pill. give it no border, and actually just remove the button when not available, it looks bad grayed out as is.
|
Co-authored-by: jwunderl <[email protected]>
Updated the Ask AI custom question delete control to use a text-only icon style with tighter padding, and the button is now omitted entirely when only one custom question exists. Commit: f23d006. Screenshot: unable to capture locally because the teachertool dev server could not start (react-app-rewired not available in this environment). |
| }, | ||
| "../node_modules/@types/react": { | ||
| "version": "0.0.1", | ||
| "version": "17.0.50", |
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.
not sure why these are popping in here? but also don't seem wrong

The Ask AI custom tab allowed creating multiple questions but provided no way to remove ones that were no longer needed. This update adds a delete affordance while preventing removal of the final remaining custom question.
> [!WARNING]
>
>
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.