Skip to content

feat(assistant-editor): group knowledge base into a contrasted inset panel#557

Merged
philmerrell merged 1 commit into
developfrom
feature/kb-section-inset-grouping
Jul 6, 2026
Merged

feat(assistant-editor): group knowledge base into a contrasted inset panel#557
philmerrell merged 1 commit into
developfrom
feature/kb-section-inset-grouping

Conversation

@philmerrell

Copy link
Copy Markdown
Contributor

What

Visually distinguishes the Knowledge base section on the Assistant editor by wrapping it in a mildly contrasted, grouped inset panel instead of the flat border-t divider used by every other section.

Why

The Knowledge base area is the densest, most interactive part of the editor (uploads, connectors, web crawls, sync controls). Giving it a grouped container makes it read as a distinct workspace rather than one more divider-separated block.

Changes

Single-line class change on the <section> in assistant-form.page.html:

  • Before: space-y-4 border-t border-gray-200 pt-8 dark:border-gray-700
  • After: space-y-4 rounded-2xl border border-gray-200 bg-gray-100/60 p-5 dark:border-gray-700 dark:bg-gray-800/40 sm:p-6

Reviewer notes

  • The form column is bg-gray-50, so a gray-100/60 fill is a deliberately mild contrast step — grouped, not loud.
  • Inner lists (Web sources / Uploaded Documents) are bg-white, so they now float on the tint, reinforcing the grouping.
  • The border-t/pt-8 divider was dropped because the card provides its own separation; the form's space-y-8 preserves the gap above it.
  • Dark mode uses dark:bg-gray-800/40 over the gray-900 column for the same subtle lift.
  • Tokens (rounded-2xl, border-gray-200, dark:border-gray-700) match the editor's existing list/dialog idiom.
  • Editor is behind auth; not driven through the live preview here. Best eyeballed by opening any assistant's editor.

🤖 Generated with Claude Code

…panel

Wrap the Knowledge base section on the Assistant editor in a rounded,
bordered inset with a subtle gray-100/60 fill instead of the flat
border-t divider shared by other sections. Against the gray-50 form
column this reads as a mildly contrasted group, and the white inner
lists (Web sources / Uploaded Documents) now float on the tint to
reinforce the grouping. Drops the border-t/pt-8 divider since the card
provides its own separation; the form's space-y-8 preserves the top gap.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
@philmerrell philmerrell merged commit 209e0cf into develop Jul 6, 2026
4 checks passed
@philmerrell philmerrell deleted the feature/kb-section-inset-grouping branch July 6, 2026 15:19
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