Skip to content

Commit 303fd7e

Browse files
Swap zoom in and zoom out button positions
Add zoom button on top as is tradition in most PDF viewers, and it conforms to user expectations
1 parent a461ee6 commit 303fd7e

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/components/pdfViewer.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -213,25 +213,25 @@ export default function PdfViewer({ url, name }: PdfViewerProps) {
213213

214214
<div className="flex items-center gap-2">
215215
<Button
216-
onClick={zoomOut}
217-
disabled={scale <= 0.25}
216+
onClick={zoomIn}
217+
disabled={scale >= 3}
218218
className="h-9 w-9 rounded p-0 text-white transition hover:bg-[#6536c1] disabled:bg-gray-300"
219219
>
220-
<ZoomOut />
220+
<ZoomIn />
221221
</Button>
222222

223223
<span className="w-10 text-center text-sm font-medium">
224224
{(scale * 100).toFixed(0)}%
225225
</span>
226226

227+
227228
<Button
228-
onClick={zoomIn}
229-
disabled={scale >= 3}
229+
onClick={zoomOut}
230+
disabled={scale <= 0.25}
230231
className="h-9 w-9 rounded p-0 text-white transition hover:bg-[#6536c1] disabled:bg-gray-300"
231232
>
232-
<ZoomIn />
233+
<ZoomOut />
233234
</Button>
234-
235235
<ShareButton />
236236

237237
<Button onClick={downloadPDF} className="h-9 w-9 rounded p-0">

0 commit comments

Comments
 (0)