docs: fix font docs #2849
Merged
docs: fix font docs #2849
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
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 fbca6b4:
|
tujoworker
approved these changes
Nov 8, 2023
langz
commented
Nov 8, 2023
There was a problem hiding this comment.
Looks great, thanks for purposing a better alternative @tujoworker 🥇
tujoworker
pushed a commit
that referenced
this pull request
Nov 12, 2023
## [10.13.0](v10.12.0...v10.13.0) (2023-11-12) ### 🐛 Bug Fixes * **Autocomplete:** ensure correct value selection during data change ([#2863](#2863)) ([d229ec2](d229ec2)) * **Autocomplete:** ensure on_change is firered during dataset update ([266ab6a](266ab6a)) * **PhoneNumber:** ensure correct selection of value ([#2852](#2852)) ([a20a764](a20a764)) * remove nullish operator to support Storybook v6 ([#2873](#2873)) ([f0bab63](f0bab63)) * typescript react type resolution in monorepos ([#2866](#2866)) ([44f04f3](44f04f3)) ### ✨ Features * **Accordion:** change sbanken expand icon to chevron ([#2854](#2854)) ([a7cb8b9](a7cb8b9)) * add "use client" to components that use client side only React APIs (support for RSC) ([#2874](#2874)) ([a84e962](a84e962)) * **Button:** change sbanken primary button text color to white ([#2855](#2855)) ([1571138](1571138)) * **Typography:** change sbanken bold to medium ([#2808](#2808)) ([4b0f930](4b0f930)) ### 📝 Documentation * **About:** add Snorre/Henrik/Ynge as contributors ([#2885](#2885)) ([fff72da](fff72da)) * **Autocomplete:** adds default value to keep_value props ([#2867](#2867)) ([7cae1fd](7cae1fd)) * fix font docs ([#2849](#2849)) ([13beec9](13beec9)) * **Ingress:** adds Ingress to docs ([#2879](#2879)) ([f771f3d](f771f3d)) * **Lead:** adds Lead to docs ([#2877](#2877)) ([0c0da7f](0c0da7f))
|
🎉 This PR is included in version 10.13.0 🎉 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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When running
/quickguide-designer/fonts/#font-weightslocally, I get the following errors:What's happening is that
\nsomehow gets inserted when a prettier formats a<p>, leading to the content being wrapped in a<p>, so that we get<p>'s inside of<p>'s.Here's a link to a related issue: storybookjs/storybook#18921
If this would only lead to the console error pasted above, it all would be fine, but in some scenarios when wrapping a
pin ap, the visual result is actually different(so what's displayed in the docs is actually not what we want), compare the fix vs. what's on main. Screenshot as reference: