epam / epam/Indigo

Clean Up and Layout ignore atom hydrogens and indexes

Open
#1,671 0 comments 0 reactions 0 assignees View on GitHub
Priority: Low
Dominant language
C++
Stars
406
Forks
134
Avg merge
2d 11h
Merged PRs (30d)
24

Description

**Environment details (please complete the following information):**
- Ketcher Version 2.18.0-rc.4 Build at 2024-02-06; 13:45:28
- Indigo Toolkit Version 1.18.0-rc.6.0-g98bf384e0-x86_64-linux-gnu-11.2.1
- Chrome Version 121.0.6167.161 (Official Build) (64-bit)
- Win10
- Issue found while testing https://github.com/epam/ketcher/issues/3934

**Steps to Reproduce**
1. Create reaction as shown on the screen
![image](https://github.com/epam/ketcher/assets/26869421/24b07508-3699-4d03-a876-bf67ad95cf00)
OR
1. Open from file as a New Project: [Clean Up causes internal sever error if atom(s) with mapping tool selected.zip](https://github.com/epam/Indigo/files/14206581/Clean.Up.causes.internal.sever.error.if.atom.s.with.mapping.tool.selected.zip)
2. Spread atoms as on the picture (it is needed because atoms position are not restored correct because of the bug)
![image](https://github.com/epam/Indigo/assets/26869421/357a86b0-4891-4d61-a477-eaa74bd0aaae)
3. Remove selection from atoms if any (it required workaround for issues https://github.com/epam/Indigo/issues/1670)
4. Press Ctrl+L (Layout)
OR
4. Press Ctrl+Shift+L (Clean Up)

**Actual behavior**
![image](https://github.com/epam/Indigo/assets/26869421/af009a78-199d-4cc2-ae0f-29410103571e)
Layout: System shrinks arrow length, put plus and arrow over H atom index and O atom hydrogens
Clean Up: Works slightly different but problems are the same

**Expected behavior**
System takes into account indexes and hyrdogens and adds extra space between atom, plus and arrow in the same manner as it does for big molecules
![image](https://github.com/epam/Indigo/assets/26869421/b38de859-1d6f-4c4f-96a0-108d1d9a882d)
Size of arrow remains the same

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.