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

Atom selection error #49

Closed
tomyan112 opened this issue Mar 7, 2020 · 7 comments
Closed

Atom selection error #49

tomyan112 opened this issue Mar 7, 2020 · 7 comments
Assignees
Labels
Bug Something isn't working Can't Reproduce We can't reproduce this bug

Comments

@tomyan112
Copy link

https://youtu.be/UneB0SY77C8

@MikeWilliams-UK MikeWilliams-UK added the Bug Something isn't working label Mar 7, 2020
@MikeWilliams-UK
Copy link
Contributor

@deadlyvices looking at the video produced by @tomyan112 it may be that ACME is inserting the selected atom instead of the one indicated by the atom dropdown.

@MikeWilliams-UK MikeWilliams-UK added the Can't Reproduce We can't reproduce this bug label Mar 10, 2020
@tomyan112
Copy link
Author

I am able to reproduce, a more detailed version is provided for you, https://youtu.be/vw_CSinjnVQ

@MikeWilliams-UK
Copy link
Contributor

@deadlyvices from @tomyan112 's latest video it looks like the glitch is in the Undo / Redo system.

@tomyan112
Copy link
Author

@deadlyvices from @tomyan112 's latest video it looks like the glitch is in the Undo / Redo system.

Also happen after using atomic properties
https://www.youtube.com/watch?v=k5AKym1Oy88

@MikeWilliams-UK
Copy link
Contributor

@tomyan112 this is by design.
After using the property dialogue the atom is selected.
Thus when the atom selection drop down is changed (all) selected atoms are changed to the selected value.

@deadlyvices maybe we should not automatically select the atom or bond after using the dialogues?

@deadlyvices
Copy link
Contributor

I'm not sure about that. We select the atom or bond to show the user the object of the last undo operation. Otherwise it's easy to miss the reversion.

@MikeWilliams-UK
Copy link
Contributor

Closing as no input from original poster since April 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Can't Reproduce We can't reproduce this bug
Development

No branches or pull requests

3 participants