From 728b0cb35f9e66b7b64ac1c3538b473fc646c15f Mon Sep 17 00:00:00 2001
From: Shin'ya UEOKA <ueokande@i-beam.org>
Date: Tue, 21 May 2019 23:33:31 +0000
Subject: [PATCH] Fix completion order

---
 src/background/usecases/CompletionsUseCase.ts | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/background/usecases/CompletionsUseCase.ts b/src/background/usecases/CompletionsUseCase.ts
index 705555b..8cd4f32 100644
--- a/src/background/usecases/CompletionsUseCase.ts
+++ b/src/background/usecases/CompletionsUseCase.ts
@@ -38,6 +38,9 @@ export default class CompletionsUseCase {
   }
 
   async queryOpen(name: string, keywords: string): Promise<CompletionGroup[]> {
+    // TODO This logic contains view entities.  They should be defined on
+    // content script
+
     let settings = await this.settingRepository.get();
     let groups: CompletionGroup[] = [];
 
@@ -195,7 +198,7 @@ export default class CompletionsUseCase {
       .map(pages => filters.filterByPathname(pages, COMPLETION_ITEM_LIMIT))
       .map(pages => filters.filterByOrigin(pages, COMPLETION_ITEM_LIMIT))[0]
       .sort((x: HistoryItem, y: HistoryItem): number => {
-        return Number(x.visitCount) - Number(y.visitCount);
+        return Number(y.visitCount) - Number(x.visitCount);
       })
       .slice(0, COMPLETION_ITEM_LIMIT);
     return histories.map(page => ({