Skip to content

BR auto-hiding chrome#1571

Open
cdrini wants to merge 16 commits into
internetarchive:masterfrom
cdrini:feature/hideable-chrome2
Open

BR auto-hiding chrome#1571
cdrini wants to merge 16 commits into
internetarchive:masterfrom
cdrini:feature/hideable-chrome2

Conversation

@cdrini

@cdrini cdrini commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Makes the chrome auto-hide when the user is in fullscreen. Specifically:

The chrome hides if:

  • there is no interaction after 2 seconds
  • the user is scrolling down in 1up mode
  • the user taps the center of the book in 2up, or taps anywhere in 1up

And shows if:

  • the user taps the centre of the book or the backdrop in 2up, or anywhere in 1up
  • begins scrolling up
  • moves their mouse close to the bottom or left edge of the screen

QA/test: https://deploy-preview-1571--ia-bookreader.netlify.app/bookreaderdemo/demo-internetarchive?ocaid=theworksofplato01platiala (be sure to go fullscreen)

@cdrini
cdrini force-pushed the feature/hideable-chrome2 branch from d3a5805 to 5a1ea9d Compare July 23, 2026 16:18
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 8.13953% with 158 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.15%. Comparing base (172ce56) to head (baa70df).

Files with missing lines Patch % Lines
src/util/ChromeFader.js 0.00% 136 Missing ⚠️
src/BookReader.js 60.00% 6 Missing ⚠️
src/BookReader/Navbar/Navbar.js 50.00% 5 Missing ⚠️
src/BookReader/Mode2UpLit.js 0.00% 3 Missing ⚠️
src/BookReader/Mode1Up.js 0.00% 2 Missing ⚠️
src/BookReader/Mode2Up.js 0.00% 2 Missing ⚠️
src/BookReader/ModeAbstract.js 0.00% 2 Missing ⚠️
src/BookReader/ModeThumb.js 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1571      +/-   ##
==========================================
- Coverage   63.64%   62.15%   -1.50%     
==========================================
  Files          68       69       +1     
  Lines        6206     6373     +167     
  Branches     1378     1416      +38     
==========================================
+ Hits         3950     3961      +11     
- Misses       2217     2373     +156     
  Partials       39       39              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

claude and others added 5 commits July 23, 2026 12:27
Co-Authored-By: Drini Cami <cdrini@gmail.com>
- Put the explicit down arrow behind an experiment option
- Fix "canceling" show after starting to scroll up
- Remove some unused code
- Fix some type errors

Co-Authored-By: Drini Cami <cdrini@gmail.com>
Co-Authored-By: Drini Cami <cdrini@gmail.com>
@cdrini
cdrini force-pushed the feature/hideable-chrome2 branch from 5a1ea9d to 9d6ecbc Compare July 23, 2026 16:28
@cdrini
cdrini marked this pull request as ready for review July 23, 2026 16:31
…Fader

Co-Authored-By: Drini Cami <cdrini@gmail.com>
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