kiwix/kiwix-desktop
1
0
Fork 0
mirror of https://github.com/kiwix/kiwix-desktop.git synced 2025-09-21 02:51:26 -04:00
Code Issues Packages Projects Releases Wiki Activity

The max page should be nbPages-1, not nbPages.

Browse Source
This commit is contained in:
Matthieu Gautier 2018-11-02 17:08:40 +01:00
parent 3311423ca1
commit f81061f1f4
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
resources/texts/_contentManager.html
View File

@ -69,7 +69,7 @@ function init() {
changePage : function(delta) {
var newPage = contentManager.currentPage+delta;
if (newPage < 0) newPage = 0;
if (newPage > contentManager.nbPages) newPage = contentManager.nbPages;
if (newPage > contentManager.nbPages-1) newPage = contentManager.nbPages-1;
contentManager.currentPage = newPage;
},
downloadBook : function(book) {
@ -220,9 +220,9 @@ button {
<div class="footer">
<button v-on:click="contentManager.currentPage = 0">First</button>
<button v-on:click="changePage(-1)">Previous</button>
{{ contentManager.currentPage+1 }} / {{ contentManager.nbPages+1 }}
{{ contentManager.currentPage+1 }} / {{ contentManager.nbPages }}
<button v-on:click="changePage(1)">Next</button>
<button v-on:click="contentManager.currentPage = contentManager.nbPages">Last</button>
<button v-on:click="contentManager.currentPage = contentManager.nbPages-1">Last</button>
</div>
</div>
</div>

2
src/contentmanager.h
View File

@ -38,7 +38,7 @@ private:
bool m_local = true;
QString m_currentLanguage;
void setCurrentPage(int currentPage) {
m_currentPage = max(0, min(currentPage, getNbPages()));
m_currentPage = max(0, min(currentPage, getNbPages()-1));
emit(booksChanged());
}
void setCurrentLanguage(QString language);