fix(platform): UI polish and layout fixes

[Israeltheminer]

Summary

• Layout fixes: error display flex stretch, data-table overflow-hidden for rounded borders, org ID truncation with text-nowrap
• UX improvements: bold entity names in delete confirmation dialogs, active state styling on chat history toggle, date-grouped chat search results, floating close button on chat sidebar, org creation page with header/logo
• Form fixes: login email .trim() validation, remove unnecessary useEffect for password error clearing, search input size flexibility (remove hardcoded size="sm")
• Misc: gate Settings menu on organizationId, rename "Manage members" to "Members"

Test plan

• Verify error display fills available space
• Verify data-table content doesn't overflow rounded corners
• Verify delete dialogs show bold entity names
• Verify chat history toggle shows active state when sidebar is open
• Verify chat search groups results by date
• Verify floating close button appears on chat sidebar
• Verify org creation page shows header with logo
• Verify login form trims email whitespace
• Verify Settings menu hidden when no organization loaded

Summary by CodeRabbit

Release Notes

• New Features

  • Added grouped chat search functionality organized by date
  • Implemented closable chat history panel with dedicated close button
  • Added header section to organization form with logo and user menu

• Bug Fixes

  • Fixed data table overflow clipping issue
  • Improved layout distribution and spacing

• UI/UX Improvements

  • Enhanced entity name styling in delete confirmation dialogs
  • Added active state styling to chat history toggle button
  • Improved field label and value layout with better text handling
  • Added informational tooltips to copyable fields

• Updates

  • Settings menu now requires organization context
  • Email input now trims whitespace
  • Updated "Manage members" label to "Members"

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR comprises multiple UI refinements and styling adjustments across the platform. Changes include adding flex layout properties to error display, introducing styled entity names in dialogs through split-based rendering, refactoring the chat search dialog from a flat list to a grouped structure by formatted date, restructuring the history panel with a close button, adding a header to the organization form, narrowing Settings menu visibility in the user button, and adjusting styling for input fields, copyable text, and data tables. The login route adds email trimming and removes automatic password error clearing on input changes. A translation key label is updated from "Manage members" to "Members".

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• feat: ui improvements, bug fixes, and accessibility enhancements #127: Both PRs modify the chat header and history panel UI rendering, including styling adjustments and structural changes to how the history toggle is displayed and interacted with.
• fix: UI improvements and bug fixes #187: Both PRs adjust UserButton menu/visibility logic and its integration within form layouts, affecting when the Settings item renders based on component state.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 18.18% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix(platform): UI polish and layout fixes' is directly related to the changeset, which contains multiple UI and layout improvements across various components.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/ui-polish

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 3

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

services/platform/app/routes/_auth/log-in.tsx (1)

145-167: ⚠️ Potential issue | 🟠 Major

Login can get stuck after wrong credentials.
errors.password disables the submit button (Line 162–167), but the error is only cleared in handleSubmit. After a wrong password, the user can’t resubmit because the button stays disabled. Clear the password error on input change (or remove it from the disabled condition).

🔧 Proposed fix (clear password error on input change)

             <Input
               id="password"
               type="password"
               size="lg"
               label={t('password')}
               placeholder={t('passwordPlaceholder')}
               disabled={isSubmitting}
               autoComplete="current-password"
               errorMessage={errors.password?.message}
               className="shadow-[0px_1px_2px_0px_rgba(0,0,0,0.05)]"
-              {...form.register('password')}
+              {...form.register('password', {
+                onChange: () => form.clearErrors('password'),
+              })}
             />

🤖 Fix all issues with AI agents

In `@services/platform/app/components/ui/entity/entity-delete-dialog.tsx`:
- Around line 99-109: The current rendering logic for styledDescription uses
translations.description.split('{name}') and only inserts entityName between
parts[0] and parts[1], losing additional {name} occurrences; update the
styledDescription construction (the parts variable and JSX that builds
styledDescription) to iterate over all parts and interleave a span with
entityName (className="text-foreground font-semibold") between each part so
every {name} placeholder is replaced, e.g., map over parts and return [part,
<span key=...>{entityName}</span>] for all but the last part, ensuring multiple
occurrences are preserved.

In `@services/platform/app/features/chat/components/chat-actions.tsx`:
- Around line 119-132: The delete confirmation currently splits
tChat('deleteConfirmation', { title: '\x00' }) and assumes parts[1] exists; add
a defensive check so missing placeholder doesn't break rendering: after
computing parts, if parts.length < 2 fall back to using the translated string
with the real title (e.g. call tChat('deleteConfirmation', { title: chat.title
})) or at minimum set post = parts[1] ?? '' and render pre, title, post; update
the inline IIFE around tChat/deleteConfirmation/parts to use this fallback.

In `@services/platform/app/routes/dashboard/`$id/chat.tsx:
- Around line 39-45: The close control button currently has no explicit type
(defaulting to "submit") which can accidentally submit a surrounding form;
update the button element that calls setIsHistoryOpen(false) and renders
<PanelLeftClose /> to include type="button" so it will act as a non-submitting
control, ensuring the onClick handler only toggles history visibility.