-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix for render problem. #9
base: main
Are you sure you want to change the base?
Conversation
…e translation differs for attribute and method texts. This fix solves the problem
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks pretty good in general, just one suggestion from me, and please check the comments from Felix :)
fix for interface, abstract class type drawing issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested with @kurunbelemir 's iPad. Resize and moving are working as expected. I was able to resize elements and move them on canvas and they were moving. These bugs are fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me, just remove those lines and we will merge it
@@ -117,6 +117,43 @@ public class UMLElement: Codable, SelectableUMLItem { | |||
self.children?.append(child) | |||
} | |||
} | |||
//// NEWLY ADDED |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this? Is this from cursor?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No it was a note to myself, I forgot to delete.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay let me k is once you have deleted those comments
self.bounds?.height = newHeight | ||
self.attributes?.removeAll { $0.id == attribute.id } | ||
} | ||
/////END OF NEWLY ADDED |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here?
When class diagram element move somewhere else translation differs for attribute and method texts. This fix solves the problem
Previous version
After fix