Skip to content

Adding enable chunks button - #11

Merged
karvyq merged 2 commits into
mainfrom
configurable-ui
Feb 24, 2026
Merged

Adding enable chunks button#11
karvyq merged 2 commits into
mainfrom
configurable-ui

Conversation

@karvyq

@karvyq karvyq commented Feb 20, 2026

Copy link
Copy Markdown
Contributor

This is a small change adding a button for enabling/disabling chunk usage in retrieval.

Screen.Recording.2026-02-20.at.10.07.24.AM.mov

@Rishit2000

Copy link
Copy Markdown
Contributor

Hi @karvyq , the functionality looks perfect. I just wanted to check if we could simplify the change without altering the ready-to-use Switch component. Given we already have the React state, can we do something like this:

<span
  className={cn(
    "pointer-events-none absolute inset-0 flex items-center justify-center text-[9px] font-medium",
    chatConfig.enableChunks
      ? "text-primary-foreground"
      : "text-muted-foreground"
  )}
>
  {chatConfig.enableChunks ? "On" : "Off"}
</span>

@karvyq

karvyq commented Feb 22, 2026

Copy link
Copy Markdown
Contributor Author

Thanks @Rishit2000! I am still new to React and learning. I have made the suggested changes. Does this look better?

@Rishit2000 Rishit2000 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. Thanks @karvyq, likewise when it comes to React!

@karvyq
karvyq merged commit c9f693b into main Feb 24, 2026
1 check passed
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.

2 participants