Skip to content

fix(frontend): align layout elements and refine breadcrumbs based on page feedback#600

Open
dembrane-sam-bot wants to merge 1 commit into
mainfrom
sam/project-page-feedback-fixes
Open

fix(frontend): align layout elements and refine breadcrumbs based on page feedback#600
dembrane-sam-bot wants to merge 1 commit into
mainfrom
sam/project-page-feedback-fixes

Conversation

@dembrane-sam-bot
Copy link
Copy Markdown
Contributor

What this changes

  • PageContainer alignment shifted from centered (mx-auto) to left-aligned (mr-auto), ensuring all project pages align with the sidebar navigation edge — PageContainer.tsx
  • QR code card on the project home page is left-aligned instead of centered — ProjectQRCode.tsx
  • "Start a chat" button on project home page uses the brand's Royal Blue default theme styling (was: off-brand Mantine blue and filled bypass) — ProjectHomeRoute.tsx
  • Breadcrumbs no longer display duplicate "Home" or redundant "Overview" crumbs when landing on workspace/project home pages — AppBreadcrumbs.tsx

Confidence

Confidence: high. Direct, focused layout and style fixes targeting the specific issues reported in page feedback, aligning perfectly with frontend brand guidelines.

…page feedback

- align PageContainer left (using mr-auto) instead of centering (mx-auto)
- left-align project sharing QR code Paper card and contents
- use primary brand styling for the 'Start a chat' button by removing color='blue' and variant='filled'
- refine breadcrumbs to avoid duplicate 'Home'/'Overview' crumbs on workspace/project home pages
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 27, 2026

Warning

Review limit reached

@dembrane-sam-bot, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 33 minutes and 59 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 68ce3895-bc6c-4ba4-af6f-f7561f3448ed

📥 Commits

Reviewing files that changed from the base of the PR and between b12baf5 and 3ea56d0.

📒 Files selected for processing (4)
  • echo/frontend/src/components/layout/PageContainer.tsx
  • echo/frontend/src/components/project/ProjectQRCode.tsx
  • echo/frontend/src/features/sidebar/breadcrumbs/AppBreadcrumbs.tsx
  • echo/frontend/src/routes/project/ProjectHomeRoute.tsx
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sam/project-page-feedback-fixes

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 and usage tips.

@github-actions
Copy link
Copy Markdown

Hi @dembrane-sam-bot!

Thank you for contributing to Dembrane ECHO! Before we consider your Pull Request, we ask that you sign our Contributor License Agreement (CLA). This is only required for your first Pull Request.

Please review the CLA, and sign it by adding your GitHub username to the contributors.yml file. Thanks!

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