Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

History fixes and misc about page fixes #5559

Closed
wants to merge 4 commits into from
Closed

History fixes and misc about page fixes #5559

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
f06f036
The big history-fixer-upper
bsclifton Nov 11, 2016
658b23c
Remove unnecessary/broken/useless context menu items for about pages
bsclifton Nov 11, 2016
3028747
Update icon for about pages to fa-list (except for about:newtab)
bsclifton Nov 11, 2016
8c17293
Had to update history to be a list (after the PR converting sites to …
bsclifton Nov 11, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions app/common/lib/historyUtil.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,11 @@ const sortTimeDescending = (left, right) => {
}

module.exports.getHistory = (sites) => {
sites = makeImmutable(sites) || new Immutable.List()
return sites.filter((site) => siteUtil.isHistoryEntry(site))
.sort(sortTimeDescending)
.slice(-aboutHistoryMaxEntries)
sites = makeImmutable(sites) || new Immutable.Map()
return sites.toList()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++

.filter((site) => siteUtil.isHistoryEntry(site))
.sort(sortTimeDescending)
.slice(-aboutHistoryMaxEntries)
}

const getDayString = (entry, locale) => {
Expand Down