Skip to content

Fix/172 the add more pins button sometimes overlaps with other pins#226

Merged
felix-schultz merged 2 commits intodevfrom
fix/172-the-add-more-pins-button-sometimes-overlaps-with-other-pins
Aug 23, 2025
Merged

Fix/172 the add more pins button sometimes overlaps with other pins#226
felix-schultz merged 2 commits intodevfrom
fix/172-the-add-more-pins-button-sometimes-overlaps-with-other-pins

Conversation

@felix-schultz
Copy link
Copy Markdown
Member

This pull request introduces several UI improvements and refactors to enhance the flow node and pin interactions, as well as the bubble actions component. The most important changes are grouped below by theme.

Flow Pin UI and Behavior Improvements

  • Added a delete button (Trash2 icon) directly to dynamic pins in FlowPinInnerComponent, allowing users to remove pins without using the context menu. The button appears on hover for better discoverability. [1] [2] [3] [4]
  • Removed the context menu for dynamic pins in FlowPin, streamlining the pin removal process to use the new inline delete button instead.

Flow Node Pin Action Positioning

  • Adjusted the positioning logic for pin actions in FlowPinAction to improve alignment, ensuring consistent placement for input and output pins.

Bubble Actions Component Enhancement

  • Added a style prop to BubbleActions and BubbleMenu, allowing for custom inline styling of bubble action menus and improving flexibility for UI placement. [1] [2] [3] [4] [5]

These changes collectively make the UI more intuitive and customizable, especially around flow pin management and bubble action presentation.

@felix-schultz felix-schultz merged commit 5591b02 into dev Aug 23, 2025
1 of 3 checks passed
@felix-schultz felix-schultz deleted the fix/172-the-add-more-pins-button-sometimes-overlaps-with-other-pins branch August 23, 2025 13:28
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.

1 participant