-
-
Notifications
You must be signed in to change notification settings - Fork 247
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
feat(suite): quick-action refactotring, simplification of the UI and adding a update icon #14014
Conversation
2e80652
to
c8ebe6f
Compare
af0d9c1
to
a88acab
Compare
@@ -0,0 +1,4 @@ | |||
export const Update = () => { | |||
// Todo: this will be implemented in future Pull Request | |||
return <></>; |
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.
a88acab
to
5472a50
Compare
packages/components/src/components/ComponentWithSubIcon/ComponentWithSubIcon.tsx
Show resolved
Hide resolved
packages/components/src/components/ComponentWithSubIcon/ComponentWithSubIcon.stories.tsx
Show resolved
Hide resolved
packages/components/src/components/ComponentWithSubIcon/ComponentWithSubIcon.stories.tsx
Outdated
Show resolved
Hide resolved
packages/components/src/components/ComponentWithSubIcon/ComponentWithSubIcon.tsx
Outdated
Show resolved
Hide resolved
packages/suite/src/components/suite/layouts/SuiteLayout/Sidebar/QuickActions/CustomBackend.tsx
Outdated
Show resolved
Hide resolved
b0bab92
to
53cbe39
Compare
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.
ship it
e58203f
to
3047b6d
Compare
const SubIconWrapper = styled.div<SubIconWrapperProps>` | ||
position: absolute; | ||
|
||
right: -${({ $subIconSize }) => $subIconSize / 2 + 3}px; |
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.
why +3?
|
||
export const Update = () => { | ||
const theme = useTheme(); | ||
// const { translationString } = useTranslation(); |
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.
leftover?
d9652c9
to
635a43a
Compare
e00ccc9
to
11d8db0
Compare
…adding a update icon chore(components): add frameProps for ComponentWithSubIcon
11d8db0
to
17a4762
Compare
QA NOK see Trezor icon its not properly aligned in white square Info:
|
@bosomt Interesting, nice catch. Hovever its not happening to me on Linux & Desktop. I have asked somebody with Mac to investigate. |
QA OK overall it looks great, if anyone else complains about that icon i will create issue Info:
|
@peter-sanderson created follow up issue :) havent noticed it till now ;) #14497 |
This PR:
ComponentWithIcon
to abstract the component with a little icon in the cornerPart of the #13990