-
Notifications
You must be signed in to change notification settings - Fork 55
Conversation
<Grid variables={{ gridGap: '2rem' }} columns="1"> | ||
{children} | ||
</Grid> | ||
</> |
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.
I don't see any reason to use there Flex
...
@@ -79,7 +78,7 @@ class ComponentExample extends React.Component<ComponentExampleProps, ComponentE | |||
knobs: this.getDefaultKnobsValue(), | |||
showCode: this.isActiveHash(), | |||
componentVariables: {}, | |||
showRtl: examplePath && examplePath.endsWith('rtl') ? true : false, | |||
showRtl: examplePath && examplePath.endsWith('rtl'), |
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.
Simplified condition
@@ -56,7 +56,6 @@ interface ComponentExampleState { | |||
const childrenStyle: React.CSSProperties = { | |||
paddingTop: 0, | |||
paddingBottom: '10px', | |||
maxWidth: pxToRem(500), |
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.
</Flex.Item> | ||
|
||
{/* TODO: bring back the right floating menu | ||
<Grid columns="auto 300px" styles={{ justifyContent: 'normal', justifyItems: 'stretch' }}> |
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.
Fixes this issue:
@@ -11,7 +11,7 @@ const [notTruncatedText, truncatedText] = [ | |||
) | |||
|
|||
const TextExampleTruncated = () => ( | |||
<div style={{ overflow: 'hidden', textOverflow: 'ellipsis' }}> | |||
<div style={{ overflow: 'hidden', textOverflow: 'ellipsis', width: 500 }}> |
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.
Generated by 🚫 dangerJS |
Codecov Report
@@ Coverage Diff @@
## master #1019 +/- ##
=======================================
Coverage 81.52% 81.52%
=======================================
Files 675 675
Lines 8756 8756
Branches 1558 1558
=======================================
Hits 7138 7138
Misses 1602 1602
Partials 16 16 Continue to review full report at Codecov.
|
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.
Changes looks good to me! Are we planning to move the dropdown for switching teams back in the Sidebar
or at least put some absolute positioning in order to prevent scrolling on each changing?
As I know there is a plan to move back theme selector |
Continues work from #1014.