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

Show underlying structure on mouse hover for functional groups and salts&solvents #1824

Closed
AlexanderSavelyev opened this issue Nov 9, 2022 · 2 comments · Fixed by #1907
Closed
Assignees
Milestone

Comments

@AlexanderSavelyev
Copy link
Contributor

AlexanderSavelyev commented Nov 9, 2022

Is your feature request related to a problem? Please describe.
Right now user need to expand structure to understand what is expanded view of the functional group

Describe the solution you'd like
User mouse over a functional group (salts and solvents in future) and ketcher shows a structure for the selected functional group. The structure should be displayed as a rectangle below the pointer with white background

image

@AlexanderSavelyev AlexanderSavelyev added this to the Release 2.7.0 milestone Nov 9, 2022
@vanoprenko vanoprenko changed the title Show underlying structure if mouse-over for functional groups Show underlying structure on mouse hover for functional groups and salts&solvents Nov 16, 2022
@Nitvex Nitvex modified the milestones: Release 2.7.0, Release 2.8.0 Nov 24, 2022
@vanoprenko
Copy link
Collaborator

Please use current design for displaying a functional group on hover:

image

@KonstantinEpam23
Copy link
Collaborator

The hover effect should be delayed by 500ms

@KonstantinEpam23 KonstantinEpam23 linked a pull request Dec 12, 2022 that will close this issue
KonstantinEpam23 added a commit that referenced this issue Dec 12, 2022
KonstantinEpam23 added a commit that referenced this issue Dec 12, 2022
KonstantinEpam23 added a commit that referenced this issue Dec 12, 2022
* #1824: expand function groups on hover

* #1824: temporary fixes for sgroup hover

* #1824: temporary fixes for sgroup hover

* #1824: fix double dependency

* #1824: remove comments

Co-authored-by: Konstantin Levin <[email protected]>
@Nitvex Nitvex added the rc.1 label Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants