Skip to content

small polishes to UI based on review - #2165

Merged
cpvalente merged 6 commits into
masterfrom
ui-polish
Aug 2, 2026
Merged

small polishes to UI based on review#2165
cpvalente merged 6 commits into
masterfrom
ui-polish

Conversation

@cpvalente

Copy link
Copy Markdown
Owner

No description provided.

@cpvalente
cpvalente requested a review from Copilot August 2, 2026 10:35
@coderabbitai

coderabbitai Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3cb45b24-a55f-47f7-b2ee-7a1bb64f2c93

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI 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.

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.

Comment thread apps/server/src/api-data/rundown/rundown.validation.ts
@cpvalente
cpvalente merged commit 12fc76b into master Aug 2, 2026
4 checks passed
@cpvalente
cpvalente deleted the ui-polish branch August 2, 2026 10:55
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