Skip to content

Commit d7837a6

Browse files
siddhant1catarak
authored andcommitted
Add sorting to sketches #789 (#910)
* reselect added * Added Reselect Sorting * Refactor App * added svgs * Refactor * Fixed Issues * re: #789, update sorting styling, create sorting actions and reducers, add sort by sketch name * re #789, change names of svg icons * re: #789, use orderBy instead of sortBy, fix styling jumps
1 parent e2676ec commit d7837a6

File tree

2 files changed

+32
-24
lines changed

2 files changed

+32
-24
lines changed

client/modules/IDE/components/SketchList.jsx

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -359,20 +359,10 @@ SketchList.propTypes = {
359359
sorting: PropTypes.shape({
360360
field: PropTypes.string.isRequired,
361361
direction: PropTypes.string.isRequired
362-
}).isRequired,
363-
project: PropTypes.shape({
364-
id: PropTypes.string,
365-
owner: PropTypes.shape({
366-
id: PropTypes.string
367-
})
368-
})
362+
}).isRequired
369363
};
370364

371365
SketchList.defaultProps = {
372-
project: {
373-
id: undefined,
374-
owner: undefined
375-
},
376366
username: undefined
377367
};
378368

@@ -381,8 +371,7 @@ function mapStateToProps(state) {
381371
user: state.user,
382372
sketches: getSortedSketches(state),
383373
sorting: state.sorting,
384-
loading: state.loading,
385-
project: state.project
374+
loading: state.loading
386375
};
387376
}
388377

package-lock.json

Lines changed: 30 additions & 11 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)