Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

patch: export tooltip ref #454

Merged
merged 2 commits into from
Jul 20, 2023
Merged

patch: export tooltip ref #454

merged 2 commits into from
Jul 20, 2023

Conversation

charlieblevins
Copy link
Contributor

@charlieblevins charlieblevins commented Jul 12, 2023

This type makes typescript happy about the forceAlign method on a tooltip ref, so I thought it would be good to go on and export it.

This makes this example compile in typescript.

@codecov
Copy link

codecov bot commented Jul 13, 2023

Codecov Report

Merging #454 (f047e84) into master (c5374d7) will not change coverage.
The diff coverage is n/a.

❗ Current head f047e84 differs from pull request most recent head 27bb7a9. Consider uploading reports for the commit 27bb7a9 to get more accurate results

@@            Coverage Diff            @@
##            master      #454   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines           26        26           
  Branches        10        10           
=========================================
  Hits            26        26           

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

@yoyo837
Copy link
Member

yoyo837 commented Jul 13, 2023

CI failed.

@charlieblevins
Copy link
Contributor Author

CI failed.

Thanks. Should be fixed now.

@yoyo837 yoyo837 merged commit 595be48 into react-component:master Jul 20, 2023
@charlieblevins charlieblevins deleted the tooltip-ref branch July 20, 2023 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants