Skip to content

Fix thumbnail alignment in PDF extension#1541

Merged
Saira-A merged 2 commits into
UniversalViewer:devfrom
Saira-A:PDFthumbnails
Sep 29, 2025
Merged

Fix thumbnail alignment in PDF extension#1541
Saira-A merged 2 commits into
UniversalViewer:devfrom
Saira-A:PDFthumbnails

Conversation

@Saira-A

@Saira-A Saira-A commented Sep 26, 2025

Copy link
Copy Markdown
Contributor

Fixes issue #1529 by adding a grid for one column thumbnails.
I also adjusted the thumbnail focus styling as previously when a PDF thumbnail or label was clicked on they were both outlined with a separating line between them, so now only the thumbnail is outlined but tabbing highlights them both as one unit which is how it works for OSD

@vercel

vercel Bot commented Sep 26, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
universalviewer Ready Ready Preview Comment Sep 26, 2025 2:55pm

@demiankatz demiankatz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @Saira-A, this looks much better to me.

I won't merge this right away in case @LanieOkorodudu or others want to give it a closer look... but please feel free to merge once you're satisfied that you've received enough feedback!

@LanieOkorodudu

Copy link
Copy Markdown
Contributor

Thanks @Saira-A, this looks much better now. @demiankatz, happy for you to go ahead and merge the fix.

@Saira-A Saira-A merged commit c5ce348 into UniversalViewer:dev Sep 29, 2025
4 checks passed
@Saira-A

Saira-A commented Sep 29, 2025

Copy link
Copy Markdown
Contributor Author

Thanks @LanieOkorodudu @demiankatz!

@Saira-A Saira-A deleted the PDFthumbnails branch September 29, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants