Skip to content

feat(list): use Bin vertical grid for List view#2237

Merged
ra3orblade merged 1 commit into
developfrom
fix/js-9573-list-view-bin-vertical-grid
May 29, 2026
Merged

feat(list): use Bin vertical grid for List view#2237
ra3orblade merged 1 commit into
developfrom
fix/js-9573-list-view-bin-vertical-grid

Conversation

@yasyuk

@yasyuk yasyuk commented May 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Compact row height 28px → 42px and detailed icon size 48px → 32px to match Bin spacing
  • Hover replaced with full-row ::before highlight (border-radius: 8px, drag handle sits outside at left: 11px)
  • Selection highlight border-radius overridden to 8px to match hover
  • dropTarget/selectionTarget flex-centered so content is vertically centered at the new row height
  • isRegular icon treatment matches Bin's detailed icon styling (border-radius, background)
  • New Object add row inherits row height and uses the same hover/centering as content rows

Closes JS-9573

Test plan

  • Compact list view: rows are 42px tall, content centered, hover shows rounded highlight with drag handle outside
  • Detailed list view: rows are 64px tall, icon is 32px with background/border-radius, same hover behaviour
  • Selecting a record: selection highlight matches hover shape (border-radius 8px)
  • New Object row: same height, hover, and vertical centering as content rows
  • No horizontal scroll introduced
  • Inline list view (in editor) is unaffected

🤖 Generated with Claude Code

- Compact row height: 28px → 42px to match Bin spacing
- Detailed row icon size: 48px → 32px to match Bin detailed view
- Hover uses full-row ::before highlight (border-radius: 8px) with drag handle outside (left: 11px)
- Selection highlight border-radius matches hover (8px override)
- dropTarget/selectionTarget flex-centered so content is vertically centered at new height
- isRegular iconObject styling matches Bin's detailed icon treatment
- New Object add row inherits row height and uses same hover/centering

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yasyuk

yasyuk commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA.

@ra3orblade ra3orblade merged commit bde69cd into develop May 29, 2026
3 checks passed
@ra3orblade ra3orblade deleted the fix/js-9573-list-view-bin-vertical-grid branch May 29, 2026 15:51
@github-actions github-actions Bot locked and limited conversation to collaborators May 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants