Skip to content

Kyle move selected spaces#406

Open
kylem038 wants to merge 11 commits intodevelopfrom
kyle-move-selected-spaces
Open

Kyle move selected spaces#406
kylem038 wants to merge 11 commits intodevelopfrom
kyle-move-selected-spaces

Conversation

@kylem038
Copy link
Copy Markdown
Collaborator

@kylem038 kylem038 commented Aug 25, 2021

This completed #376
We've added an icon to each space which will make a modal appear asking the user to adjust the space's upper-left X/Y to a new position. Hitting the Save button moves that space.
Generally happy with the implementation although I think I need to determine the upper-left corner more directly? Maybe I can follow how registerSelectEvents works?

Edit: I'm not sure registerSelectEvents directly translates. That uses the drag event to set an origin which everything works off of. Could use some help figuring out the best way to pick an origin for this dialog.

@kylem038 kylem038 self-assigned this Aug 25, 2021
@@ -79,7 +79,6 @@ export default function createFaceFromPoints(context, payload) {
// split edges where vertices touch them
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

No changes here. Just some clean up.

@kylem038 kylem038 requested a review from bgschiller August 25, 2021 18:44
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.

1 participant