Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR applies small UI/UX refinements and aligns rundown title constraints between client and server by introducing a 64-character limit and improving header/title presentation across views.
Changes:
- Add a shared server-side validator for rundown titles, including a new 64-character maximum.
- Improve header layout across multiple client views by grouping logo + title into a dedicated “brand” wrapper with consistent alignment rules.
- Refine the overview title display by trimming inputs and rendering project/rundown titles conditionally with updated styling; add
maxLength={64}to rundown title inputs.
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| apps/server/src/api-data/rundown/rundown.validation.ts | Extracts and enhances rundown title validation, adding a 64-char max length check. |
| apps/client/src/views/timeline/TimelinePage.tsx | Wraps logo + title in a brand container using cx for conditional styling. |
| apps/client/src/views/timeline/TimelinePage.scss | Adds brand wrapper styling to improve alignment/overflow behavior. |
| apps/client/src/views/studio/Studio.tsx | Wraps logo + title in a brand container for consistent header layout. |
| apps/client/src/views/studio/Studio.scss | Adds brand wrapper styling consistent with other views. |
| apps/client/src/views/countdown/Countdown.tsx | Wraps logo + title in a brand container using cx. |
| apps/client/src/views/countdown/Countdown.scss | Adds brand wrapper styling consistent with other views. |
| apps/client/src/views/backstage/Backstage.tsx | Wraps logo + title in a brand container using cx. |
| apps/client/src/views/backstage/Backstage.scss | Adds brand wrapper styling + responsive alignment adjustments. |
| apps/client/src/features/overview/composite/TitleOverview.tsx | Trims titles and conditionally renders project/rundown title blocks. |
| apps/client/src/features/overview/composite/TitleOverview.module.scss | Updates layout/typography for the revised TitleOverview rendering logic. |
| apps/client/src/features/app-settings/panel/manage-panel/ManageRundownForm.tsx | Adds maxLength={64} to the rundown title input. |
| apps/client/src/features/app-settings/panel/manage-panel/composite/RundownRenameForm.tsx | Adds maxLength={64} to the rundown rename input. |
| apps/client/src/common/components/navigation-menu/ViewNavigationMenu.tsx | Simplifies “saved changes” indicator logic to reflect URL custom params only. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.