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

S-Group: Hover over the S-Group displays always acetaldehyde #1999

Closed
Zhirnoff opened this issue Dec 26, 2022 · 0 comments · Fixed by #2037
Closed

S-Group: Hover over the S-Group displays always acetaldehyde #1999

Zhirnoff opened this issue Dec 26, 2022 · 0 comments · Fixed by #2037
Assignees
@porcelain11
Labels
bug
Milestone
Release Candidate...

Comments

@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Dec 26, 2022 •
edited
Loading

Steps to Reproduce

  1. Launch Ketcher
  2. Draw a structure
  3. Add an atom data S-Group
  4. Add a label and value
  5. Press 'Apply'
  6. Hover over the S-Group

Actual behavior
Hover over the S-Group displays always acetaldehyde

Expected behavior
Hover over the S-Group displays a preview related to the represented atom

Screenshots
Screenshot 2023-01-25 at 16 12 08

2022-12-26_14h31_52.mp4

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser Chrome
  • Version 108.0.5359.125 (Official Build) (64-bit)

Ketcher version
Version 2.7.0-rc.6
Build at 2022-12-23; 13:33:39
@Zhirnoff Zhirnoff added the bug label Dec 26, 2022
@Zhirnoff Zhirnoff added this to the Release 2.8.0 milestone Dec 26, 2022
porcelain11 added a commit that referenced this issue Dec 29, 2022
@porcelain11
#1999 Update getFunctionalGroupByName (check if searchName is empty s…
061d5c0 
…tring)
porcelain11 added a commit that referenced this issue Dec 29, 2022
@porcelain11
#1999 Update InfoPanel useEffect (initialize timer variable inside us…
aef7831 
…eEffect)
porcelain11 added a commit that referenced this issue Dec 29, 2022
@porcelain11
#1999 Render name=value pair for DATA sGroup on hover
52007dc
@porcelain11 porcelain11 linked a pull request Dec 29, 2022 that will close this issue
#1999 S-group: hover over the s group displays always acetaldehyde #2037
Merged
porcelain11 added a commit that referenced this issue Dec 29, 2022
@porcelain11
#1999 Small fix on comments (update checks inside useEffect)
2ef3855
porcelain11 added a commit that referenced this issue Dec 30, 2022
@porcelain11
Merge pull request #2037 from epam/#1999-s-group-hover-over-the-s-gro…
8c048c2 
…up-displays-always-acetaldehyde

#1999 S-group: hover over the s group displays always acetaldehyde
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@porcelain11 porcelain11

Labels
bug
Projects
None yet
Milestone
Release Candidate 2.8.0-rc.1
Development

Successfully merging a pull request may close this issue.

#1999 S-group: hover over the s group displays always acetaldehyde epam/ketcher
2 participants
@Zhirnoff @porcelain11