Skip to content

fix(dashboard): center instance card grid on wide screens - #1521

Open
HDX6692 wants to merge 1 commit into
Comfy-Org:mainfrom
HDX6692:patch-1
Open

HDX6692 wants to merge 1 commit into
Comfy-Org:mainfrom
HDX6692:patch-1

Conversation

@HDX6692

@HDX6692 HDX6692 commented Sep 11, 2026

Copy link
Copy Markdown

Fixes #1520

Problem

The Dashboard instance-card row (.chooser-family-grid) is a grid with fixed-width tracks (repeat(auto-fit, 280px)) and justify-content: start, so on wide windows the whole track group hugs the left edge while the logo and search box above are centered. Details, screenshots and DevTools traces in #1520.

Fix

Center the track group with an overflow-safe fallback:

  • justify-content: startjustify-content: safe center

With safe center, narrow windows keep the current left-aligned behavior when the tracks overflow, so only wide layouts change.

Verification

Updated justify-content to use 'safe center' for better alignment.
@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

✅ All contributors have signed the CLA. Thank you! This PR is ready to be merged.
Posted by the CLA Assistant Lite bot.

@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 7bea102e-b9d5-43af-ae66-f3e619cf674f

📥 Commits

Reviewing files that changed from the base of the PR and between b427984 and 021c541.

📒 Files selected for processing (1)
  • src/renderer/src/views/chooser/ChooserFamilyGrid.vue

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

The dashboard instance-card grid changes from start alignment to safe center alignment. Wide layouts now center the card tracks, while narrow layouts avoid overflow clipping.

Changes

Dashboard layout

Layer / File(s) Summary
Center instance-card tracks
src/renderer/src/views/chooser/ChooserFamilyGrid.vue
The grid changes justify-content from start to safe center. Suggested reviewers: kosinkadink

Priority: ➖ Normal

Severity of issue fixed: Medium

Merge Risk: ⚪ Minimal · up to 021c5

The dashboard card grid change is limited in scope and aligns with the stated layout objective. No merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Issue #1520 requires the Dashboard instance-card grid to center fixed-width tracks on wide screens and retain overflow-safe behavior on narrow screens. The PR changes .chooser-family-grid from `just…
Out of Scope Changes check ✅ Passed The only change is the justify-content declaration in the affected chooser grid. It directly addresses Issue #1520 and introduces no unrelated behavior. A small alignment change, neatly in scope—no …
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

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

@coderabbitai
coderabbitai Bot requested a review from Kosinkadink September 11, 2026 03:46
@HDX6692

HDX6692 commented Sep 11, 2026

Copy link
Copy Markdown
Author

I have read and agree to the Contributor License Agreement

comfy-legal added a commit to Comfy-Org/comfy-cla that referenced this pull request Sep 11, 2026

This branch has not been deployed

No deployments
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.

Dashboard: instance card row stays left-aligned on wide screens instead of centering

1 participant