Skip to content

Commit

Permalink
add acceptText for menus
Browse files Browse the repository at this point in the history
  • Loading branch information
LilyMakesThings authored Jan 31, 2024
1 parent ec1f7de commit 7573aa1
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/engine/runtime.js
Original file line number Diff line number Diff line change
Expand Up @@ -1225,8 +1225,8 @@ class Runtime extends EventEmitter {
outputShape: menuInfo.acceptReporters ?
ScratchBlocksConstants.OUTPUT_SHAPE_ROUND : ScratchBlocksConstants.OUTPUT_SHAPE_SQUARE,
args0: [
{
type: 'field_dropdown',
{ // to do: we could reimplement field_numberdropdown here really easily

Check failure on line 1228 in src/engine/runtime.js

View workflow job for this annotation

GitHub Actions / build

Multiple spaces found before '// to do: we c...'
type: menuInfo.acceptText ? 'field_textdropdown' : 'field_dropdown',
name: menuName,
options: menuItems
}
Expand Down Expand Up @@ -1633,6 +1633,7 @@ class Runtime extends EventEmitter {
let valueName;
let shadowType;
let fieldName;
let allowText;

Check failure on line 1636 in src/engine/runtime.js

View workflow job for this annotation

GitHub Actions / build

'allowText' is defined but never used. Allowed unused vars must match /^_/u
if (argInfo.menu) {
const menuInfo = context.categoryInfo.menuInfo[argInfo.menu];
if (menuInfo.acceptReporters) {
Expand Down

0 comments on commit 7573aa1

Please sign in to comment.