From 3fa393354b90cee5a349ab813889fe2d8cd59289 Mon Sep 17 00:00:00 2001
From: Alex Yatskov <alex@foosoft.net>
Date: Fri, 27 Jan 2017 09:01:12 -0800
Subject: [PATCH] store max number of results to display

---
 ext/bg/js/options-form.js |  6 ++++--
 ext/bg/js/options.js      |  3 ++-
 ext/bg/options.html       | 11 ++++++++---
 3 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/ext/bg/js/options-form.js b/ext/bg/js/options-form.js
index eafb3e09..92596773 100644
--- a/ext/bg/js/options-form.js
+++ b/ext/bg/js/options-form.js
@@ -30,9 +30,10 @@ function getFormData() {
 
         optionsNew.general.autoStart = $('#activate-on-startup').prop('checked');
         optionsNew.general.audioPlayback = $('#audio-playback-buttons').prop('checked');
-        optionsNew.general.softKatakana = $('#soft-katakana-search').prop('checked');
         optionsNew.general.groupResults = $('#group-terms-results').prop('checked');
+        optionsNew.general.softKatakana = $('#soft-katakana-search').prop('checked');
         optionsNew.general.showAdvanced = $('#show-advanced-options').prop('checked');
+        optionsNew.general.maxResults = parseInt($('#max-displayed-results').val(), 10);
 
         optionsNew.scanning.requireShift = $('#hold-shift-to-scan').prop('checked');
         optionsNew.scanning.selectText = $('#select-matched-text').prop('checked');
@@ -86,9 +87,10 @@ $(document).ready(() => {
     optionsLoad().then(options => {
         $('#activate-on-startup').prop('checked', options.general.autoStart);
         $('#audio-playback-buttons').prop('checked', options.general.audioPlayback);
-        $('#soft-katakana-search').prop('checked', options.general.softKatakana);
         $('#group-terms-results').prop('checked', options.general.groupResults);
+        $('#soft-katakana-search').prop('checked', options.general.softKatakana);
         $('#show-advanced-options').prop('checked', options.general.showAdvanced);
+        $('#max-displayed-results').val(options.general.maxResults);
 
         $('#hold-shift-to-scan').prop('checked', options.scanning.requireShift);
         $('#select-matched-text').prop('checked', options.scanning.selectText);
diff --git a/ext/bg/js/options.js b/ext/bg/js/options.js
index dcf9d075..066ed62c 100644
--- a/ext/bg/js/options.js
+++ b/ext/bg/js/options.js
@@ -22,8 +22,9 @@ function optionsSetDefaults(options) {
         general: {
             autoStart: true,
             audioPlayback: true,
-            softKatakana: true,
             groupResults: true,
+            softKatakana: true,
+            maxResults: 32,
             showAdvanced: false
         },
 
diff --git a/ext/bg/options.html b/ext/bg/options.html
index d2f3ebb8..bef053d0 100644
--- a/ext/bg/options.html
+++ b/ext/bg/options.html
@@ -33,16 +33,21 @@
                     <label><input type="checkbox" id="audio-playback-buttons"> Audio playback buttons</label>
                 </div>
 
+                <div class="checkbox">
+                    <label><input type="checkbox" id="group-terms-results"> Group term results</label>
+                </div>
+
                 <div class="checkbox options-advanced">
                     <label><input type="checkbox" id="soft-katakana-search"> Soft Katakana search</label>
                 </div>
 
                 <div class="checkbox">
-                    <label><input type="checkbox" id="group-terms-results"> Group term results</label>
+                    <label><input type="checkbox" id="show-advanced-options"> Show advanced options</label>
                 </div>
 
-                <div class="checkbox">
-                    <label><input type="checkbox" id="show-advanced-options"> Show advanced options</label>
+                <div class="form-group options-advanced">
+                    <label for="max-displayed-results">Maximum displayed results</label>
+                    <input type="number" min="1" id="max-displayed-results" class="form-control">
                 </div>
             </div>