Skip to content
This repository was archived by the owner on Sep 2, 2023. It is now read-only.

Commit 8deb367

Browse files
authored
Merge pull request #637 from aminroosta/aminfix
Disable selecting closed markets
2 parents aba3250 + 55aaa4b commit 8deb367

File tree

2 files changed

+14
-7
lines changed

2 files changed

+14
-7
lines changed

src/asset-picker/AssetPickerItem.js

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,14 @@ export default class AssetPickerItem extends PureComponent {
1919
selected: false,
2020
};
2121

22-
onRowClicked = (e: SyntheticEvent) => {
23-
const { asset, onClose, onSelect } = this.props;
24-
onSelect(asset.symbol);
25-
onClose();
26-
e.stopPropagation();
27-
};
22+
onRowClicked = (e: SyntheticEvent) => {
23+
const { asset, onClose, onSelect } = this.props;
24+
if (asset.isOpen) {
25+
onSelect(asset.symbol);
26+
onClose();
27+
}
28+
e.stopPropagation();
29+
};
2830

2931
onStarClicked = (e: SyntheticEvent) => {
3032
const { asset, onToggleWatchlistItem } = this.props;
@@ -35,7 +37,7 @@ export default class AssetPickerItem extends PureComponent {
3537
render() {
3638
const { asset, selected } = this.props;
3739
const { isOpen, isInWatchlist } = asset;
38-
const classes = classnames('asset-picker-item', selected);
40+
const classes = classnames('asset-picker-item', { selected, closed: !isOpen });
3941

4042
return (
4143
<tr

styles/_asset-picker.scss

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,11 @@
5454
}
5555

5656
.asset-picker-item {
57+
&.closed {
58+
cursor: default;
59+
color: $COLOR_INVERSE_TEXT;
60+
td:first-child { cursor: pointer; }
61+
}
5762
td {
5863
text-align: left;
5964
padding: 0.25em;

0 commit comments

Comments
 (0)