Skip to content

Reduce About and Relations card height on entity overview pages - #731

Open
VishishtaDilsara wants to merge 1 commit into
openchoreo:mainfrom
VishishtaDilsara:fix/3906-reduce-card-height
Open

Reduce About and Relations card height on entity overview pages#731
VishishtaDilsara wants to merge 1 commit into
openchoreo:mainfrom
VishishtaDilsara:fix/3906-reduce-card-height

Conversation

@VishishtaDilsara

@VishishtaDilsara VishishtaDilsara commented Aug 4, 2026

Copy link
Copy Markdown

Purpose

Related to openchoreo/openchoreo#3906

Reduce the height of the About and Relations cards on entity overview pages by lowering the EntityCatalogGraphCard height from 400px to 300px.

Goals

  • Reduce the vertical size of the About and Relations cards by approximately 25%.
  • Maintain the existing layout and responsiveness.

Approach

  • Updated all EntityCatalogGraphCard instances in packages/portal-app/src/components/catalog/EntityPage.tsx to use height={300} instead of height={400}.
  • The About card continues to resize automatically through the existing grid layout.

User stories

  • As a user, I can view more information on entity overview pages with reduced card height while keeping the relations graph readable.

Release note

Reduced the height of the About and Relations cards on entity overview pages.

Documentation

N/A – This is a UI layout adjustment and does not require documentation changes.

Training

N/A

Certification

N/A – No certification impact.

Marketing

N/A

Automation tests

  • Unit tests
    • N/A
  • Integration tests
    • N/A

Security checks

  • Followed secure coding standards? Yes
  • Ran FindSecurityBugs plugin? No (not applicable for this UI-only change)
  • Confirmed no secrets were committed? Yes

Samples

N/A

Related PRs

N/A

Migrations (if applicable)

N/A

Test environment

  • Code reviewed locally.
  • UI could not be verified due to local environment setup issues.

Learning

N/A

Summary by CodeRabbit

  • Style
    • Reduced the catalog relationship graph height from 400px to 300px on entity overview pages.
    • Applied the updated sizing consistently across components, APIs, systems, resources, platforms, types, and workflows.
    • Graph relationships and behavior remain unchanged.

@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Changes

Catalog graph sizing

Layer / File(s) Summary
Reduce entity overview graph heights
packages/portal-app/src/components/catalog/EntityPage.tsx, .changeset/tiny-lands-draw.md
EntityCatalogGraphCard heights change from 400px to 300px across entity overview pages. Graph relations remain unchanged. A patch changeset records the portal app update.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related issues

  • openchoreo/backstage-plugins#640 — Matches the same EntityCatalogGraphCard height reduction.
  • openchoreo/openchoreo#3906 — Tracks the height reduction across EntityPage.tsx.

Suggested reviewers: kaviththiranga

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly and concisely describes the main change to About and Relations card height on entity overview pages.
Description check ✅ Passed The description covers the purpose, goals, approach, testing status, release note, and all template sections with relevant N/A explanations.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

Signed-off-by: VishishtaDilsara <rgvdilsara2002@gmail.com>
@VishishtaDilsara
VishishtaDilsara force-pushed the fix/3906-reduce-card-height branch from 182ebd0 to c2300ef Compare August 4, 2026 08:46
@kaviththiranga

Copy link
Copy Markdown
Contributor

@VishishtaDilsara shall we update the PR with some screenshots to indicate the changes visually? Thanks

@codecov

codecov Bot commented Aug 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@LakshanSS LakshanSS 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.

@VishishtaDilsara Kind reminder on this

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.

3 participants