Disable TLBR snapping for non-container elements #1104
Assignees
Milestone
Comments
|
Can we look into this one and see if it would result in a better user experience? We can always add additional snap points, too if we receive the relevant feedback to do so. |
|
I'll have a lookie lookie 👀 |
nikku
added a commit
to bpmn-io/diagram-js
that referenced
this issue
Jun 26, 2019
Removing the default TRBL snapping in favor of fine-grained control in using applications. Related to bpmn-io/bpmn-js#1104
|
I think the solution you proposed makes a lot of sense. It feels more natural to align to the center of another shape, but never to any of the bounds.
|
|
I'm struggeling and juggeling with writing test cases for this one in bpmn-js. ⛑️ |
nikku
added a commit
that referenced
this issue
Jun 27, 2019
|
Feels way better now and moves the responsibility for special snapping to bpmn-js (and friends). |
bpmn-io-tasks
bot
added
needs review
nikku
added a commit
that referenced
this issue
Jun 27, 2019
nikku
added a commit
that referenced
this issue
Jun 27, 2019
Adds TRBL snapping with containers only. Closes #1104
|
@barmac Ready for review via bpmn-io/diagram-js#376 and #1108. Please check it out. For me it feels like another substantial UX improvement. |
nikku
added a commit
to bpmn-io/diagram-js
that referenced
this issue
Jun 27, 2019
Removing the default TRBL snapping in favor of fine-grained control in using applications. Related to bpmn-io/bpmn-js#1104
|
Closed via #1108 |
barmac
pushed a commit
that referenced
this issue
Jul 2, 2019
Adds TRBL snapping with containers only. Closes #1104
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
The latest release (
v4) introduces snapping on all sides, including top, bottom, left and right.This results in a lot of snapping noise when moving normal flow elements.
Under the assumption that plain flow elements are usually center aligned we could consider removing that feature for these elements.
Describe the solution you'd like
Snap non-container nodes, excluding text annotations only using their mid.
Describe alternatives you've considered
See the noise as a feature (I guess?).
Additional context
See the attached screencast for a demonstration of this issue:
The text was updated successfully, but these errors were encountered: