From 2092d778f05915c893ca58ab08b76e13b0bb93e9 Mon Sep 17 00:00:00 2001 From: Saturnino Abril Date: Sat, 14 Apr 2018 01:30:15 +0800 Subject: [PATCH] [MM-9216] Add date separator per search result item (#1583) * add date separator per search result item * rebase to latest and make each post of recent, flagged and search results have date separator --- app/screens/search/search.js | 14 ++------------ app/selectors/post_list.js | 9 +-------- 2 files changed, 3 insertions(+), 20 deletions(-) diff --git a/app/screens/search/search.js b/app/screens/search/search.js index f20717a42..6dcb567f2 100644 --- a/app/screens/search/search.js +++ b/app/screens/search/search.js @@ -24,6 +24,7 @@ import DateHeader from 'app/components/post_list/date_header'; import FormattedText from 'app/components/formatted_text'; import Loading from 'app/components/loading'; import PostListRetry from 'app/components/post_list_retry'; +import PostSeparator from 'app/components/post_separator'; import SafeAreaView from 'app/components/safe_area_view'; import SearchBar from 'app/components/search_bar'; import StatusBar from 'app/components/status_bar'; @@ -325,7 +326,7 @@ export default class Search extends PureComponent { let separator; const nextPost = postIds[index + 1]; if (nextPost && nextPost.indexOf(DATE_LINE) === -1) { - separator = this.renderPostSeparator(); + separator = ; } return ( @@ -355,17 +356,6 @@ export default class Search extends PureComponent { ); }; - renderPostSeparator = () => { - const {theme} = this.props; - const style = getStyleFromTheme(theme); - - return ( - - - - ); - }; - renderSectionHeader = ({section}) => { const {theme} = this.props; const {title} = section; diff --git a/app/selectors/post_list.js b/app/selectors/post_list.js index 33c9ddc4a..a8e7c447c 100644 --- a/app/selectors/post_list.js +++ b/app/selectors/post_list.js @@ -93,7 +93,6 @@ export function makePreparePostIdsForSearchPosts() { } const out = []; - let lastDate = null; for (let i = 0; i < posts.length; i++) { const post = posts[i]; @@ -106,17 +105,11 @@ export function makePreparePostIdsForSearchPosts() { continue; } - // Push on a date header if the last post was on a different day than the current one const postDate = new Date(post.create_at); - if (!lastDate || lastDate.toDateString() !== postDate.toDateString()) { - out.push(DATE_LINE + postDate.toString()); - lastDate = postDate; - } - out.push(post.id); + out.push(DATE_LINE + postDate.toString(), post.id); } - // Flip it back to newest to oldest return out; } );