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

Not able to access molecules properties with [CH₄] #12

Open
tomyan112 opened this issue Mar 31, 2020 · 12 comments
Open

Not able to access molecules properties with [CH₄] #12

tomyan112 opened this issue Mar 31, 2020 · 12 comments
Assignees
Labels
By Design This is working as designed

Comments

@tomyan112
Copy link

tomyan112 commented Mar 31, 2020

First draw this structure
image
then delete one of the CH₃
image
I notice there is no count n option in molecules properties

<cml:cml xmlns:conventions="http://www.xml-cml.org/convention/" xmlns:cml="http://www.xml-cml.org/schema" xmlns:cmlDict="http://www.xml-cml.org/dictionary/cml/" xmlns:nameDict="http://www.xml-cml.org/dictionary/cml/name/" xmlns:c4w="http://www.chem4word.com/cml" conventions="convention:molecular">
  <cml:molecule id="m1" c4w:showBrackets="true">
    <cml:atomArray>
      <cml:atom id="a1" elementType="C" x2="819" y2="649.5" />
    </cml:atomArray>
  </cml:molecule>
</cml:cml>

Edited to format as xml and remove non esental data

@MikeWilliams-UK
Copy link
Contributor

@tomyan112 This is because a single atom molecule does not (by design) have access to the molecule properties dialogue.
@deadlyvices Can you think of any case where a user would want brackets or count on a single atom molecule.

@MikeWilliams-UK
Copy link
Contributor

@deadlyvices When a molecule with multiple atoms that has brackets etc set is demoted to a single atom molecule I think we should remove the brackets etc to make it consistent with what can be drawn starting from a single atom molecule.

@tomyan112
Copy link
Author

tomyan112 commented Apr 2, 2020

@deadlyvices When a molecule with multiple atoms that has brackets etc set is demoted to a single atom molecule I think we should remove the brackets etc to make it consistent with what can be drawn starting from a single atom molecule.

Radical cation formed by electrospray ionization.

@MikeWilliams-UK
Copy link
Contributor

@tomyan112 sorry but I am not a chemist. Can you draw a picture of how you think a single atom molecule should look like for "Radical cation formed by electrospray ionization."

@tomyan112
Copy link
Author

tomyan112 commented Apr 3, 2020

@tomyan112 sorry but I am not a chemist. Can you draw a picture of how you think a single atom molecule should look like for "Radical cation formed by electrospray ionization."

image

Other issues

  1. bracket cant be set in the middle of molecule
  2. The count of molecule cannot set to be n
    image

@MikeWilliams-UK
Copy link
Contributor

@tomyan112 all of the above "features" are by design.

@deadlyvices how many of the above should we be implementing in the future?

@deadlyvices
Copy link
Contributor

Polymer brackets we probably should. Not so sure about bracketing a single atom molecule though

@tomyan112
Copy link
Author

Polymer brackets we probably should. Not so sure about bracketing a single atom molecule though

here is a slide for your reference
image

@MikeWilliams-UK
Copy link
Contributor

@deadlyvices which bit of the above diagrams is polymer brackets

@tomyan112
Copy link
Author

which bit of the above diagrams is polymer brackets

When the brackets are drawn through the middle of the bonds
image

@MikeWilliams-UK
Copy link
Contributor

Thanks @tomyan112

@MikeWilliams-UK
Copy link
Contributor

MikeWilliams-UK commented Apr 6, 2020

@deadlyvices for the scenario below
image

We could introduce a new button with the icon shown below which would be activated when a single atom molecule is selected, then when clicked the molecule properties dialogue will be shown. Thus allowing the diagram shown above to be created.

The key thing here is to the user to show the radical dot(s)

This may be a suitable icon for the button
image

@MikeWilliams-UK MikeWilliams-UK transferred this issue from Chem4Word/Version3-1 Feb 28, 2022
@MikeWilliams-UK MikeWilliams-UK transferred this issue from Chem4Word/Version3-2 Oct 8, 2024
@MikeWilliams-UK MikeWilliams-UK added the By Design This is working as designed label Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
By Design This is working as designed
Development

No branches or pull requests

3 participants