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

Macro: Cursors should be copied from small mode #4692

Closed
ilya-asiyuk-epam opened this issue May 27, 2024 · 1 comment · Fixed by #4696
Closed

Macro: Cursors should be copied from small mode #4692

ilya-asiyuk-epam opened this issue May 27, 2024 · 1 comment · Fixed by #4696
Assignees
@ilya-asiyuk-epam
Labels
feature request
Milestone
Ketcher 2.22.0-rc.2

Comments

@ilya-asiyuk-epam
Copy link
Collaborator

ilya-asiyuk-epam commented May 27, 2024 •
edited
Loading

Background
Cursors in Micro and Macro are different.

Solution
Cursors should we unified.
Namely:

  1. Replace 'crosshair' cursor to 'default' for the whole canvas
  2. Set 'move' cursor for structures if they can be moved.
  3. Use 'default' cursor for structures if they cannot be moved
@ilya-asiyuk-epam ilya-asiyuk-epam added this to the Ketcher 2.22.0-rc.1 milestone May 27, 2024
@ilya-asiyuk-epam ilya-asiyuk-epam self-assigned this May 27, 2024
ilya-asiyuk-epam added a commit that referenced this issue May 27, 2024
@ilya-asiyuk-epam
#4692 - Macro: Cursors should be copied from small mode
66d6d0b
@ilya-asiyuk-epam ilya-asiyuk-epam linked a pull request May 27, 2024 that will close this issue
#4692 - Macro: Cursors should be copied from small mode #4696
Merged
9 tasks
ilya-asiyuk-epam added a commit that referenced this issue May 28, 2024
@ilya-asiyuk-epam
Merge pull request #4696 from epam/4692-macro-cursors-should-be-copie…
af5d024 
…d-from-small-mode

#4692 - Macro: Cursors should be copied from small mode
This was referenced May 30, 2024
Open
Open
@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Jun 4, 2024

Tested. We have two open bugs.
Desktop:

  • OS: Windows 11
  • Browser Chrome
  • Version 124.0.6367.79 (Official Build) (64-bit)

Ketcher version
[Version 2.22.0-rc.2]

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

@ilya-asiyuk-epam ilya-asiyuk-epam

Labels
feature request
Projects
Ketcher
Status: Done
Milestone
Ketcher 2.22.0-rc.2
Development

Successfully merging a pull request may close this issue.

#4692 - Macro: Cursors should be copied from small mode epam/ketcher
2 participants
@Zhirnoff @ilya-asiyuk-epam