Open alveshelio opened 1 year ago
Latest commit: 099f349e8b044b1a7a41bc7f946778ea6f211216
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
EventHandlerUnion
is the correct event type in Solid, I think the one from FocusEvents
has to be the same, however i'm not sure how this will impact others packages
This PR is dependent of https://github.com/solidjs-community/solid-aria/pull/73 since it has changes to
createFocusable
This is the port of Tooltip.
TODO
This is my first contribution to the project. I am looking for feedback.
I have pretty much followed what
@react-aria/tooltip
is doing.I'm having a Typescript error with the return of
createTooltipTrigger
. I'm getting this error:Looking at the signature of
onFocus
fromFocusEvents
=>onFocus?: (e: FocusEvent) => void;
in@solid-aria/types
, this is exactly the same signature as we have in@react-aria/types
. In their implementation ofonFocus
, they are not passing the event toonFocus
and it is not complaining in@react-aria
Looking at
createFocusable
interfaceFocusableResult
,focusableProps: JSX.HTMLAttributes<any>;
andonFocus
inHTMLAttributes
is indeedonFocus?: EventHandlerUnion<T, FocusEvent>;
@fabien-ml I see you worked in
createFocusable
, do you think you could help me out?