Skip to content

Conversation

gadenbuie
Copy link
Contributor

A few style-level changes to align Python and R packages and improve styles:

  • Use same styles.css file in both packages
  • Add .querychat to the chat_ui() component used by querychat
  • Add .querychat-sidebar to the sidebar() created by querychat_sidebar()
  • Use new client argument in Python examples
  • Adjust position of sidebar collapse toggle in querychat_sidebar() to stay out of the way of tool cards in the sidebar.

@gadenbuie gadenbuie requested a review from Copilot August 19, 2025 16:22
Copy link

@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 standardizes styles between Python and R packages and improves component identification by adding CSS classes to querychat components.

  • Unified styles.css files between Python and R packages for consistent styling
  • Added .querychat and .querychat-sidebar CSS classes to components for better customization
  • Updated Python examples to use the new client argument instead of deprecated create_chat_callback

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pkg-r/man/querychat_ui.Rd Updated documentation formatting and cross-references
pkg-r/man/querychat_init.Rd Improved documentation formatting and line wrapping
pkg-r/inst/htmldep/styles.css Simplified CSS to match Python package and added sidebar collapse positioning
pkg-r/R/querychat_app.R Added bslib-page-dashboard class and cleaned up function call
pkg-r/R/querychat.R Added CSS classes to components and improved documentation
pkg-r/NEWS.md Added changelog entry for new CSS classes
pkg-py/src/querychat/static/css/styles.css Unified with R package styles
pkg-py/src/querychat/querychat.py Added querychat CSS classes to UI components
pkg-py/examples/*.py Updated to use client instead of create_chat_callback
pkg-py/CHANGELOG.md Added changelog entry for new CSS classes

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

@gadenbuie gadenbuie merged commit 673e863 into main Aug 19, 2025
16 checks passed
@gadenbuie gadenbuie deleted the feat/querychat-styles branch August 19, 2025 16:23
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.

1 participant