Skip to content

Conversation

barjin
Copy link
Contributor

@barjin barjin commented Sep 18, 2025

Adds an Ask AI button to the main top bar and an Ask AI tab to the search modal.

@barjin barjin self-assigned this Sep 18, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enables the Kapa AI chat widget by adding an "Ask AI" button to the main navigation bar alongside the existing Algolia search functionality. The implementation replaces the previous A/B testing setup with a permanent integration of the Kapa AI widget.

  • Adds a dedicated "Ask AI" button component that opens the Kapa AI chat widget
  • Removes unused Inkeep styling overrides and configuration
  • Updates the search modal package to support the new dual-search interface

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
static/inkeep-overrides.css Removes unused Inkeep font size overrides
apify-docs-theme/src/theme/SearchBar/styles.css Adds styling for the new Ask AI button and container layout
apify-docs-theme/src/theme/SearchBar/index.js Implements KapaAIButton component and integrates it with existing search
apify-docs-theme/src/config.js Enables Kapa AI widget script and removes unused Inkeep configuration
apify-docs-theme/package.json Updates docs-search-modal package version

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added this to the 123rd sprint - Tooling team milestone Sep 18, 2025
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Sep 18, 2025
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Comment @cursor review or bugbot run to trigger another review on this PR

@barjin barjin added the adhoc Ad-hoc unplanned task added during the sprint. label Sep 18, 2025
@apify-service-account
Copy link

Preview for this PR was built for commit 88354ee and is ready at https://pr-1937.preview.docs.apify.com!

'data-font-size-lg': '1.8rem',
'data-font-size-xl': '2.0rem',
async: true,
},
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure where exactly, but would it make sense to have the width set dynamically? On a 4K screen, it's just a narrow noodle:
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure what the original values were for. In the last commit, I used the data-scale-factor: '1.6' attribute, since we're still using the base font size of 10px.

This made the AI modal look much better, since we're scaling margins / paddings now as well.

)}
</BrowserOnly>
);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

If I talk to the bot, then go to search and from the search, I'll trigger the AI bot again, it will ask the question, but I'm not scrolled to it, I keep seeing the first message of my previous discussion. Could we scroll to the current question being asked?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't think this is currently possible with the Kapa AI widget API, sorry.

@apify-service-account
Copy link

Preview for this PR was built for commit 70bd5996 and is ready at https://pr-1937.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 8d1120c7 and is ready at https://pr-1937.preview.docs.apify.com!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants