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

Fix for render problem. #9

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

kurunbelemir
Copy link

@kurunbelemir kurunbelemir commented Oct 15, 2024

When class diagram element move somewhere else translation differs for attribute and method texts. This fix solves the problem

Previous version
Ekran Resmi 2024-10-16 - 00 01 43

After fix
Simulator Screenshot - iPad Air 13-inch (M2) - 2024-10-16 at 00 02 40

  • Attributes and methods stored in different lists instead of putting them together under children list.
  • Element creation and rendering methods adjusted according to structure change.

…e translation differs for attribute and method texts. This fix solves the problem
@kurunbelemir kurunbelemir linked an issue Oct 20, 2024 that may be closed by this pull request
Copy link

@FelixTJDietrich FelixTJDietrich left a comment

Choose a reason for hiding this comment

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

The initial issue is broken only on iOS 18. Your fix did not fix the issue for resizing the UML element. The resizing is now also broken on iOS 17

image

Sources/ApollonEdit/ViewModels/ApollonEditViewModel.swift Outdated Show resolved Hide resolved
Sources/ApollonShared/DataModels/UMLElement.swift Outdated Show resolved Hide resolved
Copy link
Member

@anian03 anian03 left a 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 :)

@kurunbelemir kurunbelemir linked an issue Oct 27, 2024 that may be closed by this pull request
fix for interface, abstract class type drawing issue
Copy link

@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.

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.

Copy link

@FelixTJDietrich FelixTJDietrich left a comment

Choose a reason for hiding this comment

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

The line does not reach the border on iOS 17
image

On 13inch iPad Pro iOS 18 it looks like this
image

Copy link

@FelixTJDietrich FelixTJDietrich left a 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

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?

Copy link
Author

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.

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

Choose a reason for hiding this comment

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

Same here?

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.

Resize Drawing Bug Wrong translation of attribute and methods in class diagram
4 participants