Skip to content

Commit

Permalink
Merge pull request #1606 from AmbireTech/default-token-transfer#1580
Browse files Browse the repository at this point in the history
Fix default token on transfer screen
  • Loading branch information
Ivshti authored May 23, 2024
2 parents 988a80c + c5fc98f commit 05ece99
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
1 change: 1 addition & 0 deletions src/components/Wallet/Transfer/Send/Send.js
Original file line number Diff line number Diff line change
Expand Up @@ -359,6 +359,7 @@ const Send = ({
<div className={styles.content}>
<Select
searchable
preventDefaultFirst={asset!==null}
defaultValue={asset}
items={sortedAssetsItems}
onChange={({ value }) => setAsset(value)}
Expand Down
7 changes: 4 additions & 3 deletions src/components/common/Select/Select.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@ const Select = ({
dragTarget,
drop,
draggableHeader,
displayDraggableHeader
displayDraggableHeader,
preventDefaultFirst
}) => {
const ref = useRef()
const hiddenTextInput = useRef()
Expand Down Expand Up @@ -67,8 +68,8 @@ const Select = ({
)

useEffect(() => {
const item = items.find((item) => item.value === defaultValue) || items[0]
if (item && selectedItem.value !== item.value) selectItem(item)
const item = items.find((item) => item.value === defaultValue) || (preventDefaultFirst ? {} : items[0])
if (item && (selectedItem.value !== item.value || selectedItem.label !== item.label)) selectItem(item)
}, [items, defaultValue, selectedItem, selectItem])

useEffect(() => {
Expand Down

0 comments on commit 05ece99

Please sign in to comment.