Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: kanban column margin should extend to bottom #8394

Merged
merged 4 commits into from
Nov 8, 2024

Conversation

harshrajeevsingh
Copy link
Contributor

Fixes: #7868

Screencast.from.2024-11-08.13-33-47.webm

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Fixed kanban column margins not extending to bottom of view by removing fixed height calculation from StyledBoardContentContainer in RecordBoard component.

  • Removed height: calc(100% - 48px) from StyledBoardContentContainer in /packages/twenty-front/src/modules/object-record/record-board/components/RecordBoard.tsx to allow natural flex expansion
  • Container now properly fills available vertical space based on parent height and flex properties

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

Copy link
Contributor

@ehconitin ehconitin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not fixed @harshrajeevsingh

Screenshot 2024-11-08 at 14 10 23

@harshrajeevsingh
Copy link
Contributor Author

harshrajeevsingh commented Nov 8, 2024

@ehconitin Thanks, I see. It worked fine for me because the content list was large and extended to the bottom of container. Give me more time to look into it.

Copy link
Contributor

@ehconitin ehconitin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks @harshrajeevsingh! ;)

Copy link
Member

@FelixMalfait FelixMalfait left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect thank you!

@FelixMalfait FelixMalfait merged commit 61cdc0e into twentyhq:main Nov 8, 2024
Copy link

github-actions bot commented Nov 8, 2024

Thanks @harshrajeevsingh for your contribution!
This marks your 10th PR on the repo. You're top 3% of all our contributors 🎉
See contributor page - Share on LinkedIn - Share on Twitter

Contributions

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.

Kanban columns margin no longer extend to the bottom
3 participants