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

fix(Anchor): allow rel property in TS #1849

Merged
merged 1 commit into from
Jan 4, 2023

Conversation

langz
Copy link
Contributor

@langz langz commented Jan 4, 2023

Seems like HTMLProps includes more props than HTMLAttributes, rel being one of them.

Or maybe we should use React.ComponentPropsWithoutRef<"a">? 🤔
https://github.com/typescript-cheatsheets/react/blob/main/docs/advanced/patterns_by_usecase.md#componentprops

@langz langz force-pushed the fix/use-HTMLProps-instead-of-HTMLAttributes branch from fcb6423 to 4d3c790 Compare January 4, 2023 14:16
@langz langz changed the title fix(Types): extend by HTMLProps, not HTMLAttributes fix(Anchor): allow rel property in TS Jan 4, 2023
@langz langz requested a review from tujoworker January 4, 2023 14:20
@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 4, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit dc06c80:

Sandbox Source
eufemia-starter Configuration

Copy link
Member

@tujoworker tujoworker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's interesting 🤔 we may actually switch over to use ComponentPropsWithoutRef. It sounds easier to use/read than HTMLProps etc.

@tujoworker
Copy link
Member

Do you found the commit that made the change (from prev version) by the way?

@langz
Copy link
Contributor Author

langz commented Jan 4, 2023

Do you found the commit that made the change (from prev version) by the way?

#1715 and the change https://github.com/dnbexperience/eufemia/pull/1715/files#diff-69edde89b21515b1a726d1403a98b761a9f6aeb488366bb84ea5ef565102c47aR36, I think.

@gatsby-cloud
Copy link

gatsby-cloud bot commented Jan 4, 2023

✅ DNB Eufemia Portal deploy preview ready

@langz langz force-pushed the fix/use-HTMLProps-instead-of-HTMLAttributes branch from 4d3c790 to dc06c80 Compare January 4, 2023 14:46
@langz langz merged commit e3b1be1 into main Jan 4, 2023
@langz langz deleted the fix/use-HTMLProps-instead-of-HTMLAttributes branch January 4, 2023 14:52
tujoworker pushed a commit that referenced this pull request Jan 4, 2023
## [9.45.1](v9.45.0...v9.45.1) (2023-01-04)

### Bug Fixes

* **Anchor:** allow rel property in TS ([#1849](#1849)) ([e3b1be1](e3b1be1))
@tujoworker
Copy link
Member

🎉 This PR is included in version 9.45.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants