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

Comment from 2.0 project getting imported attached to the wrong block #1705

Open
LiFaytheGoblin opened this issue Aug 29, 2018 · 1 comment

Comments

@LiFaytheGoblin
Copy link

Expected Behavior

Comments in 2.0 should be there in 3.0

Actual Behavior

Sometimes they aren't
screen shot 2018-08-29 at 4 20 45 pm
screen shot 2018-08-29 at 4 20 36 pm
(Some comments are tho!)

Steps to Reproduce

Can't reproduce. I tried creating comments, moving them to other blocks with and without deleting the original block, moving them to the scripts area and attaching it to a block again. Comments are always still there.

Found in
212388708

Operating System and Browser

MacOS, Chrome

@kchadha
Copy link
Contributor

kchadha commented Sep 24, 2018

In that particular project, it looks like the comment in question is getting imported, but it's getting added to the wrong block (it's on a block in the middle of a custom procedure definition stack) on the same sprite:

Scroll far off to the workspace of the UserScripts sprite and you will see:
image

The comment on that if then else block should be on the set MapsTotal block in the screenshots in the issue description above.

-- Needs more investigation to figure out what's actually going on here.

@kchadha kchadha changed the title Comment mysteriousely disappearing Comment from 2.0 project getting imported attached to the wrong block Sep 24, 2018
@kchadha kchadha added the bug label Sep 24, 2018
@thisandagain thisandagain added this to the Backlog milestone Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants