Skip to content

fix: simplied PDF view handling #317

Open
kmurugulla wants to merge 1 commit intomainfrom
mlpdfprevtab
Open

fix: simplied PDF view handling #317
kmurugulla wants to merge 1 commit intomainfrom
mlpdfprevtab

Conversation

@kmurugulla
Copy link
Copy Markdown
Contributor

Removes PDF blob loading complexity (~200 lines) and renders PDFs like fragments (icon + colored
background). Fixes localhost URLs in parsing by converting to sidekick config preview host and handles double-encoded
filenames. Consolidates duplicate helpers and removes dead code.

Fix #

Test URLs:

@kmurugulla kmurugulla requested a review from amol-anand April 4, 2026 18:18
@aem-code-sync
Copy link
Copy Markdown

aem-code-sync bot commented Apr 4, 2026

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

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.

1 participant