Skip to content

feat: add prop to remove vertical margin between stacked items#889

Open
franciscohanna92 wants to merge 2 commits intonamespace-ee:mainfrom
franciscohanna92:master
Open

feat: add prop to remove vertical margin between stacked items#889
franciscohanna92 wants to merge 2 commits intonamespace-ee:mainfrom
franciscohanna92:master

Conversation

@franciscohanna92
Copy link

closes #888

@lnagel lnagel added the outdated label Feb 5, 2026
@lnagel
Copy link
Member

lnagel commented Feb 5, 2026

This PR has been marked as outdated. Here's why:

This PR added a verticalMarginBetweenStackedItems boolean prop to control vertical spacing between stacked items. The codebase has since been rewritten from JavaScript to TypeScript (v0.30.0-beta.5), and this feature has been superseded by a more flexible approach: the new itemVerticalGap prop (a number, not a boolean) now provides precise control over the vertical gap between stacked items. It works alongside itemHeightRatio and takes precedence when specified. The stacking logic in src/lib/utility/calendar.tsx already uses itemVerticalGap throughout. Additionally, this PR targeted files that no longer exist in their original form — src/lib/Timeline.js is now src/lib/Timeline.tsx, tests have been migrated from Enzyme to @testing-library/react, and the __fixtures__ and __tests__ files referenced have been restructured.

Thank you for the contribution! The idea/fix was appreciated even if the codebase has moved on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Row padding

2 participants