-
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
Adrienne / Implemented new navigation flow for filtering payment methods in buy/sell page #8544
Changes from 21 commits
b0cfe05
02bdb37
6de1d1d
5c279ee
294e776
505cb2b
04c343a
ba26ef0
04d0132
68f59ee
85f9bf6
c5240de
68684be
852fc5e
ee2080b
819be93
82442ac
f4cb6ad
e740c7a
3fe19f4
d4dedc6
bcecdff
a27f4bc
5a52e47
705f009
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -98,12 +98,20 @@ const BuySellHeader = ({ table_type }) => { | |
placeholder={isDesktop() ? localize('Search by nickname') : localize('Search')} | ||
/> | ||
<SortDropdown /> | ||
<Icon | ||
className='buy-sell__header-row--filter' | ||
icon='IcFilter' | ||
onClick={() => general_store.showModal({ key: 'FilterModal', props: {} })} | ||
size={40} | ||
/> | ||
<div style={{ position: 'relative' }} className='buy-sell__header-row--filter'> | ||
<div | ||
className={classNames('buy-sell__header-row--filter-status', { | ||
'buy-sell__header-row--filter-status--disabled': | ||
buy_sell_store.selected_payment_method_value.length === 0, | ||
})} | ||
/> | ||
<Icon | ||
icon='IcFilter' | ||
className='buy-sell__header-row--filter-icon' | ||
onClick={() => general_store.showModal({ key: 'FilterModal', props: {} })} | ||
size={40} | ||
/> | ||
</div> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. POGGIES!! but it looks slightly bigger than the icon on the left There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeaa in the design it looks like that, if I make it smaller later QA will complain 🤣 |
||
</div> | ||
</div> | ||
</div> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -79,7 +79,26 @@ | |
border-radius: 4px; | ||
display: flex; | ||
margin-left: 0.8rem; | ||
padding: 1.3rem; | ||
|
||
&-icon { | ||
align-self: center; | ||
display: flex; | ||
padding: 1.3rem; | ||
} | ||
|
||
&-status { | ||
position: absolute; | ||
border-radius: 50px; | ||
background: #ff444f; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. can we take the color from themes instead of hardcoded value? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. suree! |
||
width: 7px; | ||
height: 7px; | ||
top: 4px; | ||
right: 4px; | ||
|
||
&--disabled { | ||
display: none; | ||
} | ||
} | ||
|
||
@include mobile { | ||
justify-self: center; | ||
|
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.
can we move this inline style to class?
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 forgot to remove this and add that in the class, thanks for spotting it 😅