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

Calling getField/getFieldValue/setFieldValue without supplying name gets/changes (value of) first unnamed field #4830

Closed
jschanker opened this issue May 14, 2021 · 1 comment
Labels
component: fields help wanted External contributions actively solicited issue: bug Describes why the code or behaviour is wrong

Comments

@jschanker
Copy link
Contributor

Calling block.setFieldValue without supplying the field name for the required second parameter results in the block's first unnamed field being set to the value of the first argument. For example, calling it on a built-in text_print block replaces print with whatever the new value that's passed is. Connected to this, calling block.getField or block.getFieldValue without supplying a name of the field (value) to get returns the block's first unnamed field (value).

To Reproduce

Steps to reproduce the behavior:

  1. Go to https://blockly-demo.appspot.com/static/tests/playground.html
  2. Drag in a text_print block.
  3. Open the Developer console and type Blockly.getMainWorkspace().getAllBlocks()[0].setFieldValue("Blockly")
  4. See the text print change to Blockly.

Expected behavior

Throw an error when calling setFieldValue without specifying the name of the field to set and throw a warning or error when calling getField or getFieldValue without specifying the name of the field (of the value) to get.

@jschanker jschanker added issue: triage Issues awaiting triage by a Blockly team member issue: bug Describes why the code or behaviour is wrong labels May 14, 2021
@rachel-fenichel rachel-fenichel added component: fields help wanted External contributions actively solicited and removed issue: triage Issues awaiting triage by a Blockly team member labels May 15, 2021
@rachel-fenichel rachel-fenichel added this to the 2021_q2_release milestone May 15, 2021
jschanker added a commit to jschanker/blockly that referenced this issue Jun 8, 2021
* Returns null when getting unnamed field (value), throws error Field "undefined" not found when attempting to set value of unnamed field
cpcallen pushed a commit that referenced this issue Jun 16, 2021
* Returns null when getting unnamed field (value), throws error Field "undefined" not found when attempting to set value of unnamed field
@cpcallen
Copy link
Contributor

Fixed by PR #4876 4876

LionKenedi95 added a commit to varwin/blockly that referenced this issue Oct 20, 2021
* update event type in blockly developer tools (google#4731)

* Rebuild (google#4733)

* Fix typo

* Update metadata March 2021 (google#4738)

* Bump webdriverio from 7.1.0 to 7.2.3

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.1.0 to 7.2.3.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.1.0...v7.2.3)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump jsdom from 16.5.0 to 16.5.2

Bumps [jsdom](https://github.com/jsdom/jsdom) from 16.5.0 to 16.5.2.
- [Release notes](https://github.com/jsdom/jsdom/releases)
- [Changelog](https://github.com/jsdom/jsdom/blob/master/Changelog.md)
- [Commits](jsdom/jsdom@16.5.0...16.5.2)

Signed-off-by: dependabot[bot] <support@github.com>

* Localisation updates from https://translatewiki.net.

* Bump @blockly/theme-modern from 2.1.10 to 2.1.13

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.10 to 2.1.13.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.13/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump @blockly/dev-tools from 2.0.10 to 2.1.2

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.0.10 to 2.1.2.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.1.2/plugins/dev-tools)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump y18n from 3.2.1 to 3.2.2

Bumps [y18n](https://github.com/yargs/y18n) from 3.2.1 to 3.2.2.
- [Release notes](https://github.com/yargs/y18n/releases)
- [Changelog](https://github.com/yargs/y18n/blob/master/CHANGELOG.md)
- [Commits](https://github.com/yargs/y18n/commits)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump eslint from 7.21.0 to 7.23.0

Bumps [eslint](https://github.com/eslint/eslint) from 7.21.0 to 7.23.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.21.0...v7.23.0)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump yargs-parser from 5.0.0 to 5.0.1

Bumps [yargs-parser](https://github.com/yargs/yargs-parser) from 5.0.0 to 5.0.1.
- [Release notes](https://github.com/yargs/yargs-parser/releases)
- [Changelog](https://github.com/yargs/yargs-parser/blob/v5.0.1/CHANGELOG.md)
- [Commits](yargs/yargs-parser@v5.0.0...v5.0.1)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump webdriverio from 7.2.3 to 7.3.0

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.2.3 to 7.3.0.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.2.3...v7.3.0)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump @blockly/block-test from 1.0.8 to 1.0.11

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.8 to 1.0.11.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.0.11/plugins/block-test)

Signed-off-by: dependabot[bot] <support@github.com>

* Localisation updates from https://translatewiki.net.

* Bump @blockly/theme-modern from 2.1.13 to 2.1.14 (google#4754)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.13 to 2.1.14.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.14/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/block-test from 1.0.11 to 1.0.12 (google#4755)

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.11 to 1.0.12.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.0.12/plugins/block-test)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/dev-tools from 2.1.2 to 2.2.0 (google#4756)

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.1.2 to 2.2.0.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.2.0/plugins/dev-tools)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Stop overriding @Protected methods with @Private (google#4760)

An upcoming change to emit private properties in Google's internal version of
https://github.com/angular/clutz will otherwise break any
TypeScript depending on these types. This is because TypeScript errors
on overriding protected with private (Closure Compiler does not).

* Localisation updates from https://translatewiki.net.

* Change compile scripts to use STABLE for language_in

* Issue google#4475 fix (google#4758)

* Issue google#4475 fix

* Revert "Issue google#4475 fix (google#4758)"

This reverts commit dd5d351.

* Issue google#4475 fix (google#4758)

* Issue google#4475 fix

* Update Flyout metrics and implement adjustable margin for scrollbars. (google#4763)

* Bump js-green-licenses from 3.0.0 to 3.0.1

Bumps [js-green-licenses](https://github.com/google/js-green-licenses) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/google/js-green-licenses/releases)
- [Changelog](https://github.com/google/js-green-licenses/blob/main/CHANGELOG.md)
- [Commits](google/js-green-licenses@v3.0.0...v3.0.1)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump webdriverio from 7.3.0 to 7.3.1

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.3.0 to 7.3.1.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.3.0...v7.3.1)

Signed-off-by: dependabot[bot] <support@github.com>

* fix: Fix typing error of generators (google#4753)

* Add exclude paths as config for typings (google#4721)

* Reverts google#4573 (google#4775)

* Reverts google#4573 (google#4775)

* Rebuild for patch release

* Don't explicitly set the default

* Remove compile.sh

* Remove rules forbidding es6

* Updates dependabot to ignore jsdom changes (google#4777)

* Localisation updates from https://translatewiki.net.

* Bump concurrently from 6.0.0 to 6.0.2

Bumps [concurrently](https://github.com/kimmobrunfeldt/concurrently) from 6.0.0 to 6.0.2.
- [Release notes](https://github.com/kimmobrunfeldt/concurrently/releases)
- [Commits](kimmobrunfeldt/concurrently@v6.0.0...v6.0.2)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump @blockly/dev-tools from 2.2.0 to 2.2.1 (google#4789)

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.2.0 to 2.2.1.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.2.1/plugins/dev-tools)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-modern from 2.1.14 to 2.1.15 (google#4788)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.14 to 2.1.15.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.15/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump webdriverio from 7.3.1 to 7.4.6 (google#4787)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.3.1 to 7.4.6.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.3.1...v7.4.6)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump google-closure-compiler from 20210302.0.0 to 20210406.0.0 (google#4786)

Bumps [google-closure-compiler](https://github.com/google/closure-compiler-npm) from 20210302.0.0 to 20210406.0.0.
- [Release notes](https://github.com/google/closure-compiler-npm/releases)
- [Commits](google/closure-compiler-npm@v20210302.0.0...v20210406.0.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Fix missing require - constructors (google#4676)

* Fix missing constructors

* Bump google-closure-deps from 20210202.0.0 to 20210406.0.0 (google#4785)

Bumps [google-closure-deps](https://github.com/google/closure-library) from 20210202.0.0 to 20210406.0.0.
- [Release notes](https://github.com/google/closure-library/releases)
- [Commits](google/closure-library@v20210202...v20210406)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Remove dark theme in favour of @blockly/theme-dark (google#4697)

* Remove dark theme in favour of @blockly/theme-dark

* Fix multiple strict require warnings (google#4793)

* Add better error messages to insertion marker (google#4791)

* Merge consecutive viewport change events (google#4792)

* Fix getDefinition being too restrictive

* Bump gulp-replace from 1.0.0 to 1.1.1 (google#4799)

Bumps [gulp-replace](https://github.com/lazd/gulp-replace) from 1.0.0 to 1.1.1.
- [Release notes](https://github.com/lazd/gulp-replace/releases)
- [Commits](https://github.com/lazd/gulp-replace/commits)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/block-test from 1.0.12 to 1.0.13 (google#4800)

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.12 to 1.0.13.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.0.13/plugins/block-test)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump webdriverio from 7.4.6 to 7.5.1 (google#4801)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.4.6 to 7.5.1.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.4.6...v7.5.1)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump eslint from 7.23.0 to 7.25.0 (google#4802)

Bumps [eslint](https://github.com/eslint/eslint) from 7.23.0 to 7.25.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.23.0...v7.25.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* PR Comments

* Move ProcedureBlock to interface

* Bump webdriverio from 7.5.1 to 7.5.3 (google#4804)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.5.1 to 7.5.3.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.5.1...v7.5.3)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump gulp-replace from 1.1.1 to 1.1.2 (google#4805)

Bumps [gulp-replace](https://github.com/lazd/gulp-replace) from 1.1.1 to 1.1.2.
- [Release notes](https://github.com/lazd/gulp-replace/releases)
- [Commits](lazd/gulp-replace@v1.1.1...v1.1.2)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add positioning helpers for trashcan and zoom controls (google#4807)

* Rename to IProcedureBlock

* Localisation updates from https://translatewiki.net.

* Revert "Rename to IProcedureBlock"

This reverts commit c8a065f.

* Revert "Move ProcedureBlock to interface"

This reverts commit 918bdc2.

* Localisation updates from https://translatewiki.net.

* Bump mocha from 8.3.2 to 8.4.0 (google#4814)

Bumps [mocha](https://github.com/mochajs/mocha) from 8.3.2 to 8.4.0.
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](mochajs/mocha@v8.3.2...v8.4.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump gulp-replace from 1.1.2 to 1.1.3 (google#4813)

Bumps [gulp-replace](https://github.com/lazd/gulp-replace) from 1.1.2 to 1.1.3.
- [Release notes](https://github.com/lazd/gulp-replace/releases)
- [Commits](lazd/gulp-replace@v1.1.2...v1.1.3)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump webdriverio from 7.5.3 to 7.5.7 (google#4815)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.5.3 to 7.5.7.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.5.3...v7.5.7)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump concurrently from 6.0.2 to 6.1.0 (google#4816)

Bumps [concurrently](https://github.com/kimmobrunfeldt/concurrently) from 6.0.2 to 6.1.0.
- [Release notes](https://github.com/kimmobrunfeldt/concurrently/releases)
- [Commits](kimmobrunfeldt/concurrently@v6.0.2...v6.1.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-modern from 2.1.15 to 2.1.16 (google#4836)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.15 to 2.1.16.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.16/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-dark from 1.0.5 to 1.0.6 (google#4835)

Bumps [@blockly/theme-dark](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-dark) from 1.0.5 to 1.0.6.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-dark@1.0.6/plugins/theme-dark)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump eslint from 7.25.0 to 7.26.0 (google#4833)

Bumps [eslint](https://github.com/eslint/eslint) from 7.25.0 to 7.26.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.25.0...v7.26.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump google-closure-compiler from 20210406.0.0 to 20210505.0.0 (google#4834)

Bumps [google-closure-compiler](https://github.com/google/closure-compiler-npm) from 20210406.0.0 to 20210505.0.0.
- [Release notes](https://github.com/google/closure-compiler-npm/releases)
- [Commits](google/closure-compiler-npm@v20210406.0.0...v20210505.0.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Rename ‘type’ to ‘realm’ within Blockly.Names

Elsewhere ’type’ refers to int/float/string when referencing variables.  This change is local and has no API effect.

* Use localeCompare’s case insensitivity option

Variables and procedures now sorted using local rules, not Unicode code points.  Verified that ’sensitivity’ survives Closure compiler without quotes.

* Normalize comments

* Change lastConnectionInRow to getPlaceForOrphanedOutput (google#4831)

* Localisation updates from https://translatewiki.net.

* Bump @blockly/block-test from 1.0.13 to 1.0.14 (google#4837)

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.13 to 1.0.14.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.0.14/plugins/block-test)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/dev-tools from 2.2.1 to 2.3.0 (google#4838)

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.2.1 to 2.3.0.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.3.0/plugins/dev-tools)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump lodash from 4.17.19 to 4.17.21 (google#4817)

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.19 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.19...4.17.21)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump hosted-git-info from 2.8.4 to 2.8.9 (google#4819)

Bumps [hosted-git-info](https://github.com/npm/hosted-git-info) from 2.8.4 to 2.8.9.
- [Release notes](https://github.com/npm/hosted-git-info/releases)
- [Changelog](https://github.com/npm/hosted-git-info/blob/v2.8.9/CHANGELOG.md)
- [Commits](npm/hosted-git-info@v2.8.4...v2.8.9)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump webdriverio from 7.5.7 to 7.6.0 (google#4839)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.5.7 to 7.6.0.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.5.7...v7.6.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Revert "Change lastConnectionInRow to getPlaceForOrphanedOutput (google#4831)" (google#4850)

This reverts commit 83640ef.

* Change lastConnectionInRow to getPlaceForOrphanedOutput - Take 2 (google#4851)

* Change lastConnectionInRow to getPlaceForOrphanedOutput

* PR comments

* Localisation updates from https://translatewiki.net.

* Rename PluginManager and related elements (google#4857)

* Rename PluginManager and related elements to use component instead of plugin

* Rename types to capabilities

* Add support for IAutoHideable (google#4855)

* Rename variableDB_ to nameDB_

There is significant confusion in names and comments with regards to variables and procedures.  `Blockly.Generator.prototype.variableDB_` is a Blockly.Names database, not a variable map.  This rename introduces a getter and setter so deprecated references still work.  This commit also fixes some comments which are either outright wrong or misleading regarding variable and procedure names.

* JSDoc improvements

* Enable extraction of names from name DB.

Also gets rid of hacky name_realm contatination.

* Remove unused Blockly.Generator.NAME_TYPE

* Set generator’s isInitialized to false in finish

Previously isInitialized would remain true after first generation.
Also move common init/finish code to parent class.

* Update JSDoc from Array.<> to Array<>

Same with Object.

* Use Object.values rather than loop

Only for JS, Lua, and PHP.  Dart and Python need extra logic to split out import statements.

Also use ‘this’ instead of fully qualified names in generators.

* Add missing extern

* Add procedures to name DB in init

This enables the generator for any block to see all variable names and procedure names in the whole program, including those that haven’t generated yet.

* Factor out procedure population from generators

Also fix bug in Lua generator where variables were not populated in the nameDB (only language with no variable initialization).

* Make explicit that names and IDs are conflated

This is potentially dangerous code.  And ID could theoretically collide with a name.

* Make JSDoc object nullability more strict.

Also make Blockly.utils.deprecation.warn apply to properties.

* More JSDoc improvements

* Remove useless variable declarations in PHP. (google#4849)

* Localisation updates from https://translatewiki.net.

* Code simplifications

* Bump ws from 7.2.3 to 7.4.6 (google#4858)

Bumps [ws](https://github.com/websockets/ws) from 7.2.3 to 7.4.6.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@7.2.3...7.4.6)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-dark from 1.0.6 to 1.0.7 (google#4861)

Bumps [@blockly/theme-dark](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-dark) from 1.0.6 to 1.0.7.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-dark@1.0.7/plugins/theme-dark)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump eslint from 7.26.0 to 7.27.0 (google#4860)

Bumps [eslint](https://github.com/eslint/eslint) from 7.26.0 to 7.27.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.26.0...v7.27.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-modern from 2.1.16 to 2.1.17 (google#4862)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.16 to 2.1.17.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.17/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update chrome version (google#4866)

* Add tests for connecting blocks (google#4853)

* Use String() over toLocaleString when possible

toLocaleString results in a 4.6% performance penalty when loading the Spaghetti test in Playground.

* Fixes a check in bump into bounds (google#4872)

* Fix error caused by extra teardown call (google#4877)

* Handle overriding components in ComponentManager (google#4856)

* ComponentManager API extension (google#4875)

* Remove row healing for block deletion google#4832 (google#4873)

* Use null-prototype objects for maps

A {} has a bunch of names already defined on it (like ‘toString’).  When using an object as a map with arbitrary keys, it should not inherit from Object.prototype.

* Back out variable change.

* Make comments more consistent.

* Remove extra else clauses

* Adding IDragTarget support. (google#4852)

* Prevent uncessary add/remove class calls on cursor during drag (google#4885)

* Fix removal of spaces near parens inside strings

Extra spaces should only be stripped from the inside of paren tokens.  Parens in strings (and other static content) should not be edited.

* Add removeComponent to ComponentManager (google#4884)

* Fix bug with updating cursor style in draggers (google#4889)

* Adds necessary methods for scroll-options plugin (google#4893)

* Add id to component interface (google#4887)

* Add a clone method to Coordinate

* Add API to IDragTarget to prevent block move (google#4886)

* use clone

* Add onDragOver and wouldDelete_ to DeleteArea (google#4888)

* keep track of state for whether the block or bubble would be deleted for use with drag enter exit

* Check if block and bubble is deletable in IDeleteArea

* Add to jsdoc of IDeleteArea

* Fixes block dragger after merge (google#4895)

* Fix bug in Flyout getClientRect (google#4901)

* Revert bubble dragger logic change and helper name change (google#4897)

* Revert accidental change in bubble drag logic

* Revert name change

* Remove extra space in generator error message.

* Adding parameter to dragEnter and dragExit (google#4890)

* introduce IDraggable interface
* Add parameter to drag methods in IDragTarget
* combines bubble/drag methods on IDragTarget and IDeleteArea to take an IDraggable element

* Creates `getLocationAfterMove` in BlockDragger (google#4905)

* Add getLocationAfterMove method to block dragger

* Renames and addresses comments

* Moves toolbox cursor styling out of block_dragger (google#4896)

* Use selenium-standalone-service to automatically start and stop Selenium when running the test suite.

* Bump node.js versions for CI

Remove v12 and add v16, per https://nodejs.org/en/about/releases/

* Reenable CI on macOS

Working on fixing google#2114.

* Update test setup for GitHub Actions

- Do the (hopefully now) correct test to check we are on macOS.
- Fix brew command-line syntax for installing google-chrome.

* Remove unneeded macOS setup

Chrome comes pre-installed on GitHub hosted macOS runners, and while
`Xvfb` does not seem to be installed neither does it seem to be needed.

* Re-disable CI on macOS

I have been unable to resolve the mocha test issues, so leave macOS
disabled for the moment.

* Revert removal of node v10 from build matrix.

* Use selenium-standalone-service to automatically start and stop Selenium when running the test suite.

* Updated CI config to run new simplified test target.

* Updated comment for runGeneratorsInBrowser to refer to Chrome instead of Firefox.

* Update JsDoc for IDragTarget (google#4907)

* Moved Typescript to devDependencies.

* Update positionable jsdoc (google#4908)

* Updated user agent parsing to detect iPads posing as desktop Safari.

* 'navigator' may be renamed by the compiler.

Use quoted name instead.  Also, navigator doesn't exist in Node, or non-browser environments.

* Adjusted line wrapping.

* Add msg files (google#4918)

* Use captureWarning helper from dev-tools instead of mocha/test_helpers.js implementation.

* Fix google#4830 for getting/setting unnamed fields

* Returns null when getting unnamed field (value), throws error Field "undefined" not found when attempting to set value of unnamed field

* Added tests/setFieldValue error message

* Added tests for getting/setting field (values) when names are not supplied and test for getting a field value, setting it to a new value, and getting it again.
* Added more user-friendly error message for setFieldValue telling the developer that he/she is missing the name rather than Field "undefined" not found.

* Added tests/setFieldValue error message

* Added tests for getting/setting field (values) when names are not supplied and test for getting a field value, setting it to a new value, and getting it again.
* Added more user-friendly error message for setFieldValue telling the developer that he/she is missing the name rather than Field "undefined" not found.
* Fixed lint error by removing trailing space

* Require field name to be string, test changes

* Now throws error for getField/getFieldValue/setFieldValue if provided name is not a string
* Changed error to more specific TypeError
* Type checking and error message moved up to getField
* Tests added/modified to check that non-string types for field names produce type errors

* Fixes workspace scroll bug (google#4921)

* Revert google#4697 (google#4917)

* Prevent an error beep when copying blocks in Safari.

* Don’t hide insertion marker if it’s already gone

Fixes google#2338.

I also looked at having workspace.clear delete any insertion marker, but there doesn’t appear to be a public API for this.  Shouldn’t matter, this should be sufficient.

* Fixed bug that prevented delete areas from updating when flyout width changed.

* Keep horizontal flyout delete area in sync with its bounds.

* Update toolbox drag target when visibility changes (google#4919)

* Update the registered component ids (google#4898)

* Remove return value from Field dropdownCreate_ (google#4915)

These functions have side effects and set all kinds of private fields.  It is misleading for them to return the top-level element, for the caller to assign to a private field.

* Honour option for collapse in context menu

Currently the ‘collapse’ option is always present, even if flagged off in the inject options.

* Refactor connect logic (google#4880)

* Refactor connect logic.

* Fixup from rebase

* Fix build

* PR Comments

* Bump @blockly/dev-tools from 2.3.0 to 2.4.0 (google#4936)

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.3.0 to 2.4.0.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.4.0/plugins/dev-tools)

---
updated-dependencies:
- dependency-name: "@blockly/dev-tools"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-modern from 2.1.17 to 2.1.18 (google#4944)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.17 to 2.1.18.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.18/plugins/theme-modern)

---
updated-dependencies:
- dependency-name: "@blockly/theme-modern"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/block-test from 1.0.14 to 1.1.0 (google#4869)

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.14 to 1.1.0.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.1.0/plugins/block-test)

---
updated-dependencies:
- dependency-name: "@blockly/block-test"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump selenium-standalone from 6.23.0 to 7.1.0 (google#4946)

Bumps [selenium-standalone](https://github.com/vvo/selenium-standalone) from 6.23.0 to 7.1.0.
- [Release notes](https://github.com/vvo/selenium-standalone/releases)
- [Changelog](https://github.com/vvo/selenium-standalone/blob/master/HISTORY.md)
- [Commits](webdriverio/selenium-standalone@v6.23.0...v7.1.0)

---
updated-dependencies:
- dependency-name: selenium-standalone
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump google-closure-deps from 20210406.0.0 to 20210601.0.0 (google#4947)

Bumps [google-closure-deps](https://github.com/google/closure-library) from 20210406.0.0 to 20210601.0.0.
- [Release notes](https://github.com/google/closure-library/releases)
- [Commits](google/closure-library@v20210406...v20210601)

---
updated-dependencies:
- dependency-name: google-closure-deps
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump eslint from 7.28.0 to 7.29.0 (google#4949)

Bumps [eslint](https://github.com/eslint/eslint) from 7.28.0 to 7.29.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.28.0...v7.29.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Remove deprecated build.py script

* Fix escape key on all fields. (google#4934)

Currently the escape key fails to revert the value.  This is the inherited root of all type-in fields, including number, angle, and note.

Fixes google#4910

* Tweak comment

* Group field edits together for undo/redo

Fixes google#4189.
Also use 0 instead of 0px, in accordance with the rest of Blockly and Google’s CSS style guide.

* Revert "Group field edits together for undo/redo"

This reverts commit 33ad244.

* Fix Jsdoc by removing trailing comma (google#4974)

* Localisation updates from https://translatewiki.net.

* Rebuild (google#4975)

* Bump README release date.

* Bump ws from 7.4.4 to 7.5.1

Bumps [ws](https://github.com/websockets/ws) from 7.4.4 to 7.5.1.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@7.4.4...7.5.1)

---
updated-dependencies:
- dependency-name: ws
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump lodash from 4.17.19 to 4.17.21

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.19 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.19...4.17.21)

---
updated-dependencies:
- dependency-name: lodash
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Initial commit for appengine deploy action

* Update comments to be more descriptive

* Get deploy files from the correct directory

Previous path was `_deploy/`. New path is `../_deploy`.

* Create Github Action to comment on PR while develop is frozen (google#5006)

* Create develop_freeze_comment.yml

* Update comments

* Fix typo and update uses

* Add test message

* Revert "Create Github Action to comment on PR while develop is frozen (google#5006)" (google#5013)

This reverts commit 8c635b5.

* Revert "Get deploy files from the correct directory"

* Bump hosted-git-info from 2.8.4 to 2.8.9 (google#4980)

Bumps [hosted-git-info](https://github.com/npm/hosted-git-info) from 2.8.4 to 2.8.9.
- [Release notes](https://github.com/npm/hosted-git-info/releases)
- [Changelog](https://github.com/npm/hosted-git-info/blob/v2.8.9/CHANGELOG.md)
- [Commits](npm/hosted-git-info@v2.8.4...v2.8.9)

---
updated-dependencies:
- dependency-name: hosted-git-info
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Pull Request template for goog.module PRs

Provide a standard pull request template more suited to PRs doing
goog.module conversions.

There's no way to pick a non-default template when creating a PR
manually, but this one can be used by adding
"&?template=goog_module.md" to the
https://github.com/cpcallen/bugs/compare/ URL.

* Update goog_module.md with suggested fixes

* Fix typos.
* Reinstate "Proposed Changes" section.
* Mark text that should be edit with ALL CAPS.

* Bump normalize-url from 4.5.0 to 4.5.1

Bumps [normalize-url](https://github.com/sindresorhus/normalize-url) from 4.5.0 to 4.5.1.
- [Release notes](https://github.com/sindresorhus/normalize-url/releases)
- [Commits](https://github.com/sindresorhus/normalize-url/commits)

---
updated-dependencies:
- dependency-name: normalize-url
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump trim-newlines from 3.0.0 to 3.0.1

Bumps [trim-newlines](https://github.com/sindresorhus/trim-newlines) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/sindresorhus/trim-newlines/releases)
- [Commits](https://github.com/sindresorhus/trim-newlines/commits)

---
updated-dependencies:
- dependency-name: trim-newlines
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump path-parse from 1.0.6 to 1.0.7

Bumps [path-parse](https://github.com/jbgutierrez/path-parse) from 1.0.6 to 1.0.7.
- [Release notes](https://github.com/jbgutierrez/path-parse/releases)
- [Commits](https://github.com/jbgutierrez/path-parse/commits/v1.0.7)

---
updated-dependencies:
- dependency-name: path-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump postcss from 7.0.35 to 7.0.36 (google#4928)

Bumps [postcss](https://github.com/postcss/postcss) from 7.0.35 to 7.0.36.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@7.0.35...7.0.36)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Rollup of all Q3 message changes (google#5565)

Courtesy of Translatewiki

* Fixes after update and build

* Add "--output" arg for gulp package comand

* Update README and package.json file by Varwin company credentials

* Fix render error in drawer.js and field_dropdown.js

Co-authored-by: Maribeth Bottorff <maribethb@google.com>
Co-authored-by: kozbial <kozbial@google.com>
Co-authored-by: alschmiedt <aschmiedt@google.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: translatewiki.net <l10n-bot@translatewiki.net>
Co-authored-by: Rachel Fenichel <fenichel@google.com>
Co-authored-by: Laura Harker <lharker@google.com>
Co-authored-by: Grayson Nix <61979456+nixgc@users.noreply.github.com>
Co-authored-by: Monica Kozbial <monica.kozbial@gmail.com>
Co-authored-by: 叶伟伟 <yww7289@163.com>
Co-authored-by: Sam El-Husseini <sam.elhusseini@gmail.com>
Co-authored-by: Beka Westberg <bekawestberg@gmail.com>
Co-authored-by: Monica Kozbial <6621618+moniika@users.noreply.github.com>
Co-authored-by: Neil Fraser <fraser@google.com>
Co-authored-by: jschanker <jschanker@molloy.edu>
Co-authored-by: Aaron Dodson <gonfunko@gmail.com>
Co-authored-by: Christopher Allen <cpcallen+git@google.com>
Co-authored-by: Neil Fraser <neil.fraser@gmail.com>
Co-authored-by: Beka Westberg <bwestberg@google.com>
Co-authored-by: Vadim Demchenko <vadim.demchenko@varwin.com>
LionKenedi95 added a commit to varwin/blockly that referenced this issue Oct 20, 2021
* Modules: first implementations

* Modules: order blocks by modules

* Modules: add modules context menu

* Modules: bugfixes

* Modules: add localisation

* Modules: build with modules

* Modules: fixed custom build

* Modules: bump version

* Modules: fixed dependency

* Modules: bugfixes

* modeules: fixed console log

* modeules: fixed modules movement

* modeules: record and restore workspace scroll for any module

* modeules: no record scroll to xml

* modeules: restore record scroll to xml

* modeules: restore record scroll to xml

* modeules: fixed labels

* modeules: fixed labels

* Modules: first implementations

* Modules: order blocks by modules

* Modules: add modules context menu

* Modules: bugfixes

* Modules: add localisation

* Modules: build with modules

* Modules: fixed custom build

* Modules: fixed dependency

* Modules: bugfixes

* modeules: fixed console log

* modeules: fixed modules movement

* modeules: record and restore workspace scroll for any module

* modeules: no record scroll to xml

* modeules: restore record scroll to xml

* modeules: fixed labels

* Bug fixes

* Add move block to module context menu option

* Fixed bug

* Move to ru translations

* Diable plus/minus feature

* Bugfix: activate previous module after delete current

* Bugfix: move block descendants to module

* Bump version

* Switch modules optimisation

* modules: store ws scale for module, fix bounding box bug for modules

* zoom controls: use zoom to fit instead scroll center

* bump version

* feature: implement obsolete attribute for blocks

* bump version

* bugfix: paths

* feature: removed and obsolete states for blocks

* bugfix: fixed blocks

* fix scroll on menu highlight
added highlight menu item on input

* bump version

* revert previous scrolling fix
fix scrolling on menu css styling

* Add custom text element in block dropdown options

* Modules feature update 6.20210701.1 (#35)

* update event type in blockly developer tools (google#4731)

* Rebuild (google#4733)

* Fix typo

* Update metadata March 2021 (google#4738)

* Bump webdriverio from 7.1.0 to 7.2.3

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.1.0 to 7.2.3.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.1.0...v7.2.3)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump jsdom from 16.5.0 to 16.5.2

Bumps [jsdom](https://github.com/jsdom/jsdom) from 16.5.0 to 16.5.2.
- [Release notes](https://github.com/jsdom/jsdom/releases)
- [Changelog](https://github.com/jsdom/jsdom/blob/master/Changelog.md)
- [Commits](jsdom/jsdom@16.5.0...16.5.2)

Signed-off-by: dependabot[bot] <support@github.com>

* Localisation updates from https://translatewiki.net.

* Bump @blockly/theme-modern from 2.1.10 to 2.1.13

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.10 to 2.1.13.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.13/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump @blockly/dev-tools from 2.0.10 to 2.1.2

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.0.10 to 2.1.2.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.1.2/plugins/dev-tools)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump y18n from 3.2.1 to 3.2.2

Bumps [y18n](https://github.com/yargs/y18n) from 3.2.1 to 3.2.2.
- [Release notes](https://github.com/yargs/y18n/releases)
- [Changelog](https://github.com/yargs/y18n/blob/master/CHANGELOG.md)
- [Commits](https://github.com/yargs/y18n/commits)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump eslint from 7.21.0 to 7.23.0

Bumps [eslint](https://github.com/eslint/eslint) from 7.21.0 to 7.23.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.21.0...v7.23.0)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump yargs-parser from 5.0.0 to 5.0.1

Bumps [yargs-parser](https://github.com/yargs/yargs-parser) from 5.0.0 to 5.0.1.
- [Release notes](https://github.com/yargs/yargs-parser/releases)
- [Changelog](https://github.com/yargs/yargs-parser/blob/v5.0.1/CHANGELOG.md)
- [Commits](yargs/yargs-parser@v5.0.0...v5.0.1)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump webdriverio from 7.2.3 to 7.3.0

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.2.3 to 7.3.0.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.2.3...v7.3.0)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump @blockly/block-test from 1.0.8 to 1.0.11

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.8 to 1.0.11.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.0.11/plugins/block-test)

Signed-off-by: dependabot[bot] <support@github.com>

* Localisation updates from https://translatewiki.net.

* Bump @blockly/theme-modern from 2.1.13 to 2.1.14 (google#4754)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.13 to 2.1.14.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.14/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/block-test from 1.0.11 to 1.0.12 (google#4755)

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.11 to 1.0.12.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.0.12/plugins/block-test)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/dev-tools from 2.1.2 to 2.2.0 (google#4756)

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.1.2 to 2.2.0.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.2.0/plugins/dev-tools)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Stop overriding @Protected methods with @Private (google#4760)

An upcoming change to emit private properties in Google's internal version of
https://github.com/angular/clutz will otherwise break any
TypeScript depending on these types. This is because TypeScript errors
on overriding protected with private (Closure Compiler does not).

* Localisation updates from https://translatewiki.net.

* Change compile scripts to use STABLE for language_in

* Issue google#4475 fix (google#4758)

* Issue google#4475 fix

* Revert "Issue google#4475 fix (google#4758)"

This reverts commit dd5d351.

* Issue google#4475 fix (google#4758)

* Issue google#4475 fix

* Update Flyout metrics and implement adjustable margin for scrollbars. (google#4763)

* Bump js-green-licenses from 3.0.0 to 3.0.1

Bumps [js-green-licenses](https://github.com/google/js-green-licenses) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/google/js-green-licenses/releases)
- [Changelog](https://github.com/google/js-green-licenses/blob/main/CHANGELOG.md)
- [Commits](google/js-green-licenses@v3.0.0...v3.0.1)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump webdriverio from 7.3.0 to 7.3.1

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.3.0 to 7.3.1.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.3.0...v7.3.1)

Signed-off-by: dependabot[bot] <support@github.com>

* fix: Fix typing error of generators (google#4753)

* Add exclude paths as config for typings (google#4721)

* Reverts google#4573 (google#4775)

* Reverts google#4573 (google#4775)

* Rebuild for patch release

* Don't explicitly set the default

* Remove compile.sh

* Remove rules forbidding es6

* Updates dependabot to ignore jsdom changes (google#4777)

* Localisation updates from https://translatewiki.net.

* Bump concurrently from 6.0.0 to 6.0.2

Bumps [concurrently](https://github.com/kimmobrunfeldt/concurrently) from 6.0.0 to 6.0.2.
- [Release notes](https://github.com/kimmobrunfeldt/concurrently/releases)
- [Commits](kimmobrunfeldt/concurrently@v6.0.0...v6.0.2)

Signed-off-by: dependabot[bot] <support@github.com>

* Bump @blockly/dev-tools from 2.2.0 to 2.2.1 (google#4789)

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.2.0 to 2.2.1.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.2.1/plugins/dev-tools)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-modern from 2.1.14 to 2.1.15 (google#4788)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.14 to 2.1.15.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.15/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump webdriverio from 7.3.1 to 7.4.6 (google#4787)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.3.1 to 7.4.6.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.3.1...v7.4.6)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump google-closure-compiler from 20210302.0.0 to 20210406.0.0 (google#4786)

Bumps [google-closure-compiler](https://github.com/google/closure-compiler-npm) from 20210302.0.0 to 20210406.0.0.
- [Release notes](https://github.com/google/closure-compiler-npm/releases)
- [Commits](google/closure-compiler-npm@v20210302.0.0...v20210406.0.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Fix missing require - constructors (google#4676)

* Fix missing constructors

* Bump google-closure-deps from 20210202.0.0 to 20210406.0.0 (google#4785)

Bumps [google-closure-deps](https://github.com/google/closure-library) from 20210202.0.0 to 20210406.0.0.
- [Release notes](https://github.com/google/closure-library/releases)
- [Commits](google/closure-library@v20210202...v20210406)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Remove dark theme in favour of @blockly/theme-dark (google#4697)

* Remove dark theme in favour of @blockly/theme-dark

* Fix multiple strict require warnings (google#4793)

* Add better error messages to insertion marker (google#4791)

* Merge consecutive viewport change events (google#4792)

* Fix getDefinition being too restrictive

* Bump gulp-replace from 1.0.0 to 1.1.1 (google#4799)

Bumps [gulp-replace](https://github.com/lazd/gulp-replace) from 1.0.0 to 1.1.1.
- [Release notes](https://github.com/lazd/gulp-replace/releases)
- [Commits](https://github.com/lazd/gulp-replace/commits)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/block-test from 1.0.12 to 1.0.13 (google#4800)

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.12 to 1.0.13.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.0.13/plugins/block-test)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump webdriverio from 7.4.6 to 7.5.1 (google#4801)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.4.6 to 7.5.1.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.4.6...v7.5.1)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump eslint from 7.23.0 to 7.25.0 (google#4802)

Bumps [eslint](https://github.com/eslint/eslint) from 7.23.0 to 7.25.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.23.0...v7.25.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* PR Comments

* Move ProcedureBlock to interface

* Bump webdriverio from 7.5.1 to 7.5.3 (google#4804)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.5.1 to 7.5.3.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.5.1...v7.5.3)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump gulp-replace from 1.1.1 to 1.1.2 (google#4805)

Bumps [gulp-replace](https://github.com/lazd/gulp-replace) from 1.1.1 to 1.1.2.
- [Release notes](https://github.com/lazd/gulp-replace/releases)
- [Commits](lazd/gulp-replace@v1.1.1...v1.1.2)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add positioning helpers for trashcan and zoom controls (google#4807)

* Rename to IProcedureBlock

* Localisation updates from https://translatewiki.net.

* Revert "Rename to IProcedureBlock"

This reverts commit c8a065f.

* Revert "Move ProcedureBlock to interface"

This reverts commit 918bdc2.

* Localisation updates from https://translatewiki.net.

* Bump mocha from 8.3.2 to 8.4.0 (google#4814)

Bumps [mocha](https://github.com/mochajs/mocha) from 8.3.2 to 8.4.0.
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](mochajs/mocha@v8.3.2...v8.4.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump gulp-replace from 1.1.2 to 1.1.3 (google#4813)

Bumps [gulp-replace](https://github.com/lazd/gulp-replace) from 1.1.2 to 1.1.3.
- [Release notes](https://github.com/lazd/gulp-replace/releases)
- [Commits](lazd/gulp-replace@v1.1.2...v1.1.3)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump webdriverio from 7.5.3 to 7.5.7 (google#4815)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.5.3 to 7.5.7.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.5.3...v7.5.7)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump concurrently from 6.0.2 to 6.1.0 (google#4816)

Bumps [concurrently](https://github.com/kimmobrunfeldt/concurrently) from 6.0.2 to 6.1.0.
- [Release notes](https://github.com/kimmobrunfeldt/concurrently/releases)
- [Commits](kimmobrunfeldt/concurrently@v6.0.2...v6.1.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-modern from 2.1.15 to 2.1.16 (google#4836)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.15 to 2.1.16.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.16/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-dark from 1.0.5 to 1.0.6 (google#4835)

Bumps [@blockly/theme-dark](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-dark) from 1.0.5 to 1.0.6.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-dark@1.0.6/plugins/theme-dark)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump eslint from 7.25.0 to 7.26.0 (google#4833)

Bumps [eslint](https://github.com/eslint/eslint) from 7.25.0 to 7.26.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.25.0...v7.26.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump google-closure-compiler from 20210406.0.0 to 20210505.0.0 (google#4834)

Bumps [google-closure-compiler](https://github.com/google/closure-compiler-npm) from 20210406.0.0 to 20210505.0.0.
- [Release notes](https://github.com/google/closure-compiler-npm/releases)
- [Commits](google/closure-compiler-npm@v20210406.0.0...v20210505.0.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Rename ‘type’ to ‘realm’ within Blockly.Names

Elsewhere ’type’ refers to int/float/string when referencing variables.  This change is local and has no API effect.

* Use localeCompare’s case insensitivity option

Variables and procedures now sorted using local rules, not Unicode code points.  Verified that ’sensitivity’ survives Closure compiler without quotes.

* Normalize comments

* Change lastConnectionInRow to getPlaceForOrphanedOutput (google#4831)

* Localisation updates from https://translatewiki.net.

* Bump @blockly/block-test from 1.0.13 to 1.0.14 (google#4837)

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.13 to 1.0.14.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.0.14/plugins/block-test)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/dev-tools from 2.2.1 to 2.3.0 (google#4838)

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.2.1 to 2.3.0.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.3.0/plugins/dev-tools)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump lodash from 4.17.19 to 4.17.21 (google#4817)

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.19 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.19...4.17.21)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump hosted-git-info from 2.8.4 to 2.8.9 (google#4819)

Bumps [hosted-git-info](https://github.com/npm/hosted-git-info) from 2.8.4 to 2.8.9.
- [Release notes](https://github.com/npm/hosted-git-info/releases)
- [Changelog](https://github.com/npm/hosted-git-info/blob/v2.8.9/CHANGELOG.md)
- [Commits](npm/hosted-git-info@v2.8.4...v2.8.9)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump webdriverio from 7.5.7 to 7.6.0 (google#4839)

Bumps [webdriverio](https://github.com/webdriverio/webdriverio) from 7.5.7 to 7.6.0.
- [Release notes](https://github.com/webdriverio/webdriverio/releases)
- [Changelog](https://github.com/webdriverio/webdriverio/blob/main/CHANGELOG.md)
- [Commits](webdriverio/webdriverio@v7.5.7...v7.6.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Revert "Change lastConnectionInRow to getPlaceForOrphanedOutput (google#4831)" (google#4850)

This reverts commit 83640ef.

* Change lastConnectionInRow to getPlaceForOrphanedOutput - Take 2 (google#4851)

* Change lastConnectionInRow to getPlaceForOrphanedOutput

* PR comments

* Localisation updates from https://translatewiki.net.

* Rename PluginManager and related elements (google#4857)

* Rename PluginManager and related elements to use component instead of plugin

* Rename types to capabilities

* Add support for IAutoHideable (google#4855)

* Rename variableDB_ to nameDB_

There is significant confusion in names and comments with regards to variables and procedures.  `Blockly.Generator.prototype.variableDB_` is a Blockly.Names database, not a variable map.  This rename introduces a getter and setter so deprecated references still work.  This commit also fixes some comments which are either outright wrong or misleading regarding variable and procedure names.

* JSDoc improvements

* Enable extraction of names from name DB.

Also gets rid of hacky name_realm contatination.

* Remove unused Blockly.Generator.NAME_TYPE

* Set generator’s isInitialized to false in finish

Previously isInitialized would remain true after first generation.
Also move common init/finish code to parent class.

* Update JSDoc from Array.<> to Array<>

Same with Object.

* Use Object.values rather than loop

Only for JS, Lua, and PHP.  Dart and Python need extra logic to split out import statements.

Also use ‘this’ instead of fully qualified names in generators.

* Add missing extern

* Add procedures to name DB in init

This enables the generator for any block to see all variable names and procedure names in the whole program, including those that haven’t generated yet.

* Factor out procedure population from generators

Also fix bug in Lua generator where variables were not populated in the nameDB (only language with no variable initialization).

* Make explicit that names and IDs are conflated

This is potentially dangerous code.  And ID could theoretically collide with a name.

* Make JSDoc object nullability more strict.

Also make Blockly.utils.deprecation.warn apply to properties.

* More JSDoc improvements

* Remove useless variable declarations in PHP. (google#4849)

* Localisation updates from https://translatewiki.net.

* Code simplifications

* Bump ws from 7.2.3 to 7.4.6 (google#4858)

Bumps [ws](https://github.com/websockets/ws) from 7.2.3 to 7.4.6.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@7.2.3...7.4.6)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-dark from 1.0.6 to 1.0.7 (google#4861)

Bumps [@blockly/theme-dark](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-dark) from 1.0.6 to 1.0.7.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-dark@1.0.7/plugins/theme-dark)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump eslint from 7.26.0 to 7.27.0 (google#4860)

Bumps [eslint](https://github.com/eslint/eslint) from 7.26.0 to 7.27.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.26.0...v7.27.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-modern from 2.1.16 to 2.1.17 (google#4862)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.16 to 2.1.17.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.17/plugins/theme-modern)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update chrome version (google#4866)

* Add tests for connecting blocks (google#4853)

* Use String() over toLocaleString when possible

toLocaleString results in a 4.6% performance penalty when loading the Spaghetti test in Playground.

* Fixes a check in bump into bounds (google#4872)

* Fix error caused by extra teardown call (google#4877)

* Handle overriding components in ComponentManager (google#4856)

* ComponentManager API extension (google#4875)

* Remove row healing for block deletion google#4832 (google#4873)

* Use null-prototype objects for maps

A {} has a bunch of names already defined on it (like ‘toString’).  When using an object as a map with arbitrary keys, it should not inherit from Object.prototype.

* Back out variable change.

* Make comments more consistent.

* Remove extra else clauses

* Adding IDragTarget support. (google#4852)

* Prevent uncessary add/remove class calls on cursor during drag (google#4885)

* Fix removal of spaces near parens inside strings

Extra spaces should only be stripped from the inside of paren tokens.  Parens in strings (and other static content) should not be edited.

* Add removeComponent to ComponentManager (google#4884)

* Fix bug with updating cursor style in draggers (google#4889)

* Adds necessary methods for scroll-options plugin (google#4893)

* Add id to component interface (google#4887)

* Add a clone method to Coordinate

* Add API to IDragTarget to prevent block move (google#4886)

* use clone

* Add onDragOver and wouldDelete_ to DeleteArea (google#4888)

* keep track of state for whether the block or bubble would be deleted for use with drag enter exit

* Check if block and bubble is deletable in IDeleteArea

* Add to jsdoc of IDeleteArea

* Fixes block dragger after merge (google#4895)

* Fix bug in Flyout getClientRect (google#4901)

* Revert bubble dragger logic change and helper name change (google#4897)

* Revert accidental change in bubble drag logic

* Revert name change

* Remove extra space in generator error message.

* Adding parameter to dragEnter and dragExit (google#4890)

* introduce IDraggable interface
* Add parameter to drag methods in IDragTarget
* combines bubble/drag methods on IDragTarget and IDeleteArea to take an IDraggable element

* Creates `getLocationAfterMove` in BlockDragger (google#4905)

* Add getLocationAfterMove method to block dragger

* Renames and addresses comments

* Moves toolbox cursor styling out of block_dragger (google#4896)

* Use selenium-standalone-service to automatically start and stop Selenium when running the test suite.

* Bump node.js versions for CI

Remove v12 and add v16, per https://nodejs.org/en/about/releases/

* Reenable CI on macOS

Working on fixing google#2114.

* Update test setup for GitHub Actions

- Do the (hopefully now) correct test to check we are on macOS.
- Fix brew command-line syntax for installing google-chrome.

* Remove unneeded macOS setup

Chrome comes pre-installed on GitHub hosted macOS runners, and while
`Xvfb` does not seem to be installed neither does it seem to be needed.

* Re-disable CI on macOS

I have been unable to resolve the mocha test issues, so leave macOS
disabled for the moment.

* Revert removal of node v10 from build matrix.

* Use selenium-standalone-service to automatically start and stop Selenium when running the test suite.

* Updated CI config to run new simplified test target.

* Updated comment for runGeneratorsInBrowser to refer to Chrome instead of Firefox.

* Update JsDoc for IDragTarget (google#4907)

* Moved Typescript to devDependencies.

* Update positionable jsdoc (google#4908)

* Updated user agent parsing to detect iPads posing as desktop Safari.

* 'navigator' may be renamed by the compiler.

Use quoted name instead.  Also, navigator doesn't exist in Node, or non-browser environments.

* Adjusted line wrapping.

* Add msg files (google#4918)

* Use captureWarning helper from dev-tools instead of mocha/test_helpers.js implementation.

* Fix google#4830 for getting/setting unnamed fields

* Returns null when getting unnamed field (value), throws error Field "undefined" not found when attempting to set value of unnamed field

* Added tests/setFieldValue error message

* Added tests for getting/setting field (values) when names are not supplied and test for getting a field value, setting it to a new value, and getting it again.
* Added more user-friendly error message for setFieldValue telling the developer that he/she is missing the name rather than Field "undefined" not found.

* Added tests/setFieldValue error message

* Added tests for getting/setting field (values) when names are not supplied and test for getting a field value, setting it to a new value, and getting it again.
* Added more user-friendly error message for setFieldValue telling the developer that he/she is missing the name rather than Field "undefined" not found.
* Fixed lint error by removing trailing space

* Require field name to be string, test changes

* Now throws error for getField/getFieldValue/setFieldValue if provided name is not a string
* Changed error to more specific TypeError
* Type checking and error message moved up to getField
* Tests added/modified to check that non-string types for field names produce type errors

* Fixes workspace scroll bug (google#4921)

* Revert google#4697 (google#4917)

* Prevent an error beep when copying blocks in Safari.

* Don’t hide insertion marker if it’s already gone

Fixes google#2338.

I also looked at having workspace.clear delete any insertion marker, but there doesn’t appear to be a public API for this.  Shouldn’t matter, this should be sufficient.

* Fixed bug that prevented delete areas from updating when flyout width changed.

* Keep horizontal flyout delete area in sync with its bounds.

* Update toolbox drag target when visibility changes (google#4919)

* Update the registered component ids (google#4898)

* Remove return value from Field dropdownCreate_ (google#4915)

These functions have side effects and set all kinds of private fields.  It is misleading for them to return the top-level element, for the caller to assign to a private field.

* Honour option for collapse in context menu

Currently the ‘collapse’ option is always present, even if flagged off in the inject options.

* Refactor connect logic (google#4880)

* Refactor connect logic.

* Fixup from rebase

* Fix build

* PR Comments

* Bump @blockly/dev-tools from 2.3.0 to 2.4.0 (google#4936)

Bumps [@blockly/dev-tools](https://github.com/google/blockly-samples/tree/HEAD/plugins/dev-tools) from 2.3.0 to 2.4.0.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/dev-tools@2.4.0/plugins/dev-tools)

---
updated-dependencies:
- dependency-name: "@blockly/dev-tools"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/theme-modern from 2.1.17 to 2.1.18 (google#4944)

Bumps [@blockly/theme-modern](https://github.com/google/blockly-samples/tree/HEAD/plugins/theme-modern) from 2.1.17 to 2.1.18.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/theme-modern@2.1.18/plugins/theme-modern)

---
updated-dependencies:
- dependency-name: "@blockly/theme-modern"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @blockly/block-test from 1.0.14 to 1.1.0 (google#4869)

Bumps [@blockly/block-test](https://github.com/google/blockly-samples/tree/HEAD/plugins/block-test) from 1.0.14 to 1.1.0.
- [Release notes](https://github.com/google/blockly-samples/releases)
- [Commits](https://github.com/google/blockly-samples/commits/@blockly/block-test@1.1.0/plugins/block-test)

---
updated-dependencies:
- dependency-name: "@blockly/block-test"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump selenium-standalone from 6.23.0 to 7.1.0 (google#4946)

Bumps [selenium-standalone](https://github.com/vvo/selenium-standalone) from 6.23.0 to 7.1.0.
- [Release notes](https://github.com/vvo/selenium-standalone/releases)
- [Changelog](https://github.com/vvo/selenium-standalone/blob/master/HISTORY.md)
- [Commits](webdriverio/selenium-standalone@v6.23.0...v7.1.0)

---
updated-dependencies:
- dependency-name: selenium-standalone
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump google-closure-deps from 20210406.0.0 to 20210601.0.0 (google#4947)

Bumps [google-closure-deps](https://github.com/google/closure-library) from 20210406.0.0 to 20210601.0.0.
- [Release notes](https://github.com/google/closure-library/releases)
- [Commits](google/closure-library@v20210406...v20210601)

---
updated-dependencies:
- dependency-name: google-closure-deps
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump eslint from 7.28.0 to 7.29.0 (google#4949)

Bumps [eslint](https://github.com/eslint/eslint) from 7.28.0 to 7.29.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.28.0...v7.29.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Remove deprecated build.py script

* Fix escape key on all fields. (google#4934)

Currently the escape key fails to revert the value.  This is the inherited root of all type-in fields, including number, angle, and note.

Fixes google#4910

* Tweak comment

* Group field edits together for undo/redo

Fixes google#4189.
Also use 0 instead of 0px, in accordance with the rest of Blockly and Google’s CSS style guide.

* Revert "Group field edits together for undo/redo"

This reverts commit 33ad244.

* Fix Jsdoc by removing trailing comma (google#4974)

* Localisation updates from https://translatewiki.net.

* Rebuild (google#4975)

* Bump README release date.

* Bump ws from 7.4.4 to 7.5.1

Bumps [ws](https://github.com/websockets/ws) from 7.4.4 to 7.5.1.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@7.4.4...7.5.1)

---
updated-dependencies:
- dependency-name: ws
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump lodash from 4.17.19 to 4.17.21

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.19 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.19...4.17.21)

---
updated-dependencies:
- dependency-name: lodash
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Initial commit for appengine deploy action

* Update comments to be more descriptive

* Get deploy files from the correct directory

Previous path was `_deploy/`. New path is `../_deploy`.

* Create Github Action to comment on PR while develop is frozen (google#5006)

* Create develop_freeze_comment.yml

* Update comments

* Fix typo and update uses

* Add test message

* Revert "Create Github Action to comment on PR while develop is frozen (google#5006)" (google#5013)

This reverts commit 8c635b5.

* Revert "Get deploy files from the correct directory"

* Bump hosted-git-info from 2.8.4 to 2.8.9 (google#4980)

Bumps [hosted-git-info](https://github.com/npm/hosted-git-info) from 2.8.4 to 2.8.9.
- [Release notes](https://github.com/npm/hosted-git-info/releases)
- [Changelog](https://github.com/npm/hosted-git-info/blob/v2.8.9/CHANGELOG.md)
- [Commits](npm/hosted-git-info@v2.8.4...v2.8.9)

---
updated-dependencies:
- dependency-name: hosted-git-info
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Pull Request template for goog.module PRs

Provide a standard pull request template more suited to PRs doing
goog.module conversions.

There's no way to pick a non-default template when creating a PR
manually, but this one can be used by adding
"&?template=goog_module.md" to the
https://github.com/cpcallen/bugs/compare/ URL.

* Update goog_module.md with suggested fixes

* Fix typos.
* Reinstate "Proposed Changes" section.
* Mark text that should be edit with ALL CAPS.

* Bump normalize-url from 4.5.0 to 4.5.1

Bumps [normalize-url](https://github.com/sindresorhus/normalize-url) from 4.5.0 to 4.5.1.
- [Release notes](https://github.com/sindresorhus/normalize-url/releases)
- [Commits](https://github.com/sindresorhus/normalize-url/commits)

---
updated-dependencies:
- dependency-name: normalize-url
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump trim-newlines from 3.0.0 to 3.0.1

Bumps [trim-newlines](https://github.com/sindresorhus/trim-newlines) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/sindresorhus/trim-newlines/releases)
- [Commits](https://github.com/sindresorhus/trim-newlines/commits)

---
updated-dependencies:
- dependency-name: trim-newlines
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump path-parse from 1.0.6 to 1.0.7

Bumps [path-parse](https://github.com/jbgutierrez/path-parse) from 1.0.6 to 1.0.7.
- [Release notes](https://github.com/jbgutierrez/path-parse/releases)
- [Commits](https://github.com/jbgutierrez/path-parse/commits/v1.0.7)

---
updated-dependencies:
- dependency-name: path-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump postcss from 7.0.35 to 7.0.36 (google#4928)

Bumps [postcss](https://github.com/postcss/postcss) from 7.0.35 to 7.0.36.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@7.0.35...7.0.36)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Rollup of all Q3 message changes (google#5565)

Courtesy of Translatewiki

* Fixes after update and build

* Add "--output" arg for gulp package comand

* Update README and package.json file by Varwin company credentials

* Fix render error in drawer.js and field_dropdown.js

Co-authored-by: Maribeth Bottorff <maribethb@google.com>
Co-authored-by: kozbial <kozbial@google.com>
Co-authored-by: alschmiedt <aschmiedt@google.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: translatewiki.net <l10n-bot@translatewiki.net>
Co-authored-by: Rachel Fenichel <fenichel@google.com>
Co-authored-by: Laura Harker <lharker@google.com>
Co-authored-by: Grayson Nix <61979456+nixgc@users.noreply.github.com>
Co-authored-by: Monica Kozbial <monica.kozbial@gmail.com>
Co-authored-by: 叶伟伟 <yww7289@163.com>
Co-authored-by: Sam El-Husseini <sam.elhusseini@gmail.com>
Co-authored-by: Beka Westberg <bekawestberg@gmail.com>
Co-authored-by: Monica Kozbial <6621618+moniika@users.noreply.github.com>
Co-authored-by: Neil Fraser <fraser@google.com>
Co-authored-by: jschanker <jschanker@molloy.edu>
Co-authored-by: Aaron Dodson <gonfunko@gmail.com>
Co-authored-by: Christopher Allen <cpcallen+git@google.com>
Co-authored-by: Neil Fraser <neil.fraser@gmail.com>
Co-authored-by: Beka Westberg <bwestberg@google.com>
Co-authored-by: Vadim Demchenko <vadim.demchenko@varwin.com>

* Update README.md

Co-authored-by: Anton Prusov <anton.prusov@cyberlit.ru>
Co-authored-by: rubinshtein <aleksey.rubinshtein@varwin.com>
Co-authored-by: Vadim Demchenko <vadim.demchenko@varwin.com>
Co-authored-by: Vadim Demchenko <lionkenedi95@gmail.com>
Co-authored-by: Maribeth Bottorff <maribethb@google.com>
Co-authored-by: kozbial <kozbial@google.com>
Co-authored-by: alschmiedt <aschmiedt@google.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: translatewiki.net <l10n-bot@translatewiki.net>
Co-authored-by: Rachel Fenichel <fenichel@google.com>
Co-authored-by: Laura Harker <lharker@google.com>
Co-authored-by: Grayson Nix <61979456+nixgc@users.noreply.github.com>
Co-authored-by: Monica Kozbial <monica.kozbial@gmail.com>
Co-authored-by: 叶伟伟 <yww7289@163.com>
Co-authored-by: Sam El-Husseini <sam.elhusseini@gmail.com>
Co-authored-by: Beka Westberg <bekawestberg@gmail.com>
Co-authored-by: Monica Kozbial <6621618+moniika@users.noreply.github.com>
Co-authored-by: Neil Fraser <fraser@google.com>
Co-authored-by: jschanker <jschanker@molloy.edu>
Co-authored-by: Aaron Dodson <gonfunko@gmail.com>
Co-authored-by: Christopher Allen <cpcallen+git@google.com>
Co-authored-by: Neil Fraser <neil.fraser@gmail.com>
Co-authored-by: Beka Westberg <bwestberg@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: fields help wanted External contributions actively solicited issue: bug Describes why the code or behaviour is wrong
Projects
None yet
Development

No branches or pull requests

3 participants