From c39718ca81b2b6c5023500c41dca6af659223771 Mon Sep 17 00:00:00 2001 From: Matthew Olsson Date: Sun, 27 Mar 2022 09:14:09 -0700 Subject: [PATCH] PDFViewer: Don't change pages on horizontal scroll This was a bit jarring, and didn't align with the behavior of other PDF renderers. --- Userland/Applications/PDFViewer/PDFViewer.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Userland/Applications/PDFViewer/PDFViewer.cpp b/Userland/Applications/PDFViewer/PDFViewer.cpp index eedee72d24..049447baab 100644 --- a/Userland/Applications/PDFViewer/PDFViewer.cpp +++ b/Userland/Applications/PDFViewer/PDFViewer.cpp @@ -126,7 +126,7 @@ void PDFViewer::mousewheel_event(GUI::MouseEvent& event) if (scrolled_down) { if (scrollbar.value() == scrollbar.max()) { - if (m_current_page_index < m_document->get_page_count() - 1) { + if (m_current_page_index < m_document->get_page_count() - 1 && !event.shift()) { m_current_page_index++; if (on_page_change) on_page_change(m_current_page_index); @@ -137,7 +137,7 @@ void PDFViewer::mousewheel_event(GUI::MouseEvent& event) } } else { if (scrollbar.value() == 0) { - if (m_current_page_index > 0) { + if (m_current_page_index > 0 && !event.shift()) { m_current_page_index--; if (on_page_change) on_page_change(m_current_page_index);