Skip to content
This repository was archived by the owner on Oct 23, 2023. It is now read-only.

feat: add polymorphic factory for solid #27

Merged
merged 7 commits into from
Nov 15, 2022
Merged

feat: add polymorphic factory for solid #27

merged 7 commits into from
Nov 15, 2022

Conversation

TimKolberger
Copy link
Collaborator

@TimKolberger TimKolberger commented Nov 14, 2022

I ported the polymorphic factory to solid and used the built in Dynamic component.
Seems to work as good as with React 🎉

Event types in callbacks e.g. onClick still have the old component type

@TimKolberger TimKolberger marked this pull request as ready for review November 14, 2022 20:31
@TimKolberger TimKolberger changed the base branch from main to feat/explicit-types-and-exports November 14, 2022 20:31
@codecov
Copy link

codecov bot commented Nov 14, 2022

Codecov Report

Merging #27 (ddc01ca) into main (dd062b4) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              main       #27    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files            3         5     +2     
  Lines          145       269   +124     
  Branches        10        18     +8     
==========================================
+ Hits           145       269   +124     
Impacted Files Coverage Δ
packages/react/src/forwardRef.tsx 100.00% <100.00%> (ø)
packages/react/src/index.ts 100.00% <100.00%> (ø)
packages/react/src/polymorphic-factory.tsx 100.00% <100.00%> (ø)
packages/solid/src/index.ts 100.00% <100.00%> (ø)
packages/solid/src/polymorphic-factory.tsx 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Base automatically changed from feat/explicit-types-and-exports to main November 15, 2022 10:30
@TimKolberger TimKolberger merged commit eb3d8a1 into main Nov 15, 2022
@TimKolberger TimKolberger deleted the feat/solid branch November 15, 2022 11:43
@github-actions github-actions bot mentioned this pull request Nov 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant