From 0a478dac7fcfb579d7d79c18171e950e82cba020 Mon Sep 17 00:00:00 2001 From: Norman Feske Date: Tue, 12 Jan 2021 14:15:16 +0100 Subject: [PATCH] pdf_view: tweaks for showing 16:10 slides - Shave off two pixels on right, which are rendered white for no reason. - Adjust resolution to make the slides fit vertically by default. --- repos/libports/src/app/pdf_view/main.cc | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/repos/libports/src/app/pdf_view/main.cc b/repos/libports/src/app/pdf_view/main.cc index 09a3b9a4aa..deaeedd60d 100644 --- a/repos/libports/src/app/pdf_view/main.cc +++ b/repos/libports/src/app/pdf_view/main.cc @@ -136,7 +136,7 @@ class Pdf_view * use case of showing slides. */ _pdfapp.resolution = Genode::min(_nit_mode.area.w()/5, - _nit_mode.area.h()/3.8); + _nit_mode.area.h()/4); typedef Gui::Session::Command Command; _gui.enqueue(_view, Rect(Point(), _nit_mode.area)); @@ -282,8 +282,11 @@ class Pdf_view void Pdf_view::show() { + auto reduce_by = [] (auto value, auto diff) { + return (value >= diff) ? value - diff : 0; }; + Framebuffer::Area const fb_size = _fb_mode.area; - int const x_max = Genode::min((int)fb_size.w(), _pdfapp.image->w); + int const x_max = Genode::min((int)fb_size.w(), reduce_by(_pdfapp.image->w, 2)); int const y_max = Genode::min((int)fb_size.h(), _pdfapp.image->h); /* clear framebuffer */