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

Incorrect brackets rendering with S-Group and Functional Group #899

Closed
vitalgerko opened this issue Oct 29, 2021 · 3 comments
Closed

Incorrect brackets rendering with S-Group and Functional Group #899

vitalgerko opened this issue Oct 29, 2021 · 3 comments
Assignees
@AndreiMazol
Labels
bug Priority: Low
Milestone
Ketcher 2.9.0-rc.1

Comments

@vitalgerko
Copy link

Steps to Reproduce

  1. Launch Ketcher with the following display settings (settings: 'Atoms', 'Show hydrogen labels' - on).
  2. Create chain structure.
  3. Create S-Group using previously added chain.
  4. Add on canvas FG (C2H5 for example) and Expanded FG abbreviation.

Expected behavior
All structures are inside the brackets.

Actual behavior

Incorrect brackets rendering with S-Group and Functional Group

Desktop:
OS: Windows 10
Browser: Chrome Version 93.0.4577.63, Firefox Version 92.0, Microsoft Edge Version 93.0.961.47
@r1z3rISGOD r1z3rISGOD assigned r1z3rISGOD and unassigned AndreiMazol Nov 11, 2021
@AndreiMazol AndreiMazol added this to the Release 2.5.0 milestone Nov 11, 2021
@r1z3rISGOD r1z3rISGOD assigned AndreiMazol and unassigned r1z3rISGOD Dec 1, 2021
@AndreiMazol AndreiMazol modified the milestones: Release 2.5.0, Backlog Apr 5, 2022
@ansivgit ansivgit assigned ansivgit and unassigned AndreiMazol May 12, 2022
@ansivgit
Copy link
Collaborator

01 02

As you can see in the pictures, the position of the brackets does not depend on the structures and their width, but on the svg elements that highlight the structures

@ansivgit ansivgit assigned AndreiMazol and unassigned ansivgit May 13, 2022
@KonstantinEpam23 KonstantinEpam23 mentioned this issue Jan 26, 2023
Merged
@KonstantinEpam23
Copy link
Collaborator

This should be resolved by: #2125

@KonstantinEpam23
Copy link
Collaborator

Closing as a duplicate of #2033

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

@AndreiMazol AndreiMazol

Labels
bug Priority: Low
Projects
None yet
Milestone
Ketcher 2.9.0-rc.1
Development

No branches or pull requests
5 participants
@ansivgit @AndreiMazol @r1z3rISGOD @vitalgerko @KonstantinEpam23