Skip to content

Humanreadable improvement fixes and refresh for release - #788

Merged
SharonStrats merged 2 commits into
stagingfrom
feat/humanreadable-improvements
Oct 31, 2026
Merged

SharonStrats merged 2 commits into
stagingfrom
feat/humanreadable-improvements

Conversation

@SharonStrats

Copy link
Copy Markdown
Contributor
  • restore pdf rendering, use chrome viewer
  • humanreadable panes on mobile, no scroll fit the whole page
  • desktop - extend the page
  • Support JSON - no special treatment now Prompt: pdf is not appearing - chrome blocked it worked in main branch please investigate and fix

Copilot AI 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.

🟡 Changes recommended

Unresolved security issues let shared HTML previews read private same-origin files and unsandboxed SVG previews access the parent application and its storage.

5 open issues
What changed in this PR

This PR updates human-readable previews to restore native PDF viewing, improve desktop and mobile sizing, and support JSON and signed-in Dokieli sessions.

Changes:

  • Adjust iframe sandboxing for browser viewers and display JSON as plain text.
  • Add preview height reporting and revise responsive layouts.
  • Extend Dokieli’s session bridge and request handling, with corresponding unit tests.
File Description
test/​unit/​humanReadablePane.test.ts Tests JSON rendering, PDF sandbox behavior, and Dokieli bridge changes.
src/​humanReadablePane.ts Updates preview rendering, automatic sizing, and Dokieli session integration.
src/​humanReadablePane.css Revises pane sizing, scrolling, and mobile layout rules.

Review effort: Balanced

Comment thread src/humanReadablePane.ts Resolved
Comment thread src/humanReadablePane.ts Resolved
Comment thread src/humanReadablePane.ts Resolved
Comment thread src/humanReadablePane.ts Resolved
Comment thread src/humanReadablePane.ts Resolved
- restore pdf rendering, use chrome viewer
- humanreadable panes on mobile, no scroll fit the whole page
- desktop - extend the page
- Support JSON - no special treatment now
Prompt: pdf is not appearing - chrome blocked it worked in main branch please investigate and fix

Co-authored-by: GPT-5.6 Luna <gpt-5.6-luna@openai.com>
@SharonStrats
SharonStrats force-pushed the feat/humanreadable-improvements branch from 1ed08e7 to 87d55fa Compare October 31, 2026 22:14
…@5.0.2-5 pane-registry@5.0.2-2 folder-pane@4.0.1-1 profile-pane@4.1.0-0 source-pane@4.0.1-0) (latest: rdflib@2.4.1 activitystreams-pane@2.0.0 chat-pane@4.0.0 contacts-pane@4.0.0 issue-pane@4.0.0 meeting-pane@4.0.0)
@SharonStrats
SharonStrats merged commit 892e067 into staging Oct 31, 2026
8 checks passed
@SharonStrats
SharonStrats deleted the feat/humanreadable-improvements branch October 31, 2026 22:30
@github-project-automation github-project-automation Bot moved this from In review to Done in SolidOS NLNet UI Oct 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

HumanReadable Pane does not display JSON files

2 participants