Skip to content

Add Kapa assistant in-page button#9651

Merged
chalin merged 3 commits intoopen-telemetry:mainfrom
chalin:chalin-m24-kapa-button-2026-0414
Apr 14, 2026
Merged

Add Kapa assistant in-page button#9651
chalin merged 3 commits intoopen-telemetry:mainfrom
chalin:chalin-m24-kapa-button-2026-0414

Conversation

@chalin
Copy link
Copy Markdown
Contributor

@chalin chalin commented Apr 14, 2026 •
edited
Loading

  • Contributes to Improve on-site Agent UX #9662
  • Enables a "launcher" (Ask AI) button at the bottom right of each viewport
  • Makes the dialog a bit bigger
  • Adds two questions, which act as an incentive for those not sure what or how to ask
  • Shortens the disclaimer
  • Fixes the white SVG logo (it wasn't the official one, and it was truncated)
  • Note that "Ask AI" is still in the search text, and Command-K still triggers the dialog
  • Preview: https://deploy-preview-9651--opentelemetry.netlify.app

Screenshots

Mobile:

Button Modal
image image

Desktop:

image

References

/cc @dvdksn

@chalin chalin requested a review from a team as a code owner April 14, 2026 22:46
@github-actions github-actions bot added the i18n Internationalization and localization label Apr 14, 2026
@otelbot-docs otelbot-docs bot added the missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer label Apr 14, 2026
tiffany76
tiffany76 approved these changes Apr 14, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it! ✨ Approving with a rubber stamp and one suggestion. Also, the alignment of the modal heading seems to be a little skewed to the right compared to everything underneath it.

Image

Comment thread i18n/en.yaml Outdated
@otelbot-docs otelbot-docs bot added ready-to-be-merged This PR is ready to be merged by a maintainer and removed missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer labels Apr 14, 2026
@chalin @tiffany76
Tiffany's feedback
9bb539f 
Co-authored-by: Tiffany Hrabusa <30397949+tiffany76@users.noreply.github.com>
@tiffany76
Copy link
Copy Markdown
Member

tiffany76 commented Apr 14, 2026

Also just noticed that the modal doesn't transition when dark mode is enabled.

I have no idea if any of this matters. Just reporting it in case.

Screenshot 2026-04-14 at 4 04 08 PM

@chalin
Copy link
Copy Markdown
Contributor Author

chalin commented Apr 14, 2026

I'll fix the header (it's actually an issue with the logo being white on white).
Thanks for pointing out the issue with dark mode. Let me see what I can do.

@chalin
Copy link
Copy Markdown
Contributor Author

chalin commented Apr 14, 2026

Icon fix. Dark-mode support pending. Let's merge this now and I'll followup with another PR.

@chalin chalin merged commit 2dc53a5 into open-telemetry:main Apr 14, 2026
24 checks passed
@chalin chalin deleted the chalin-m24-kapa-button-2026-0414 branch April 14, 2026 23:18
@chalin chalin mentioned this pull request Apr 14, 2026
Merged
@chalin
Copy link
Copy Markdown
Contributor Author

chalin commented Apr 14, 2026

Dark mode fix:

chalin
chalin commented Apr 14, 2026
View reviewed changes
Comment thread layouts/_partials/hooks/head-end.html
data-launcher-button-image="{{ "img/logos/opentelemetry-icon-white.svg" | relURL }}"
xx-DISABLED-data-launcher-button-text=""
xx-DOES-NOT-WORK-data-launcher-button-font-size="1rem"
data-font-size-lg="1rem"
Copy link
Copy Markdown
Contributor Author

@chalin chalin Apr 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dvdksn - is setting the font globally the only solution here? data-launcher-button-font-size="1rem" doesn't work (and I'm not even sure it should, even if the Kapa AI claims that it should :))

Copy link
Copy Markdown

@dvdksn dvdksn Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @chalin

I'm not sure tbh and I'm not w/ the kapa team anymore but let me cc @bauefikapa who might know :-)

Copy link
Copy Markdown
Contributor Author

@chalin chalin Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @dvdksn for cc'ing @bauefikapa

chalin
chalin commented Apr 14, 2026
View reviewed changes
Comment thread assets/scss/_styles_project.scss
Comment on lines +66 to +68
#kapa-widget-container {
--modal-inner-align: flex-end !important;
}
Copy link
Copy Markdown
Contributor Author

@chalin chalin Apr 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dvdksn - it would also be great if I didn't have to have this CSS override.

Copy link
Copy Markdown

@dvdksn dvdksn Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @bauefikapa

This was referenced Apr 14, 2026
Merged
Open
@KalleOlaviNiemitalo KalleOlaviNiemitalo mentioned this pull request Apr 17, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tiffany76 tiffany76 tiffany76 approved these changes

+1 more reviewer

@dvdksn dvdksn dvdksn left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

i18n Internationalization and localization ready-to-be-merged This PR is ready to be merged by a maintainer

Projects

SIG Comms: PRs & Issues
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chalin @tiffany76 @dvdksn
⚡