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

Playground context menu missing "create" item in develop #5975

Closed
cpcallen opened this issue Mar 3, 2022 · 2 comments
Closed

Playground context menu missing "create" item in develop #5975

cpcallen opened this issue Mar 3, 2022 · 2 comments
Labels
issue: bug Describes why the code or behaviour is wrong type: regression

Comments

@cpcallen
Copy link
Contributor

cpcallen commented Mar 3, 2022

Describe the bug

I was attempting the reproduce steps for issue #5972 in develop, but was unable to to find the "create 'do something'" item in the procedure_defreturn block's context menu. It is present in master.

To Reproduce

Steps to reproduce the behavior:

  1. Check out develop branch.
  2. npm start
  3. Open the "Functions" section of the toolbox.
  4. Drag a procedure_defreturn block (the one with a "return" input) to the workspace.
  5. Right-click (or appropriate gesture) to open the context menu for this newly created block.

Expected behaviour

The last item in the menu will be "create 'do something'"

Expected behaviour

The menu ends with "Help".

Screenshots

See #5972, step 2 for expected.

Actual:
Screenshot 2022-03-03 at 12 12 16

Desktop (please complete the following information):

  • OS: macOS 12.2.1
  • Browser: Chrome 99.0.4844.51
  • Version: develop, at 2a0e547

Additional context

I wondered if this have been caused by my changes to playground loading in PR #5931, but multi_playground.html has the same problem despite not being converted yet.

@cpcallen cpcallen added the issue: triage Issues awaiting triage by a Blockly team member label Mar 3, 2022
@cpcallen
Copy link
Contributor Author

cpcallen commented Mar 3, 2022

After a quick manual bisect, I see that the problem began when PR #5959 #5952 was merged. @BeksOmega: any idea what might have caused this?

@cpcallen cpcallen added the issue: bug Describes why the code or behaviour is wrong label Mar 3, 2022
@alschmiedt
Copy link
Contributor

Fixed in #5976

@maribethb maribethb removed the issue: triage Issues awaiting triage by a Blockly team member label Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue: bug Describes why the code or behaviour is wrong type: regression
Projects
None yet
Development

No branches or pull requests

3 participants