rays/ipx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 20 Wiki Activity

An item's text takes four fifths of the reading pane, not 80 characters (#360)

Browse Source 
Asked for as a share of the window. The pane is the window less the
feed list, so a share of the window would never bind; a share of the
pane does. On a phone the pane is the screen, and keeps all of it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Ray Slakinski
2026-10-25 01:49:44 +00:00
parent cad14ec09d
commit 2c7e23c346
3 changed files with 12 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
tests/ui/app.spec.js
View File

@@ -189,14 +189,11 @@ test('the reading font is an item\'s title and text, and the interface keeps its
// A serif gets more room between its lines (1.75 of the text's size, against 1.65).
const lh = await page.locator('#detail .dbody').evaluate(el => { const c = getComputedStyle(el); return parseFloat(c.lineHeight) / parseFloat(c.fontSize); });
expect(lh).toBeCloseTo(1.75, 2);
// About 80 characters a line, however wide the pane (#360).
const measure = await page.locator('#detail .dbody').evaluate(el => {
const ch = document.createElement('span'); ch.textContent = '0'; el.append(ch);
const n = el.getBoundingClientRect().width / ch.getBoundingClientRect().width; ch.remove(); return n; });
expect(measure).toBeLessThanOrEqual(80.5);
const [pane, text] = await page.locator('#detail .dbody').evaluate(el =>
[el.parentElement.clientWidth, el.getBoundingClientRect().width]);
expect(text, 'the pane is wider, so the measure is what holds it').toBeLessThan(pane - 100);
// Four fifths of the pane, whatever the font (#360).
const share = await page.locator('#detail .dbody').evaluate(el => {
const p = el.parentElement, c = getComputedStyle(p);
return el.getBoundingClientRect().width / (p.clientWidth - parseFloat(c.paddingLeft) - parseFloat(c.paddingRight)); });
expect(share).toBeCloseTo(0.8, 2);
await page.locator('#srfont').selectOption('same');
await page.locator('#sfont').selectOption('theme');