Skip to content

SCHOL-314: Re-enable and update pdf tests#191

Merged
jackiequach merged 3 commits intoSCHOL-305/implement-designsfrom
SCHOL-314/reenable-pdf-tests
Feb 18, 2026
Merged

SCHOL-314: Re-enable and update pdf tests#191
jackiequach merged 3 commits intoSCHOL-305/implement-designsfrom
SCHOL-314/reenable-pdf-tests

Conversation

@jackiequach
Copy link
Collaborator

@jackiequach jackiequach commented Feb 12, 2026

SCHOL-314

Base branch is the design implementation branch for UI changes.

  • Re-enables PDF tests with a new webpub manifest using a PDF hosted in our S3 bucket.
  • Adds a test for page input updates.
  • Removes PDF settings (paginated/scrolling mode) tests since the button no longer exists.
  • Updates addTocToManifest to handle PDFs with no outlines.

@jackiequach jackiequach requested a review from alea12 as a code owner February 12, 2026 22:25
@vercel
Copy link

vercel bot commented Feb 12, 2026

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

Project Deployment Actions Updated (UTC)
web-reader Ready Ready Preview, Comment Feb 13, 2026 7:50pm

Request Review

Copy link
Member

@alea12 alea12 left a comment

Choose a reason for hiding this comment

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

Looking good!

@jackiequach jackiequach merged commit 336675f into SCHOL-305/implement-designs Feb 18, 2026
4 of 6 checks passed
@jackiequach jackiequach deleted the SCHOL-314/reenable-pdf-tests branch February 18, 2026 14:49
jackiequach added a commit that referenced this pull request Feb 18, 2026
* initial design changes

* update html settings

* implement reset settings and current page for PDFs

* add fit to height and width functionality for pdf

* implement fit to height/width for HTMLReader

* fix flickering on pdf reader

* add tooltips

* fix scrolling fit to height zoom issue

* add rotate left functionality

* address comment, update epub settings menu and icon sizes

* fix chapter nav

* implement full screen

* update and remove tests

* update unit tests

* fix scaling issue

* update playwright test to run against localhost

* remove console log and fix flickering issue

* SCHOL-314: Re-enable and update pdf tests (#191)

* re-enable pdf tests and add test for page input

* fix flaky locator

* address comments
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.

2 participants