Jean-Yves Moyen

Results 290 comments of Jean-Yves Moyen

@RyanCavanaugh I've managed to extract a MNWE 🎉 Updated the original message with code and link to playground. It is still 80 lines long, and the union should be around...

@gabritto Thanks for looking into this. So, if I get this correctly, this boils down to expanding cross products like `(A | B) & (C | D)` into `(A &...

Digging a bit more into it… It seems my MWNE is actually incorrect in what it tries to do. ```typescript function computed( property: Longhands[N][1], specified: Longhands[N][0] ) { // error...

Facing the same issue and digging a bit. ~I think it is a problem with [`dependency-tree`](https://github.com/dependents/node-dependency-tree), likely https://github.com/dependents/node-dependency-tree/issues/118, which Madge uses to get its dependencies. At least [the "skipping" message...

OK, I think this is a `dependency-tree` problem that was fixed in v10, `madge` is still running v9 and thus has the problem. * Adding a `console.log` statement to see...

…which should make it easy to fix 😁 The [latest release of madge, v6.1.0](https://github.com/pahen/madge/releases/tag/v6.1.0) is from June '23. `dependency-tree` has been [updated to v10.0.9](https://github.com/pahen/madge/commit/5bfd142a5cb78ee5abb0a2346ea2c25ea15c0127) in October '23. So, I assume...

> Is there anything else you did to get it to work? Since I am only at the "experiment and play" step, I brutally edited `madge.js` in my `node_modules` to...

For yarn, the workaround in `package.json` is: ``` "resolutions": { "dependency-tree": "^10.0.9" } ```

FYI, updating to madge 7.0.0 fixed this for me (no need for custom resolution anymore). The correct dependency-tree is now a dependency 🎉

So, it seems that `CSSStyleDeclaration.style[i]` only exposes longhands, but not the shorthands that are defined. Upon looking at the declaration corresponding to that block, we get: ``` … 0: "background-image"...