-
Notifications
You must be signed in to change notification settings - Fork 26
feat: Add buttons to suggest agency tools #424
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
Conversation
Signed-off-by: Marcel Klehr <[email protected]>
Signed-off-by: Marcel Klehr <[email protected]>
c3a4357 to
04e444c
Compare
marcoambrosini
left a comment
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.
I'd use tertiary buttons here and a flex layout for the container, wrapping them only when there's no available space.
@marcoambrosini Good point! Thanks! What do you think about the general idea? |
Signed-off-by: Marcel Klehr <[email protected]>
Signed-off-by: Marcel Klehr <[email protected]>
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.
Considering they are hardcoded suggestions, I’d say the first one is better as:
- What’s the weather tomorrow?
Automatically taking your location from the profile.
And just layout-wise:
- It would be good to put the suggestions directly above the input field so it’s related
- Left-align them
- Display them more as chips, so it seems a bit clickable (I preferred secondary to tertiary, but I get the concern with the nav on the top)
Agreed. Context agent doesn't offer this, yet, though :S |
Is the problem the "tomorrow" part or the "place" part? Cause we could also do "What’s the weather today?", automatically taking your city from the profile, or asking a follow-up if it’s not set? Would just be strange if we hardcode "Berlin". |
Yep, we can do that :) |
|
@marcoambrosini mentioned that it might be nice to group suggested prompts by categories (maybe apps?) and only display the category names in the buttons, instead of the full prompts |
Yeah, we talked about that (also e.g. vertically aligning the input field on emptycontent, like others do) – but I would recommend to tackle this step by step to prevent blocking or overloading this. Let’s get this adjusted and in, and then continue to the next PR. Does that make sense @marcelklehr? :) |
…sage input Signed-off-by: Marcel Klehr <[email protected]>
jancborchardt
left a comment
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.
Yup, let's go with this and then we can continue with the other changes @marcoambrosini and you suggested. :)



Kooha-2025-12-11-15-02-25.webm
These buttons would only be present if context_agent is enabled.