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.

Sign up for GitHub

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

Jump to bottom

Macro: Monomers are positioned below bonds that connect them #3981

Closed
Zhirnoff opened this issue Jan 29, 2024 · 1 comment · Fixed by #4038 or #4041
Closed

Macro: Monomers are positioned below bonds that connect them #3981

Zhirnoff opened this issue Jan 29, 2024 · 1 comment · Fixed by #4038 or #4041
Assignees
@ilya-asiyuk-epam
Labels
bug Firefox only Macromolecules
Milestone
Ketcher 2.18.0-rc.5

Comments

@Zhirnoff
Copy link
Collaborator

Steps to Reproduce

  1. Open Ketcher and switch to Macro mode
  2. Open any attached file
    rna-mod-phosphate-example.zip
    dna.zip

Actual behavior

The monomers are positioned below bonds that connect them.

Expected behavior

The bonds connect centers of the monomers.

Screenshots

2024-01-26_22h20_59
2024-01-26_22h21_41
2024-01-26_22h22_13
2024-01-26_22h22_29
2024-01-26_22h22_58
2024-01-26_22h23_08
2024-01-26_22h25_54

Desktop (please complete the following information):

  • OS: Windows 11
  • Browser Firefox
  • Version 122.0 (64-bit)

Ketcher version
Version 2.18.0-rc.2
@Zhirnoff Zhirnoff added this to the Ketcher 2.19.0-rc.1 milestone Jan 29, 2024
ilya-asiyuk-epam added a commit that referenced this issue Feb 6, 2024
@ilya-asiyuk-epam
#3981 - Macro: Monomers are positioned below bonds that connect them
4cde869
@ilya-asiyuk-epam ilya-asiyuk-epam linked a pull request Feb 6, 2024 that will close this issue
#3981 - Macro: Monomers are positioned below bonds that connect them #4038
Merged
9 tasks
ilya-asiyuk-epam added a commit that referenced this issue Feb 6, 2024
@ilya-asiyuk-epam
#3981 - Macro: Monomers are positioned below bonds that connect them
328cc2a
ilya-asiyuk-epam added a commit that referenced this issue Feb 6, 2024
@ilya-asiyuk-epam
#3981 - Macro: Monomers are positioned below bonds that connect them
17d36f9
ilya-asiyuk-epam added a commit that referenced this issue Feb 6, 2024
@ilya-asiyuk-epam
#3981 - Macro: Monomers are positioned below bonds that connect them
d9856d4
ilya-asiyuk-epam added a commit that referenced this issue Feb 7, 2024
@ilya-asiyuk-epam
Merge pull request #4038 from epam/3981-macro-monomers-are-positioned…
a733d0f 
…-below-bonds-that-connect-them

#3981 - Macro: Monomers are positioned below bonds that connect them
ilya-asiyuk-epam added a commit that referenced this issue Feb 7, 2024
@ilya-asiyuk-epam
Backmerge: #3981 - Macro: Monomers are positioned below bonds that co…
6054a9e 
…nnect them
@ilya-asiyuk-epam ilya-asiyuk-epam linked a pull request Feb 7, 2024 that will close this issue
Backmerge: #3981 - Macro: Monomers are positioned below bonds that connect them #4041
Merged
9 tasks
ilya-asiyuk-epam added a commit that referenced this issue Feb 7, 2024
@ilya-asiyuk-epam
Backmerge: #3981 - Macro: Monomers are positioned below bonds that co…
87596e4 
…nnect them
ilya-asiyuk-epam added a commit that referenced this issue Feb 7, 2024
@ilya-asiyuk-epam
Merge pull request #4041 from epam/3981-macro-monomers-are-positioned…
227d039 
…-below-bonds-that-connect-them

Backmerge: #3981 - Macro: Monomers are positioned below bonds that connect them
@Zhirnoff
Copy link
Collaborator Author

Tested. Bug is fixed.
2024-02-12_15h20_00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ilya-asiyuk-epam ilya-asiyuk-epam

Labels
bug Firefox only Macromolecules
Projects
None yet
Milestone
Ketcher 2.18.0-rc.5
3 participants
@rrodionov91 @Zhirnoff @ilya-asiyuk-epam