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

Allow comments to be associated with point entities #1035

Merged
merged 1 commit into from May 12, 2021

Conversation

ruevs
Copy link
Member

@ruevs ruevs commented May 11, 2021

If a single point is selected when a "Constrain | Comment" (;) is added
then the comment is associated with the point and its position becomes
relative to the point. In this way the comment will move with the point.

If nothing is selected or more than a single point is selected then
the behaviour is as before and the comment is "floating".

Closes #1032

Alternative to #1034

If a single point is selected when a "Constrain | Comment" (`;`) is added
then the comment is associated with the point and its position becomes
relative to the point. In this way the comment will move with the point.

If nothing is selected or more than a single point is selected then
the behaviour is as before and the comment is "floating".

Closes solvespace#1032
@phkahler
Copy link
Member

@ruevs This is strange. I tried adding a new comment to a point and it just highlighted the existing comment. Is that intended? We want to associate the point with each comment created, not a specific comment to the point.

@ruevs
Copy link
Member Author

ruevs commented May 19, 2021

Hmmm. Yes it does... Until you modify the comment from the default. After that it would add a new one. I have to search where is the logic that does this, but I kind of like it.

@phkahler
Copy link
Member

Oh that's actually really nice!

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.

The comments can't move with the shapes Request option for comments to be constrained/tied to other objects
2 participants