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

Update from upstream #3

Draft
wants to merge 32 commits into
base: master
Choose a base branch
from
Draft

Update from upstream #3

wants to merge 32 commits into from

Conversation

sgammon
Copy link

@sgammon sgammon commented Feb 7, 2022

No description provided.

Sepand Parhami and others added 30 commits September 4, 2019 15:56
- This avoids package-lock.json from updating during the install,
causing the checksum to change when saving the cache.
Remove dependency on global `Node` object.
These are auto-generated files on macOS and typically shouldn't be
versioned with source code: https://en.wikipedia.org/wiki/.DS_Store

Also added `.DS_Store` to `.gitignore` to prevent regressions.
Bumps [lodash](https://github.com/lodash/lodash) from 4.17.11 to 4.17.15.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.11...4.17.15)

Signed-off-by: dependabot[bot] <[email protected]>
Add an explicit target for publishing, since it can be hard to find from
the bazel documentation.
* Add the patch context to the context object

* Update index.ts

add space
This will fix the violation only if the input is Trusted Type which I'm not sure about.
…457)

This wil allow two different frameworks using attr() in different ways to coexist (c3 and soy).
* Allow applyAttrs and applyStatics to use a different mutator object

This wil allow two different frameworks using attr() in different ways to coexist (c3 and soy).

* Fix typos
If the node is appended to the DOM before the nonce is set then Chrome exclaims with something like: “Refused to apply inline style because it violates the following Content Security Policy directive”.
* Allow applyAttrs and applyStatics to use a different mutator object

This wil allow two different frameworks using attr() in different ways to coexist (c3 and soy).

* Fix typos

* Fix tests

Fix tests
This fixes false positives that produce warning spam.

This is especially annoying for attributes templates, which can be stringified via patchOuter().
* Fix compilation issues with TypeScript 4.4.

Typescript 4.4 removes namespaceURI from the Node interface as it's not present there for all Node types. 

This change is documented in: microsoft/TypeScript-DOM-lib-generator#1029

* Update nodes.ts
The change to make the code compatible with TypeScript 4.4 broke SVG
rendering in certain cases in Chrome.
Use "@license SPDX-License-Identifier" instead of long form license to reduce bloat in compiled builds.
This includes:

- Adding a .bazelignore so building `...` doesn't try to build the node_modules directory.
- Removing a bazelrc flag that no longer exists and isn't needed anymore.
- Switching to a modern rollup_bundle implementation. This requires:
  - Adding a config file to set the global name and license banner.
  - Generating output formats separately, since it no longer generates all of them.
  - Using the Rollup Buble plugin to downlevel (transpile) to ES5.
- Switching npm_package to the modern pkg_npm.
  - This cleans up some "replacements" (formerly called "substitutions")
- Switching from `bazel` to `bazelisk` as `bazel` is now [deprecated](https://www.npmjs.com/package/@bazel/bazel) for this use case.
- Upgrading to rules_nodejs version 4.6.0.
- Switching `ts_web_test` to the modern `karma_web_test` (which is in the `@bazel/concatjs` package).

The JS output is roughly the same. There are some small changes related to variable renaming and optimizations that shouldn't affect runtime behavior. The main difference is that the const `DEBUG` replacements now work (before the `DEBUG` const got inlined before the replacement step).
This disables Chrome's sandbox when the test in run on macOS.

It's unclear why this is necessary, but it does fix the test. https://gist.github.com/mattem/f6e85437b0dbcca661013a19247889a9#file-karma-conf-js-L49 does something similar. The error in Chrome's logs is "SeatbeltExecServer: Failed to initialize sandbox: -1 Operation not permitted". I couldn't get any more details about the failure or find any details about it online. I don't think sandboxing should matter for these tests though, since it's a security feature.
Co-authored-by: Eric Spishak-Thomas <[email protected]>
This is useful for Soy functions that cun run in both idom and classic contexts.
* Pierce ShadowRoot boundary when getting ancestry

With on-demand ShadyDOM, the DOM is rendered without shadow nodes, but the API acts as if these shadow nodes exist. This can cause issues with getFocusedPath calls where the activeElement is inside of a "shadow node", thus getting the ancestry of the activeElement to the root results in an assertion error because `n.parentNode` returns undefined. We can get around this by piercing through the shadow node boundary.

This should only affect on-demand ShadyDOM since for regular ShadowDOM, the `!node.contains(activeElement)` check would be true causing getFocusedPath to short circuit.

* Remove unecessary optional chaining
In the case that the root is null, getAncestry shouldn't pierce through the shadow DOM. This only happens when `node.parentNode` is null, which only happens when the node is a document fragment or ShadowRoot.
attachShadow is part of the shadowDOM v1 spec, createShadow no longer works.
dependabot bot and others added 2 commits October 30, 2022 18:53
Bumps [socket.io-parser](https://github.com/socketio/socket.io-parser) to 4.2.1 and updates ancestor dependency [karma](https://github.com/karma-runner/karma). These dependencies need to be updated together.


Updates `socket.io-parser` from 3.2.0 to 4.2.1
- [Release notes](https://github.com/socketio/socket.io-parser/releases)
- [Changelog](https://github.com/socketio/socket.io-parser/blob/main/CHANGELOG.md)
- [Commits](socketio/socket.io-parser@3.2.0...4.2.1)

Updates `karma` from 4.2.0 to 6.4.1
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](karma-runner/karma@v4.2.0...v6.4.1)

---
updated-dependencies:
- dependency-name: socket.io-parser
  dependency-type: indirect
- dependency-name: karma
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.