Skip to content

Commit 184b2ec

Browse files
author
Ryan
committed
tri-click sorts
1 parent 629b5f8 commit 184b2ec

File tree

2 files changed

+17
-11
lines changed

2 files changed

+17
-11
lines changed

src/components/browse.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -57,11 +57,10 @@ class Browse extends Component {
5757
}
5858

5959
handleSortButtonChange(name, sortStatus) {
60-
console.log(name, sortStatus);
61-
if (name === this.state.sortValue) {
62-
this.setState({sortValue: null})
63-
} else this.setState({
64-
sortStatus: sortStatus
60+
if (sortStatus === false) {name = null}
61+
this.setState({
62+
sortStatus: sortStatus,
63+
sortValue: name
6564
})
6665
}
6766

src/components/sortValueButton.js

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,9 @@ class SortValueButton extends Component {
2929
this.setState({sortStatus: sortStatus}, () => this.pushStateUp(name))
3030
}
3131

32-
pushStateUp() {
33-
console.log(name);
34-
// this.props.onSortButtonChange(name, sortStatus)
32+
pushStateUp(name) {
33+
// console.log(name);
34+
this.props.onSortButtonChange(name, this.state.sortStatus)
3535
}
3636

3737
flipArrow(sortStatus) {
@@ -50,13 +50,20 @@ class SortValueButton extends Component {
5050
const buttonName = this.props.name;
5151
const buttonValue = this.props.buttonValue;
5252

53+
const button = (sortValue === buttonName) ? (
54+
<b><button name={buttonName} className="sortButton" onClick={this.handleSortButtonChange}>
55+
{buttonValue} {this.flipArrow(this.state.sortStatus)}
56+
</button></b>
57+
) : (
58+
<button name={buttonName} className="sortButton" onClick={this.handleSortButtonChange}>
59+
{buttonValue}
60+
</button>
61+
)
5362

5463
return (
5564
<div className="row sort-row">
5665
<div className="col">
57-
<button name={buttonName} className="sortButton" onClick={this.handleSortButtonChange}>
58-
{buttonValue} {this.flipArrow(this.state.sortStatus)}
59-
</button>
66+
{button}
6067
</div>
6168
</div>
6269
)

0 commit comments

Comments
 (0)