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

Cursor position misalignment after switching in fullscreen after changing ketcher settings #4097

Closed
rrodionov91 opened this issue Feb 16, 2024 · 0 comments · Fixed by #4098 or #4104
Closed

Cursor position misalignment after switching in fullscreen after changing ketcher settings #4097

rrodionov91 opened this issue Feb 16, 2024 · 0 comments · Fixed by #4098 or #4104
Assignees
@rrodionov91
Labels
bug
Milestone
Ketcher 2.18.0-rc.7

Comments

@rrodionov91
Copy link
Collaborator

Steps to Reproduce

  1. Apply git patch to wrap ketcher in dialog and set settings after ketcher initialization
    ketcher_wrapped_in_dialog.patch
  2. Run ketcher in dev mode (npm run dev:standalone in ./example folder)
  3. Add any structure on canvas
  4. Open fullscreen
  5. Hover on structure

Actual behavior
Hover does not appear. Rectangle selection appears with offset from cursor.

Expected behavior
Hover and selection works under cursor.
@rrodionov91 rrodionov91 added this to the Ketcher 2.18.0-rc.7 milestone Feb 16, 2024
rrodionov91 added a commit that referenced this issue Feb 16, 2024
@rrodionov91
#4097 - Cursor position misalignment after switching in fullscreen af…
9fab3a6 
…ter changing ketcher settings

- fixed canvas resize observer
@rrodionov91 rrodionov91 linked a pull request Feb 16, 2024 that will close this issue
#4097 - Cursor position misalignment after switching in fullscreen after changing ketcher settings #4098
Merged
9 tasks
rrodionov91 added a commit that referenced this issue Feb 16, 2024
@rrodionov91
#4097 - Cursor position misalignment after switching in fullscreen af…
5b84bd5 
…ter changing ketcher settings (#4098)

- fixed canvas resize observer

Co-authored-by: Roman Rodionov <roman_rodionov@epam.com>
@rrodionov91 rrodionov91 assigned rrodionov91 and unassigned Nitvex Feb 16, 2024
rrodionov91 added a commit that referenced this issue Feb 19, 2024
@rrodionov91
#4097 - Cursor position misalignment after switching in fullscreen af…
3e7647e 
…ter changing ketcher settings

- fixed canvas resize observer
rrodionov91 added a commit that referenced this issue Feb 19, 2024
@rrodionov91
#4097 - Cursor position misalignment after switching in fullscreen af…
2f510bf 
…ter changing ketcher settings (#4104)

- fixed canvas resize observer

Co-authored-by: Roman Rodionov <roman_rodionov@epam.com>
@rrodionov91 rrodionov91 linked a pull request Feb 19, 2024 that will close this issue
Backmerge: #4097 - Cursor position misalignment after switching in fullscreen after changing ketcher settings #4104
Merged
9 tasks
@github-project-automation github-project-automation bot moved this to Done in Ketcher Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rrodionov91 rrodionov91

Labels
bug
Projects
Ketcher
Status: Done
Milestone
Ketcher 2.18.0-rc.7
2 participants
@rrodionov91 @Nitvex