Conversation
layershifter
requested review from
dvdzkwsk,
kuzhelov,
levithomason,
miroslavstastny and
mnajdova
as code owners
Codecov Report
@@ Coverage Diff @@
## master #1716 +/- ##
==========================================
- Coverage 70.97% 70.95% -0.02%
==========================================
Files 860 860
Lines 7142 7144 +2
Branches 2072 2056 -16
==========================================
Hits 5069 5069
- Misses 2067 2069 +2
Partials 6 6
Continue to review full report at Codecov.
|
|
@codepretty can you please check these changes? 🙏 |
| fontWeight: v.contentFontWeight, | ||
|
|
||
| ...(p.size === 'small' && { | ||
| fontSize: v.sizeSmallFontSize, |
There was a problem hiding this comment.
I think this need to have a line-height set for the small size and one for the default (medium) size to ensure that no descender are cut off...
There was a problem hiding this comment.
Adding the medium line-height as a default will also fix this - #1376
…at/button-size # Conflicts: # CHANGELOG.md
| /> | ||
| <ComponentExample | ||
| title="Size" | ||
| description="A button can have assorted sizes." |
There was a problem hiding this comment.
It would be nice to consolidate the description of the size in the components' examples. We have different for Avatar, Text, Button.. :\
| minWidth: height, | ||
| ...(p.circular && | ||
| !p.text && { | ||
| minWidth: v.height, |
There was a problem hiding this comment.
Hmm, shouldn't this be updated depending on the size? It seems like a special case..
There was a problem hiding this comment.
<Button size="small" circular />
<Button circular />Yep, it should
4 participants
Fixes #1376.
Fixes #1671.
BREAKING CHANGES
paddingLeftRightValuewas renamed topadding.Changes
This PR adds a new
sizeprop and styles forsize=small.I also made a small refactor to use
p&vpattern.