-
Notifications
You must be signed in to change notification settings - Fork 0
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
chore(icons): remove all uikit icons #163
Conversation
92151c2
to
40b69ab
Compare
40b69ab
to
291ce47
Compare
291ce47
to
0f9b607
Compare
0f9b607
to
2e1aa33
Compare
2e1aa33
to
b91d116
Compare
b91d116
to
594f6a9
Compare
594f6a9
to
694e862
Compare
fdc968b
to
8d556c4
Compare
8d556c4
to
9389e14
Compare
@@ -1,7 +1,8 @@ | |||
import React, { useEffect, useState } from 'react'; | |||
import Resizer from 'react-image-file-resizer'; | |||
import UIkit from 'uikit'; | |||
import { Icon } from 'src/components/utils'; | |||
import PencilIcon from 'assets/icons/pencil.svg'; | |||
import {} from 'src/components/utils'; |
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.
?
@@ -1,7 +1,7 @@ | |||
export interface HeaderConnectedItemProps { | |||
href: string; | |||
badge?: string; | |||
icon?: string; | |||
icon?: JSX.Element; |
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.
React.reactNode; ?
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.
Alors ici on sait qu'on veut passer des icônes SVG du style <UserIcon />
, du coup on sait que ce sera forcément un JSX.Element
. React.ReactNode
englobe aussi les string et les autres types de node, etc...
@@ -8,7 +8,7 @@ interface ButtonPostProps { | |||
color?: string; | |||
action?: (event?: FormEvent) => Promise<void>; | |||
style?: '' | UIKIT_BUTTON_STYLES_SPEC; | |||
icon?: string; | |||
icon?: JSX.Element; |
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.
idem
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.
Alors ici on sait qu'on veut passer des icônes SVG du style , du coup on sait que ce sera forcément un JSX.Element. React.ReactNode englobe aussi les string et les autres types de node, etc...
import { StyledButtonIcon } from './ButtonIcon.styles'; | ||
|
||
interface ButtonIconProps { | ||
icon: JSX.Element; |
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.
idem
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.
Alors ici on sait qu'on veut passer des icônes SVG du style , du coup on sait que ce sera forcément un JSX.Element. React.ReactNode englobe aussi les string et les autres types de node, etc...
…#164) * chore(icons): remove all uikit icons * started public offer form * [EN-6475] feat(offer form): separate public and private offer form * [EN-6475] tests(offer form): update tests * [EN-6475] fix(offer form): fix modal scroll to top * [EN-6475] feat(offer form): change wording * [EN-6475] chore(icons): manage auto margin if icons in buttons
No description provided.