box-content-preview icon indicating copy to clipboard operation
box-content-preview copied to clipboard

Duplicated previewer when preview watermarked file

Open dolgachio opened this issue 5 years ago • 2 comments

Please fill out the following template so we can reproduce and fix your issue as quickly as possible!

Environment:

  • Preview version: 2.16.0 (latest Box Content Preview version)

Desktop (please complete the following information):

  • OS: [MacOS]
  • Browser [Chrome 75.0.3770.142]

Steps to reproduce the problem:

  1. Preview file by providing file object to the show method
  2. Try to zoom in and zoom out file
  3. Try to use aside thumbnails

What is the expected behavior? (Screenshots can be helpful here) Document is properly zoomed and thumbnails sidebar navigation works

What went wrong? (Screenshots, console logs, or HAR files can be helpful here) Document is duplicated, zoom does not work, thumbnails sidebar navigation does not work image

Link to application or sample code: CodePen example You have to generate your own access token to reproduce it in this CodePen, my access token is valid for 1 hour Video recording of the issue

Expected behavior Document is properly zoomed and thumbnails sidebar navigation works.

Additional context I debugged the source code and from my perspective, problem can be fixed here by calling this.destroy() after uncaching watermarked file in Preview.js file - link to the specific line of code This bug happens only for watermarked files and only when we provide file instead of file id to the show() method.

dolgachio avatar Aug 07 '19 14:08 dolgachio

@stas-dolgachov Thanks for reporting the issue! I was able to reproduce the issue, we'll add it to the backlog

ConradJChan avatar Aug 07 '19 17:08 ConradJChan

@ConradJChan thanks for a quick response! Could you provide some approximate estimations on when you plan to fix it?

dolgachio avatar Aug 08 '19 08:08 dolgachio