kdl icon indicating copy to clipboard operation
kdl copied to clipboard

Published 20 hours ago • verified publisher icon kdl-org

KQL: remove map operator and accessors

Open zkat opened this issue 3 years ago • 3 comments

Honestly, they're just too implementation-specific

zkat avatar Oct 06 '21 05:10 zkat

My poor heart when I read this just as I finally finished getting my map operator and accessor implementations working 😭

danini-the-panini avatar Dec 22 '21 21:12 danini-the-panini

Map operator is still great, and if implementations want something like that, it makes sense!

I just didn't think it made sense for the spec

zkat avatar Dec 23 '21 16:12 zkat

The individual accessors without the map operator line up very nicely to JSX land concepts.

The map operator on the back of the accessors works really well too.

Since they have stayed in the spec this long, and map operators are optional now, instead of removing them both, maybe we should just map accessors optional too?

I have included both accessors and map operator support in the JSX query engine.

https://github.com/virtualstate/kdl/blob/464d0cd5e2aae99bdcc97f00ea8a50ce00a4021a/src/tests/readme.tsx#L79-L88

fabiancook avatar Jul 30 '22 07:07 fabiancook