Resizing Files window doesn't render quick view of a pdf file correctly |
|||||||||||
Issue descriptionWhat steps will reproduce the problem? 1. Open Files app and select a pdf file from Downloads folder 2. Press Space key to preview the file 3. Resize the Files app window to different sizes Expected behavior: Quick view should be rendered correctly according to the different sizes of Files window Actual behavior: Quick view is not rendered correctly Please refer video attachments for the expected and actual behaviors.
,
Feb 7 2018
This is also happening to PDFs via the Chrome browser, so redirecting to the PDF team to take a look.
,
Feb 9 2018
When the File Manager opened the PDF in quick view, it sets the zoom to a specific level. When this happens, the zoom stays at the specific level. The File Manager needs to open the PDF Viewer in Fit-to-width or Fit-to-page mode. Then the PDF Viewer will get the desired behavior.
,
Feb 13 2018
thanks thestig@! fukino@ could you please take a look?
,
Feb 15 2018
Moving back to M66 to investigate.
,
Feb 22 2018
,
Feb 28 2018
,
Mar 1 2018
,
Mar 28 2018
WIP CL: http://crrev.com/c/984912
,
Mar 28 2018
,
Apr 4 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e0fb2a3bc358481049677a2aec556ef8d5d81120 commit e0fb2a3bc358481049677a2aec556ef8d5d81120 Author: Naoki Fukino <fukino@chromium.org> Date: Wed Apr 04 09:45:09 2018 Files app: Preview PDF file in fit-to-page mode in quick view. Instead of using the default zoom level, we can set fit-to-zoom mode by default to resize the preview automatically when Files app window is resized. Bug: 809314 Test: Manually tested Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I6e8d2113c9bbbd65071de2191f4be37b62ef6fd0 Reviewed-on: https://chromium-review.googlesource.com/984912 Reviewed-by: Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Commit-Queue: Naoki Fukino <fukino@chromium.org> Cr-Commit-Position: refs/heads/master@{#548013} [modify] https://crrev.com/e0fb2a3bc358481049677a2aec556ef8d5d81120/ui/file_manager/file_manager/foreground/js/quick_view_controller.js
,
Apr 5 2018
,
May 3 2018
Verified on Chrome OS 10575.26.0, 67.0.3396.31 |
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by mkarkada@chromium.org
, Feb 6 2018