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

Mark all modified monomers on canvas (flex and snake modes) #5630

Closed
ljubica-milovic opened this issue Oct 3, 2024 · 1 comment · Fixed by #5980
Closed

Mark all modified monomers on canvas (flex and snake modes) #5630

ljubica-milovic opened this issue Oct 3, 2024 · 1 comment · Fixed by #5980
Assignees
Labels
Test Cases Written Test cases has been written for that issue Usability UX improvement

Comments

@ljubica-milovic
Copy link
Collaborator

ljubica-milovic commented Oct 3, 2024

Background
Especially on a crowded canvas, modified monomers are not that easy to find. Introducing marked monomers on canvas can more easily draw the attention of the user to them.

Requirement

  • Every modified amino acid, sugar, base and phosphate on canvas (flex and snake modes) should be marked as modified (see mockup).

Only amino acids, sugars, phosphates and bases can be considered modified. That means that there is no need for marking CHEMs, (unsplit) nucleotides, and unresolved nucleotides.
The list of non-modified amino acids can be found here: #5629, and the list of non-modified bases, sugars and phosphates here: #3734
Ambiguous monomers (amino acids/bases) that we have in the library are to be considered non-modified. All other ambiguous monomers should be considered modified.

UX
image

image

@AlexeyGirin
Copy link
Collaborator

Verified.
image

  • Ketcher Version 2.28.0-rc.1 Build at 2024-12-03; 17:04:52
  • Indigo Version 1.27.0-rc.1.0-g8d656a341-wasm32-wasm-clang-19.0.0
  • Chrome Version 131.0.6778.109 (Official Build) (64-bit)
  • Win10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Test Cases Written Test cases has been written for that issue Usability UX improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants