diff --git a/CHANGELOG.md b/CHANGELOG.md index e3a906c..c2948db 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,8 @@ - `path` and `permalink` now end with a `/` - Generate table of contents for each page - Add `section` to a page Tera context if there is one +- Reverse `order` sorting to be more intuitive: they are now desc, think of them +as 1st, 2nd in the list ## 0.0.6 (2017-05-24) diff --git a/src/content/sorting.rs b/src/content/sorting.rs index 68a933c..247f1a9 100644 --- a/src/content/sorting.rs +++ b/src/content/sorting.rs @@ -38,7 +38,7 @@ pub fn sort_pages(pages: Vec, sort_by: SortBy) -> (Vec, Vec) { cannot_be_sorted.push(page); } } - can_be_sorted.sort_by(|a, b| b.meta.order().cmp(&a.meta.order())); + can_be_sorted.sort_by(|a, b| a.meta.order().cmp(&b.meta.order())); (can_be_sorted, cannot_be_sorted) }, @@ -120,10 +120,10 @@ mod tests { create_page_with_order(1), ]; let (pages, _) = sort_pages(input, SortBy::Order); - // Should be sorted by date - assert_eq!(pages[0].clone().meta.order.unwrap(), 3); + // Should be sorted by order + assert_eq!(pages[0].clone().meta.order.unwrap(), 1); assert_eq!(pages[1].clone().meta.order.unwrap(), 2); - assert_eq!(pages[2].clone().meta.order.unwrap(), 1); + assert_eq!(pages[2].clone().meta.order.unwrap(), 3); } #[test]