catalyst icon indicating copy to clipboard operation
catalyst copied to clipboard

V2

Open keithamus opened this issue 2 years ago • 3 comments

v2 is a branch made of the following PRs:

  • https://github.com/github/catalyst/pull/219
  • https://github.com/github/catalyst/pull/220
  • https://github.com/github/catalyst/pull/240
  • https://github.com/github/catalyst/pull/250
  • https://github.com/github/catalyst/pull/251
  • https://github.com/github/catalyst/pull/252
  • https://github.com/github/catalyst/pull/257
  • https://github.com/github/catalyst/pull/259
  • https://github.com/github/catalyst/pull/260
  • https://github.com/github/catalyst/pull/261
  • https://github.com/github/catalyst/pull/284

These PRs are introducing breaking changes to the codebase, which is why they've been merged into this branch.

keithamus avatar May 16 '22 07:05 keithamus

I've released a 2.0.0-alpha of this branch: https://github.com/github/catalyst/releases/tag/v2.0.0-alpha1. We'll test this out and kick the tires on it for a while before merging.

keithamus avatar Jun 06 '22 17:06 keithamus

:wave: Hello and thanks for pinging us! This issue or PR has been added to our inbox and a Design Infrastructure first responder will review it soon.

  • :art: If this is a PR that includes a visual change, please make sure to add screenshots in the description or deploy this code to a lab machine with instructions for how to test.
  • :fast_forward: If this is a PR that includes changes to an interaction, please include a video recording in the description.
  • :warning: If this is urgent, please visit us in #primer on Slack and tag the first responders listed in the channel topic.

primer-css avatar Oct 10 '22 16:10 primer-css

I am unsure what the status is with v2 so I understand that it's still unstable and subject to changes. I am however finding in comparison to v1 that I am unable to use the target decorator to find the an element that is inside a shadowRoot

Hopefully I am not speaking out of turn with this. But I think the issue was due to the order in with the abilities are called in the in the controller decorator. Seems when I change the function body to register(targetable(actionable(attrable((Class))))); 

In the targetable.ts file it set's the shadowRoot to the weakmap which I think was not being called (or overwritten) with the original controller decorator

adoublef avatar Sep 08 '23 13:09 adoublef