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

Issue with Box and JA's "Hako" #2095

Closed
pikurasa opened this issue Feb 14, 2020 · 7 comments
Closed

Issue with Box and JA's "Hako" #2095

pikurasa opened this issue Feb 14, 2020 · 7 comments
Labels
Issue-Regression Something that worked in the past is now broken Issue-Translation Issues related to translation WF6-Needs testing Please test

Comments

@pikurasa
Copy link
Collaborator

There may be a somewhat major conflict with "box" (English project) loading in Japanese.

Screenshot at 2020-02-14 17:21:24 box and hako conflict

Test file:
Tonal Pitch Paint box and JA hako conflict.html.zip

Needs a bit more investigation, but is causing me problems at the moment.

@walterbender
Copy link
Member

Can you please explain the steps you took to create this project?

@walterbender walterbender added Issue-Regression Something that worked in the past is now broken Issue-Translation Issues related to translation labels Feb 15, 2020
@pikurasa
Copy link
Collaborator Author

  1. Start MB
  2. Change language to Japanese (tested in Kanji)
  3. Open a project that uses "box" (tested "Pitch Paint")
  4. "Box" blocks for project will still be in English, but "box" blocks in palette will be in Japanese.

At the moment, I am not experiencing the problematic behavior of not being able to copy any of the box blocks (in either language), which was making it impossible to find a workaround.

walterbender added a commit that referenced this issue Feb 16, 2020
@walterbender walterbender added the WF6-Needs testing Please test label Feb 16, 2020
walterbender added a commit that referenced this issue Feb 16, 2020
@pikurasa
Copy link
Collaborator Author

pikurasa commented May 8, 2020

Most aspects of this issue have been resolved.

I tested with https://github.com/sugarlabs/musicblocks/files/4207147/Tonal.Pitch.Paint.box.and.JA.hako.conflict.html.zip

I did get a box with name "hako1" (see block at the right) when I opened the file, which is not recognized by MB. However, all the other blocks were called "hakonosuuchi1" (see box blocks at left that are attached) and they worked (I replaced the "hako1" block by hand).

Screenshot at 2020-05-08 12:58:08 hako

I think the main issue has been resolved, but it may be wise to investigate why this shorter named (perhaps older?) block is being generated at all. I would hate for students in Japan to open some of these projects and they just do not work for them.

@walterbender
Copy link
Member

I will investigate. But in the meantime, I found another bug.
Screenshot from 2020-05-09 18-04-32

It seems that number to pitch fails with float values. It works with integers.

@walterbender
Copy link
Member

pushed a fix for the number to pitch bug :)

@walterbender
Copy link
Member

I have looked at the project and cannot figure out how it was created. Most of the named blocks are stored internal as:
[15,["namedbox",{"value":"box1"}]

But one block is stored as:
[67,["namedbox",{"value":"箱1"}]

We translate the box name in the interface but the internal value is in English -- a special case for box1, box2, and box. I don't know how we can determine the intention of "箱1" when we encounter it other than to assume it is a different box.

@pikurasa
Copy link
Collaborator Author

I just remember switching to Japanese and reloading this project, but cannot remember the details. I do not think I explicitly named any box blocks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Regression Something that worked in the past is now broken Issue-Translation Issues related to translation WF6-Needs testing Please test
Projects
None yet
Development

No branches or pull requests

2 participants