-
Notifications
You must be signed in to change notification settings - Fork 55
Conversation
Codecov Report
@@ Coverage Diff @@
## master #205 +/- ##
=======================================
Coverage 89.23% 89.23%
=======================================
Files 50 50
Lines 836 836
Branches 130 119 -11
=======================================
Hits 746 746
Misses 86 86
Partials 4 4
Continue to review full report at Codecov.
|
@@ -3,21 +3,21 @@ import { Text } from '@stardust-ui/react' | |||
|
|||
const TextSizesExampleShorthand = () => ( | |||
<div> | |||
<Text size="xs">Dicta voluptatum dolorem.</Text> | |||
<Text size="xs" content="Dicta voluptatum dolorem." /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|
||
const TextSizesExample = () => ( | ||
<div> | ||
<Text size="xs">Dicta voluptatum dolorem.</Text> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually, this raises another point related to the values used for sizes. The problem is that it is hard to intuitively reason for the client about, say, which one is bigger: xl
or 2x
. Would propose to use values that are more intuitive in this sense (and address the issue by means of separate PR).
Quite similar sentiments are expressed in #136.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Completely agree. My next PR is going to use a theme-based enum to show the appropriate sizes.
What I'm still not sure about is what to do for naming and how that could vary among themes...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changelog.md
should be updated as this PR introduces publicly visible changes
Updating the text doc to have all examples & have accessibility info
Before
After