Skip to content

Conversation

@ShiboSoftwareDev
Copy link
Contributor

@ShiboSoftwareDev ShiboSoftwareDev commented Dec 10, 2025

Is this good naming for customizing that margin?
Screenshot 2025-12-10 at 8 42 55 AM

Copy link
Member

@imrishabh18 imrishabh18 left a comment

Choose a reason for hiding this comment

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

Margin is pretty confusing, it could also mean margin between two boards which are in the same panel. From the image it seems you want the edge margin, so something like edgeMarginRight or edgePaddingRight seems appropriate to me.

@ShiboSoftwareDev ShiboSoftwareDev changed the title add margin prop to panel add edge margin props to panel Dec 10, 2025
Copy link
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

when something is an inner margin its a padding. edgePadding seems okay

slightly worried about the interaction between this prop and tooling rails but we’ll figure it out

@ShiboSoftwareDev ShiboSoftwareDev merged commit cb5d718 into tscircuit:main Dec 10, 2025
@tscircuitbot
Copy link
Contributor


Thank you for your contribution! 🎉

PR Rating:
Impact: Tiny

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


1 similar comment
@tscircuitbot
Copy link
Contributor


Thank you for your contribution! 🎉

PR Rating:
Impact: Tiny

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


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.

4 participants