f.weber/homebrewery
0
0
Fork 0
mirror of https://github.com/naturalcrit/homebrewery.git synced 2026-01-06 07:52:40 +00:00
Code Wiki Activity

Change page range to only display a single range

Browse Source 
Having multiple page ranges visible is a weird edge case that only happens in two-page view. Simplifying logic to just group all page ranges together if a middle page is partly obscured.
This commit is contained in:
Trevor Buckner
2024-12-23 18:37:20 -05:00
parent 2b7a1e1cb2
commit d588a92147
1 changed files with 1 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
client/homebrew/brewRenderer/toolBar/toolBar.jsx
View File

@@ -79,20 +79,7 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa
if(pages.length === 0) return ''; if(pages.length === 0) return '';
const sortedPages = [...pages].sort((a, b)=>a - b); // Copy and sort the array const sortedPages = [...pages].sort((a, b)=>a - b); // Copy and sort the array
const ranges = []; return sortedPages.length == 1 ? `${sortedPages[0]}` : `${sortedPages[0]} - ${sortedPages.at(-1)}`;
let start = sortedPages[0];
for (let i = 1; i <= sortedPages.length; i++) {
// If the current page is the end of the list or not consecutive
if(i === sortedPages.length || sortedPages[i] !== sortedPages[i - 1] + 1) {
ranges.push(
start === sortedPages[i - 1] ? `${start}` : `${start} - ${sortedPages[i - 1]}`
);
start = sortedPages[i]; // Start a new range
}
}
return ranges.join(', ');
}; };
return ( return (