-
Notifications
You must be signed in to change notification settings - Fork 457
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Copiable text component - Closes #6224 #6275
Conversation
c1189cd
to
bd6dec5
Compare
|
||
interface Props { | ||
name: string; | ||
size?: 's' | 'm' | 'l' | 'xl'; |
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.
these property i think you can export Props from Text and reference rather than redefining
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.
Yes it should be exported from Text/index.tsx
and then union those types here.
framework-plugins/lisk-framework-dashboard-plugin/src/ui/components/CopyText/index.tsx
Outdated
Show resolved
Hide resolved
framework-plugins/lisk-framework-dashboard-plugin/src/ui/components/CopyText/index.tsx
Outdated
Show resolved
Hide resolved
...work-plugins/lisk-framework-dashboard-plugin/src/ui/components/CopyText/CopyText.module.scss
Outdated
Show resolved
Hide resolved
} | ||
|
||
.clickableRow { | ||
float: left; |
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.
you can use flex
here
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.
when using flex, Text and Icon components don't align side by side and icon goes to the next line
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.
the flex needs to be added to the container
framework-plugins/lisk-framework-dashboard-plugin/src/ui/components/CopyText/index.tsx
Outdated
Show resolved
Hide resolved
|
||
interface Props { | ||
name: string; | ||
size?: 's' | 'm' | 'l' | 'xl'; |
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.
Yes it should be exported from Text/index.tsx
and then union those types here.
♻️ Hide copy icon on copied ♻️ Clear timeout
56d521c
to
77fb864
Compare
77fb864
to
d1283b2
Compare
What was the problem?
This PR resolves #6224
How was it solved?
How was it tested?
npm run start:web