The media pager no longer goes round from its last post to its first (or first to last): a step, a flick or the wheel at an end stays there.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_013ZDf1cyCQLkfjftYhymZAg

2 files changed, 17 insertions, 16 deletions

diff --git a/tools/pager-swipe-fly-test.js b/tools/pager-swipe-fly-test.js
index 707aad6..657ea75 100644
--- a/tools/pager-swipe-fly-test.js
+++ b/tools/pager-swipe-fly-test.js
@@ -1,8 +1,9 @@
 // On a phone, a fast flick up or down in the media pager flies past several
 // posts and lands snapped on one, the way a spin of the wheel does; a
 // gentler swipe is one post, as before; a fling at the end lands on the
-// last post rather than wrapping past it; a tap mid-flight stops it on the
-// post most on screen, and a swipe back from there goes to the one before.
+// last post and goes no further (no going round to the first); a tap
+// mid-flight stops it on the post most on screen, and a swipe back from
+// there goes to the one before.
 // Run: bun tools/pager-swipe-fly-test.js
 import puppeteer from './puppeteer.js';
 import { cacheKeyFor } from '../src/wallet.js';
@@ -121,15 +122,18 @@ try {
   const seen = await page.evaluate(() => document.querySelector('.mp-position').textContent);
   check('the counter says where it landed', seen === landed + ' / ' + N, seen);
 
-  console.log('\n[a flick near the end coasts round to the start]');
+  console.log('\n[the ends are ends]');
   while ((await at()) !== 7) { await page.keyboard.press((await at()) < 7 ? 'ArrowDown' : 'ArrowUp'); await settled(); }
   const before = await at();
   await swipe(600, 80, 3); await settled();
-  check('from post ' + before + ' (a flight\'s worth would be past the end), a flick comes round to the first post', before >= 7 && (await at()) === 1, String(await at()));
-  await swipe(-600, 80, 3); await settled();
-  check('…and a flick back up from the first comes round to the last', (await at()) === N, String(await at()));
+  check('from post ' + before + ', a flick whose flight would be past the end lands on the last post', before === 7 && (await at()) === N, String(await at()));
   await swipe(600, 80, 3); await settled();
-  check('…a flick down from the last, to the first again', (await at()) === 1, String(await at()));
+  check('…a flick on from the last stays there (no going round to the first)', (await at()) === N, String(await at()));
+  await page.keyboard.press('ArrowDown'); await settled();
+  check('…and so does ↓', (await at()) === N, String(await at()));
+  while ((await at()) !== 1) { await page.keyboard.press('ArrowUp'); await settled(); }
+  await swipe(-600, 80, 3); await settled();
+  check('a flick back up from the first stays on the first', (await at()) === 1, String(await at()));
   await swipe(320, 130); await settled();
   check('a gentle swipe from the first: the second, as ever', (await at()) === 2, String(await at()));
 
diff --git a/tools/profile-pager-test.js b/tools/profile-pager-test.js
index aac681a..2a78598 100644
--- a/tools/profile-pager-test.js
+++ b/tools/profile-pager-test.js
@@ -105,18 +105,15 @@ try {
   await page.keyboard.press('ArrowDown');
   check('down again: her last', await resting('d.png'), await curPic());
   await page.keyboard.press('ArrowDown'); await sleep(600);
-  check('past her last wraps to her first post', await resting('a2.png'), await curPic());
-  check('counter counts the first post once', await page.$eval('.mp-position', (e) => e.textContent) === '1 / 3');
+  check('past her last it stays on her last (no going round)', await resting('d.png'), await curPic());
+  check('counter still at the last post', await page.$eval('.mp-position', (e) => e.textContent) === '3 / 3');
   check('no loading slide', await page.$('.mp-tail') === null);
-  await page.keyboard.press('ArrowUp'); await resting('d.png');
-  check('up from first wraps to last', await curPic() === 'd.png');
-  check('counter at the last post', await page.$eval('.mp-position', (e) => e.textContent) === '3 / 3');
   await page.touchscreen.touchStart(195, 650);
   for (let i = 1; i <= 8; i++) { await page.touchscreen.touchMove(195, 650 - 40 * i); await sleep(16); }
-  await page.touchscreen.touchEnd();
-  check('touch swipe wraps to first post', await resting('a2.png'), await curPic());
-  await page.evaluate(() => document.querySelector('.media-pager').dispatchEvent(new WheelEvent('wheel', { deltaY: -3, deltaMode: 1, bubbles: true, cancelable: true })));
-  check('wheel wraps back to last post', await resting('d.png'), await curPic());
+  await page.touchscreen.touchEnd(); await sleep(600);
+  check('a touch swipe on from her last stays there', await resting('d.png'), await curPic());
+  await page.evaluate(() => document.querySelector('.media-pager').dispatchEvent(new WheelEvent('wheel', { deltaY: 3, deltaMode: 1, bubbles: true, cancelable: true }))); await sleep(600);
+  check('…and so does the wheel', await resting('d.png'), await curPic());
   await page.keyboard.press('ArrowUp');
   check('up: her post before', await resting('b.png'), await curPic());
   await page.keyboard.press('Escape');