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

Hotfix Update Marker Ends to Make Pixel Perfect #86

Merged
merged 3 commits into from
Jan 27, 2025

Conversation

kurunbelemir
Copy link
Collaborator

There is a gap between source and target ends of the paths.
This PR is created in order to resolve issue and make the ends pixel perfect.

Checklist

  • I linked PR with a related issue
  • I added multiple screenshots/screencasts of my UI changes

Motivation and Context

Issue is the gap between the source and target ends.
Handles was creating gaps between nodes and paths.

This PR completes https://github.com/ls1intum/Apollon2/issues/xx

Description

I have updated handles and marker paddings, so that marker ends and start of the path would be directly connected to node.

Steps for Testing

  1. Change the edges and make sure there is no gap between the source and target end of the paths.
  2. Repeat this for every path.
  3. Create a path between two nodes, and make sure they are also connected without a gap.

Screenshots

image

@kurunbelemir kurunbelemir added the bug Something isn't working label Jan 27, 2025
@kurunbelemir kurunbelemir requested a review from egenerse January 27, 2025 11:59
@kurunbelemir kurunbelemir self-assigned this Jan 27, 2025
@kurunbelemir kurunbelemir linked an issue Jan 27, 2025 that may be closed by this pull request
@egenerse
Copy link
Collaborator

@kurunbelemir newly created edges doesn't follow this marker end rule

Screen.Recording.2025-01-27.at.15.38.07.mov

@kurunbelemir
Copy link
Collaborator Author

@kurunbelemir newly created edges doesn't follow this marker end rule

Screen.Recording.2025-01-27.at.15.38.07.mov

You are right, BiAssociation Edge wasnt updated earlier, now I fixed it.

Copy link
Collaborator

@egenerse egenerse left a comment

Choose a reason for hiding this comment

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

working well now 👍 nice

@kurunbelemir kurunbelemir merged commit 9573766 into main Jan 27, 2025
2 checks passed
@kurunbelemir kurunbelemir deleted the 84-hotfix-edge-markers-pixel-perfect branch January 27, 2025 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Hotfix Edge Markers Pixel Perfect
2 participants