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

Cannot open IDT with MOE sugar. #1966

Closed
NadezhdaPeskun opened this issue May 23, 2024 · 2 comments · Fixed by #1967
Closed

Cannot open IDT with MOE sugar. #1966

NadezhdaPeskun opened this issue May 23, 2024 · 2 comments · Fixed by #1967

Comments

@NadezhdaPeskun
Copy link
Collaborator

NadezhdaPeskun commented May 23, 2024

Steps to Reproduce

  1. Switch to Macro
  2. Open >> select Open from clipboard (see .mp4 file below)
  3. Insert >> /52MOErA//i2MOErC//32MOErT/
  4. Select IDT type >> Add to Canvas
    image

Actual behavior
Cannot open IDT with MOE sugar.

Expected behavior

  • User should be able to open from clipboard or from file the IDT with MOE sugar
    For example:
/2MOErC/ >> should be **Error** - and cannot be opened

/52MOErA/*/i2MOErC/*/32MOErT/ >> should be Save to IDT and Open from IDT successfully.
/52MOErA//i2MOErC//32MOErT/ >> should be Save to IDT and Open from IDT successfully.
/i2MOErC/*/32MOErT/ >> should be opened
/52MOErA/*/i2MOErC/ >> should be opened
/52MOErA/*/32MOErC/ >> should be opened
/52MOErA//32MOErC/ >> should be opened
/i2MOErC/ >> should be opened
/32MOErT/  >> should be opened
/52MOErA/  >> should be opened
/52MOErA/*/i2MOErC/*/i2MOErG/*G*+A*mT*A*T*rA*G*/i2MOErG/*/32MOErT/ 

image

Tasks:

Versions:
Windows 11
Chrome Version Version 123.0.6312.106 (Official Build) (64-bit)

CI version:
Ketcher Version 2.22.0-rc.1 Build at 2024-04-15; 20:00:15
Indigo Toolkit Version 1.21.0-dev4

@NadezhdaPeskun
Copy link
Collaborator Author

NadezhdaPeskun commented Jun 3, 2024

Verification on the CI
New bug is added

Opened as expected:

/32MOErA/
/52MOErA/*/32MOErC/
/52MOErA//32MOErC/
/52MOErA//i2MOErC//32MOErT/ 
/52MOErA/*/i2MOErC/*/32MOErT/ 
/52MOErA/*/i2MOErC/*/i2MOErG/*G*+A*mT*A*T*rA*G*/i2MOErG/*/32MOErT/ 

Errors:
Paste from Clipboard:

  1. /52MOErA/
image

2) /i2MOErA/
image

  1. /52MOErA//i2MOErA/ and for /52MOErA/*/i2MOErA/
image
  1. /i2MOErA/*/32MOErA/
image

@AlexeyGirin
Copy link
Collaborator

Verified.
Some not working cases moved to #1982

  • Indigo Toolkit Version 1.21.0-rc.1.0-g9194599b2-wasm32-wasm-clang-19.0.0
  • Ketcher Version 2.22.0-rc.2 Build at 2024-06-03; 18:43:00
  • Chrome Version 125.0.6422.142 (Official Build) (64-bit)
  • Windows 10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants