From 41095155682e39149facf648fa9c483bbda96dd0 Mon Sep 17 00:00:00 2001
From: Matthew Grove <me@mgrove.uk>
Date: Sun, 18 Sep 2022 22:33:14 +0100
Subject: [PATCH] Efficiency improvements

---
 package.json      |  2 +-
 src/Home.js       | 10 +++++++++-
 src/SearchSets.js |  4 ++--
 3 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/package.json b/package.json
index f25c187..df992b5 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "parandum",
-  "version": "2.3.2",
+  "version": "2.3.3",
   "private": true,
   "dependencies": {
     "@babel/core": "^7.16.0",
diff --git a/src/Home.js b/src/Home.js
index da67fc6..787ec69 100644
--- a/src/Home.js
+++ b/src/Home.js
@@ -46,7 +46,15 @@ export default function Home(props) {
 					<br/>
 					<div className="whats-new">
 						<h2>What's New?</h2>
-						<Collapsible open={true} transitionTime={500} easing="ease" trigger={<>v2.3.2<ArrowDropDownRoundedIcon /></>}>
+						<Collapsible open={true} transitionTime={500} easing="ease" trigger={<>v2.3.3<ArrowDropDownRoundedIcon /></>}>
+							<ul>
+								<li>Bug fixes:</li>
+								<ul>
+									<li>On the search page, all sets are shown instead of only the last 50 showing and previous ones being overwritten</li>
+								</ul>
+							</ul>
+						</Collapsible>
+						<Collapsible transitionTime={500} easing="ease" trigger={<>v2.3.2<ArrowDropDownRoundedIcon /></>}>
 							<ul>
 								<li>Bug fixes:</li>
 								<ul>
diff --git a/src/SearchSets.js b/src/SearchSets.js
index 74f367f..7101550 100644
--- a/src/SearchSets.js
+++ b/src/SearchSets.js
@@ -102,7 +102,7 @@ export default withRouter(class SearchSets extends Component {
 					sets: reload ? querySnapshot.docs : this.state.sets.concat(querySnapshot.docs),
 					selections: selections,
 					pageNumber: this.state.pageNumber + 1,
-					loadedAllSets: querySnapshot.docs.length === 0,
+					loadedAllSets: querySnapshot.docs.length < paginationFrequency,
 					loadingSets: false,
 				});
 			});
@@ -334,7 +334,7 @@ export default withRouter(class SearchSets extends Component {
 						)}
 					</div>
 					{
-						!this.state.loadedAllSets && this.state.sets.length % paginationFrequency === 0 &&
+						!this.state.loadedAllSets &&
 						<Button
 							onClick={() => this.loadSets()}
 							disabled={this.state.loadingSets}