-
Notifications
You must be signed in to change notification settings - Fork 299
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
Mahdi.aryayi / Fix design bugs #447
Conversation
packages/trader/src/Modules/Account/Sections/Profile/PersonalDetails/personal-details.jsx
Outdated
Show resolved
Hide resolved
packages/trader/src/Modules/Account/Sections/Profile/PersonalDetails/personal-details.jsx
Outdated
Show resolved
Hide resolved
packages/trader/src/App/Components/Elements/SideMenu/side-menu-headers.jsx
Show resolved
Hide resolved
packages/trader/src/Modules/Account/Sections/Profile/PersonalDetails/personal-details.jsx
Outdated
Show resolved
Hide resolved
packages/trader/src/Modules/Account/Sections/Profile/PersonalDetails/personal-details.jsx
Outdated
Show resolved
Hide resolved
4761016
to
a0401e3
Compare
packages/trader/src/Modules/Account/Sections/Profile/PersonalDetails/personal-details.jsx
Outdated
Show resolved
Hide resolved
3337d42
to
90bed41
Compare
@mahdiaryayi-fs travis failing |
className='account-form__footer-btn btn--primary--default' | ||
className={classNames('account-form__footer-btn', { | ||
'btn--primary--default': !is_submit_success, | ||
'btn--primary--green' : is_submit_success, |
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.
@mahdiaryayi-fs ahh just one more suggestion: could you make this changes in the <Button />
component itself? this should be applied to all usages of Button, not only on this page 😄
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.
@kelly-binary I checked <Button />
use cases in deriv trader. The component is style agnostic. We are passing different classNames in different cases. If I add this logic by adding btn--primary--default
and btn--primary--green
inside <Button />
, it doesn't work in other places that we are not using primary button classes and it may cause styling conflict.
4a9dded
to
90f6370
Compare
Fix design bugs