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

LEAF 4585 prevent negative coordinates for workflow steps #2600

Conversation

aerinkayne
Copy link
Contributor

Applies a minimum value of 0 to the coordinates of workflow steps to resolve an issue where users could drag and save negative coordinates.

Server-side:
-saved x and y positions have a minimum value of 0.

Front end:
-add a local minimum x value used for CSS positioning to handle any existing saved negative x values so that they display on the screen (a min y value was already present).

-add jsPlumb draggable config option 'allowNegative' with a false value to prevent dragging steps offscreen (left or top)

Testing / Impact

Workflow Editor

-steps cannot be dragged to negative coordinates (outside of left or top of screen)
-steps that have negative coordinates saved still show on screen (portal workflow_steps can be edited in adminer)

@Pelentan Pelentan changed the base branch from master to rc/2024-11-20/Sprint-85-c2 November 20, 2024 13:18
@Pelentan Pelentan added the With QA Ticket is to QA. No changes unless pulled back to in progress label Nov 20, 2024
@Pelentan Pelentan merged commit 8e11090 into rc/2024-11-20/Sprint-85-c2 Nov 20, 2024
5 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
With QA Ticket is to QA. No changes unless pulled back to in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants