Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jun 2, 2025

Bumps google-closure-compiler from 20240317.0.0 to 20250528.0.0.

Release notes

Sourced from google-closure-compiler's releases.

20250528.0.0

Closure-compiler 20250528 release

What's Changed

Full Changelog: google/closure-compiler-npm@v20250526.0.0...v20250528.0.0

20250526.0.0

Closure-compiler 20250526 release

Full Changelog: google/closure-compiler-npm@v20250520.0.0...v20250526.0.0

20250520.0.0

Closure-compiler 20250520 release

Full Changelog: google/closure-compiler-npm@v20250519.0.0...v20250520.0.0

20250519.0.0

Closure-compiler 20250519 release

What's Changed

Full Changelog: google/closure-compiler-npm@v20250518.0.0...v20250519.0.0

20250518.0.0

Closure-compiler 20250518 release

What's Changed

Full Changelog: google/closure-compiler-npm@v20250514.0.0...v20250518.0.0

20250514.0.0

Closure-compiler 20250514 release

Full Changelog: google/closure-compiler-npm@v20250512.0.0...v20250514.0.0

20250512.0.0

Closure-compiler 20250512 release

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [google-closure-compiler](https://github.com/google/closure-compiler-npm) from 20240317.0.0 to 20250528.0.0.
- [Release notes](https://github.com/google/closure-compiler-npm/releases)
- [Commits](google/closure-compiler-npm@v20240317.0.0...v20250528.0.0)

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

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added PR: chore General chores (dependencies, typos, etc) PR: dependencies Pull requests that update a dependency file labels Jun 2, 2025
@dependabot dependabot bot requested a review from a team as a code owner June 2, 2025 10:11
@dependabot dependabot bot requested a review from gonfunko June 2, 2025 10:11
@dependabot dependabot bot added PR: dependencies Pull requests that update a dependency file PR: chore General chores (dependencies, typos, etc) labels Jun 2, 2025
@dependabot dependabot bot requested a review from rachel-fenichel June 2, 2025 10:11
@github-actions github-actions bot added PR: chore General chores (dependencies, typos, etc) and removed PR: chore General chores (dependencies, typos, etc) labels Jun 2, 2025
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Jun 9, 2025

Superseded by #9131.

@dependabot dependabot bot closed this Jun 9, 2025
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/develop/google-closure-compiler-20250528.0.0 branch June 9, 2025 09:47
BenHenning added a commit that referenced this pull request Jul 8, 2025
## The basics

- [x] I [validated my changes](https://developers.google.com/blockly/guides/contribute/core#making_and_verifying_a_change)

## The details
### Resolves

Fixes RaspberryPiFoundation/blockly-keyboard-experimentation#563

### Proposed Changes

This expands the functionality introduced in #9213 to also include widget divs.

### Reason for Changes

MakeCode makes use of widget div in several field editors, so the issues described in RaspberryPiFoundation/blockly-keyboard-experimentation#563 aren't fully mitigated with #9213 alone.

This PR essentially adds the same support for auto-closing as drop-down divs now have, and enables this functionality by default.

Note the drop-down div change: it was missed in #9123 that the API change for drop-down div's `show` function is actually API-breaking, so this updates that API to be properly backward compatible (and reverts one test change that makes use of it).

The `FocusManager` change actually corrects an implementation issue from #9123: not updating the tracked focus status before calling the callback can result in focus being inadvertently restored if the callback triggers returning focus due to a lost focus situation. This was wrong for drop-down divs, too, but it's harder to notice there because the dismissal of the drop-down div happens on a timer (which means there's sufficient time for `FocusManager`'s state to correct prior to attempting to return from the ephemeral focus state).

Demonstration of fixed behavior (since the inline number editor uses a widget div):

[Screen recording 2025-07-08 2.12.31 PM.webm](https://github.com/user-attachments/assets/7c3c7c3c-224c-48f4-b4af-bde86feecfa8)

### Test Coverage

New widget div tests have been added to verify the new parameter and auto-close functionality.

The `FocusManager` test was updated to account for the new, and correct, behavior around the internal tracked ephemeral focus state.

Note that some `tabindex` state has been clarified and cleaned up in the test index page and `FocusManager`. It's fine (and preferable) for ephemeral-used elements to always be focusable rather than making them dynamically so (which avoids state bleed across test runs which was happening one of the new tests).

RaspberryPiFoundation/blockly-keyboard-experimentation#649 includes additional tests for validating widget behaviors.

### Documentation

No new documentation should be needed here--the API documentation changes should be sufficient.

One documentation update was made in `dropdowndiv.ts` that corrects the documentation parameter ordering.

### Additional Information

Nothing further to add.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: chore General chores (dependencies, typos, etc) PR: dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants