-
Notifications
You must be signed in to change notification settings - Fork 80
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃挕 Proposal - Allow to edit which heading element is used in components #1879
Comments
I can create a PR for this, just checking if it would be okay to do so. |
Hey @YvesRijckaert! Sorry for the delay. I think it totally makes sense, I would even say that we have a bug right now, since it might actually create an incorrect structure. We should consider changing the default to something different than |
Rendering the |
@denkristoffer I think we use this component in a lot of places, and often we would need more flexibility when it comes to which heading element to use.. I ran into this issue when using this component in compose. |
I am all for being able to change the element. I wanted to add information that |
Didn't know that the browsers didn't implement this, good to know. |
Coming back to this, I guess an API that fits with our existing guidelines would be something like passing an object to <Card headingProps={{ as: 'h3' }} /> What do you think? |
Marking issue as stale since there was no acitivty for 30 days |
this was fixed |
Forma 36 contribution proposal
The problem
When using the EntryCard component, I wanted to change the heading element from h1 to h3. But this does not seem possible right now.
forma-36/packages/components/card/src/EntryCard/EntryCard.tsx
Line 36 in 1f3b3eb
The proposed solution
Being able to change the heading element with a prop.
The text was updated successfully, but these errors were encountered: