Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edit Content: Thumbnail in the Binary Preview Punchlist #26907

Closed
10 tasks done
Tracked by #25445
fmontes opened this issue Dec 6, 2023 · 1 comment · Fixed by #27160, #27321 or #27358
Closed
10 tasks done
Tracked by #25445

Edit Content: Thumbnail in the Binary Preview Punchlist #26907

fmontes opened this issue Dec 6, 2023 · 1 comment · Fixed by #27160, #27321 or #27358

Comments

@fmontes
Copy link
Member

fmontes commented Dec 6, 2023

Parent

#25445

User Story

As a content creator, I want to be able to render PDF thumbnails in the binary preview, so I can easily identify PDF files.

Tasks

Proposed Objective

Core Features

Proposed Priority

Priority 3 - Average

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.:

N/A

Assumptions & Initiation Needs

  • The PDF file must be properly formatted
  • Appropriate memory resources should be available

Quality Assurance Notes & Workarounds

  • Thumbnail should be properly rendered on all major browsers

Sub-Tasks & Estimates

N/A

@fmontes fmontes changed the title Edit Content: Render PDF Thumbnail in the Binary Preview Edit Content: Thumbnail in the Binary Preview Punchlist Dec 21, 2023
@rjvelazco rjvelazco self-assigned this Jan 3, 2024
github-merge-queue bot pushed a commit that referenced this issue Jan 10, 2024
…#27160)

* dev: show correct thumbail for images && pdf

* dev: show correct file size in Bytes, KB, MB, and GB

* clean up

* dev: show correct thumbnail

* dev: allow user edit the image in old JSP

* dev: fix DotContentThumbnailComponent tests

* dev: fix tests

* dev: clean up && update prime icon for web-components

* clean up

* update story

* fix test

* clean up

* clean up

* update styles

* update styles v2
@rjvelazco rjvelazco reopened this Jan 11, 2024
@rjvelazco rjvelazco reopened this Jan 17, 2024
@josemejias11
Copy link
Contributor

Approved: Tested on master_1894351, Docker, macOS 13.0, FF v121.0.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment