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

Stereomarks and stereoflag are not displayed for templates with stereocenters #1258

Closed
Svetlana-Asinovskaia opened this issue Feb 15, 2022 · 0 comments · Fixed by #1265
Closed
Assignees
Labels
Milestone

Comments

@Svetlana-Asinovskaia
Copy link
Collaborator

Svetlana-Asinovskaia commented Feb 15, 2022

Steps to Reproduce

  1. Select any template with stereobonds from the Template library.
  2. In the structure select any stereocenter.
  3. Click the 'Stereochemistry button'.
  4. Select the 'Create new AND group'/'Create new OR Group' option.
  5. Close the 'Enhanced Stereochemistry' dialog.

Expected behavior
A stereomark '&1'/'or1' is displayed near the selected stereocenter.
'Mixed' stereoflag is shown near the structure.

Actual behavior
A stereomark and stereocenter are not displayed.

Screenshots
2022-02-15_22h17_14
2022-02-15_22h21_06

Desktop:

  • OS: Windows 10
  • Browser: Chrome, Version 98.0.4758.82

Ketcher version: v2.4.1-99-g81423737

@Svetlana-Asinovskaia Svetlana-Asinovskaia added this to the Release 2.4.2 milestone Feb 15, 2022
@Svetlana-Asinovskaia Svetlana-Asinovskaia removed this from the Release 2.4.2 milestone Feb 15, 2022
@AndreiMazol AndreiMazol added this to the Release 2.5.0 milestone Feb 16, 2022
@karen-sarkisyan karen-sarkisyan linked a pull request Feb 17, 2022 that will close this issue
AndreiMazol pushed a commit that referenced this issue Feb 18, 2022
* Mark fragments before rendering template

* Refactor StructRender to TypeScript
@akolyakov42 akolyakov42 reopened this Feb 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants