-
-
Notifications
You must be signed in to change notification settings - Fork 185
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
template tool doesn't actually paste template that is selected #1655
Comments
Thanks for opening your first issue here! Please try to include example files and screenshots if possible. If you're looking for support, please post on our forum: https://discuss.avogadro.cc/ |
This seems like more of a documentation / UI hint thing. Click on a hydrogen atom. The template tool started with "build up metal complexes easily":
|
I agree with UI, that it needs to be more clear that this will attach a fragment to an existing atom, or replace an existing atom with a fragment. But in that case, I'd also argue that it should do nothing (or maybe give a popup error?) if you click on blank canvas space, or at least not paste a random molecule that seems to come out of nowhere (why FeH6?) Also, putting in a single H then clicking on it with the template tool (amide selected), resulted in a crash:
|
Putting in a CH4 and clicking on one of the hydrogens with the template tool worked nicely. |
Thanks for the single-H report. I'll hunt down that crash. I think the problem is that it's trying to find the atom the hydrogen is attached to. |
Makes sense. Thanks. |
I tested it a little myself and the reason that FeH6 appeared in your case is because that was what was currently selected under the "Centers" tab. Equally, if you change to e.g. nickel, then go back to the ligands or groups tab, then click empty space, a nickel atom will appear. I agree with @bernstei that from a UX perspective new metal centres shouldn't be created when the Ligands or Groups tabs are open, only when the Centers tab is the currently selected one. If Ligands or Groups are selected and one clicks on empty space rather than a H atom, nothing should appear at all. |
I've also put some changes in the docs to try and make it as clear as possible that the ligands and groups work by replacing existing H atoms and they can't be placed into empty space. |
Avogadro version: (please complete the following information from the About box):
Desktop version: (please complete the following information):
Describe the bug
template tool pastes FeH$_6$ no matter what template is selected in "Template" panel
To Reproduce
Expected behavior
The chosen template is pasted
Screenshots
Image of window after following directions in "To Reproduce"
The text was updated successfully, but these errors were encountered: