Skip to content
Snippets Groups Projects
Commit 90b71f3b authored by Bogdan Cochior's avatar Bogdan Cochior
Browse files

fix(filters): wip: fix ordering

parent 9ac14acd
No related branches found
No related tags found
1 merge request!6Agree/Decline to work on a manuscript
...@@ -55,13 +55,13 @@ export default compose( ...@@ -55,13 +55,13 @@ export default compose(
}, },
getItems: ({ getItems: ({
filters, filters,
sortOrder, filterValues = {},
currentUser, currentUser,
dashboard, dashboard,
filterItems, filterItems,
}) => () => }) => () =>
filterItems(dashboard.all).sort((a, b) => { filterItems(dashboard.all).sort((a, b) => {
if (sortOrder === 'newest') return a.created - b.created < 0 if (filterValues.order === 'newest') return a.created - b.created < 0
return a.created - b.created > 0 return a.created - b.created > 0
}), }),
}), }),
......
...@@ -3,11 +3,6 @@ import { Menu, th } from '@pubsweet/ui' ...@@ -3,11 +3,6 @@ import { Menu, th } from '@pubsweet/ui'
import { compose, withHandlers } from 'recompose' import { compose, withHandlers } from 'recompose'
import styled from 'styled-components' import styled from 'styled-components'
const sortOptions = [
{ label: 'Newest first', value: 'newest' },
{ label: 'Oldest first', value: 'oldest' },
]
const DashboardFilters = ({ const DashboardFilters = ({
view, view,
status, status,
...@@ -39,7 +34,11 @@ const DashboardFilters = ({ ...@@ -39,7 +34,11 @@ const DashboardFilters = ({
</FilterGroup> </FilterGroup>
<FilterGroup> <FilterGroup>
<span>Sort</span> <span>Sort</span>
<Menu inline onChange={changeSort} options={sortOptions} /> <Menu
inline
onChange={changeFilterValue('order')}
options={getFilterOptions('order')}
/>
</FilterGroup> </FilterGroup>
</FiltersContainer> </FiltersContainer>
</Root> </Root>
......
...@@ -59,6 +59,10 @@ export default compose( ...@@ -59,6 +59,10 @@ export default compose(
{ label: 'All', value: 'all' }, { label: 'All', value: 'all' },
{ label: 'Submitted', value: 'submitted' }, { label: 'Submitted', value: 'submitted' },
{ label: 'Draft', value: 'draft' }, { label: 'Draft', value: 'draft' },
{ label: 'HE Invited', value: 'heInvited' },
{ label: 'HE Assigned', value: 'heAssigned' },
{ label: 'Reviewers Invited', value: 'reviewersInvited' },
{ label: 'Under Review', value: 'underReview' },
], ],
filterFn: filterValue => item => { filterFn: filterValue => item => {
if (filterValue === 'all' || filterValue === '') return true if (filterValue === 'all' || filterValue === '') return true
...@@ -86,6 +90,13 @@ export default compose( ...@@ -86,6 +90,13 @@ export default compose(
return false return false
}, },
}, },
order: {
options: [
{ label: 'Newest first', value: 'newest' },
{ label: 'Oldest first', value: 'oldest' },
],
filterFn: () => () => true,
},
}), }),
withContext( withContext(
{ {
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment