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

Clipboard Improvements #225

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

bjoernh2000
Copy link

Updated Clipboard to allow multiple clippings.
Further fixes needed:

  • Connections
  • Red Parts of certain components are not generating due to the manual generation of renders, will be implementing a fix that automates this process

@rkrishnasanka
Copy link
Collaborator

rkrishnasanka commented Jun 8, 2021

Things that need to get worked on:

  • Connections Copying capability
  • Ability to generate a render that indicates to the user that the copy tool is still active
  • Change how to clone the features, we need to consider what the relative placement needs to be on the level

@rkrishnasanka
Copy link
Collaborator

@bjoernh2000 let me know if you can work on this?

Comment on lines +115 to 118
let selection = reference.view.getSelectedFeatures();
if (selectedFeatures.length > 0) {
reference.pasteboard[0] = selectedFeatures[0];
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

It seems like the cut system also got modified during the variable change but its not using the latest selection system

Copy link
Author

Choose a reason for hiding this comment

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

Ok I can change that to also use the recent selection system

Comment on lines +132 to +141

// if (pasteboardFeatures.length == 1) { // 1 feature
// reference.updateDefaultsFromFeature(pasteboardFeatures[0]);
// reference.activateTool(pasteboardFeatures[0].getType());

// } else if (pasteboardFeatures.length > 1) { // multiple features
// console.log("multiple features detected")
// reference.updateDefaultsFromFeatures(pasteboardFeatures);
// reference.activateTools(pasteboardFeatures);
// }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Delete these if you are not planning to use them anymore

Copy link
Collaborator

@rkrishnasanka rkrishnasanka left a comment

Choose a reason for hiding this comment

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

The copying of connections needs to be completed along with some slight code modifications

@bjoernh2000 bjoernh2000 marked this pull request as draft July 2, 2021 15:46
@rkrishnasanka rkrishnasanka changed the base branch from master to webpack-build July 21, 2021 23:46
@rkrishnasanka rkrishnasanka changed the base branch from webpack-build to master July 21, 2021 23:46
@rkrishnasanka rkrishnasanka linked an issue Aug 11, 2021 that may be closed by this pull request
@rkrishnasanka rkrishnasanka marked this pull request as ready for review September 28, 2021 17:54
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.

Improve clipboard capability
2 participants