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

Refactor the plugin to use the lexical variable mixin from App Inventor #45

Open
6 tasks
mark-friedman opened this issue Jan 31, 2024 · 1 comment
Open
6 tasks
Labels
enhancement New feature or request lexical variables Use this tag for the lexical variable plugin

Comments

@mark-friedman
Copy link
Collaborator

mark-friedman commented Jan 31, 2024

  • Remove the mixin defined methods from each block.
  • Add the methods required by the mixin to each block.
  • Mix the mixin into each block. Test the result.
  • For each block.foo(...) method defined in the mixin create a new foo(block, ...) function.
  • Change the code in field_lexical_variable.js (and any other code) that calls a block.foo(...) method defined in the mixin to call the new foo(block, ...) function instead. Test the result.
  • Remove the mixin
@mark-friedman mark-friedman added enhancement New feature or request lexical variables Use this tag for the lexical variable plugin labels Jan 31, 2024
@mark-friedman
Copy link
Collaborator Author

Note that once this is done, we can then do following to App Inventor (similar to the above):

  • Remove the mixin defined methods from each block.
  • Add the methods required by the mixin to each block.
  • Remove the mixin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lexical variables Use this tag for the lexical variable plugin
Projects
None yet
Development

No branches or pull requests

1 participant