Where.src/popup/styles/main.css (line 31) makes the popup body 396px wide. README.md (UI Design Philosophy, "360x600 popup"), two comments in src/popup/index.html (lines 93 and 215) and POPUP_VIEWPORT in tests/e2e/run.js (line 1833, used by the Chrome layout tests) all say 360x600.
What happens. The Chrome layout tests measure the popup at a width it never has, so a screen that overflows at 396 can pass them, and the README describes a size the popup is not. The Chrome suite's "scrolls sideways" check on the transaction approval passes whatever the page does, because it runs in a 1280x720 window.
Should. The README, the comments and the tests state and use the popup's real size.
Severity: minor
Definition of done
README.md, the two comments in src/popup/index.html and POPUP_VIEWPORT give the popup's real size.
The Chrome layout tests pass at that size, and the transaction approval's sideways-scroll check runs at the approval window's real size, so it can fail.
Model: opus-5-5
Found by the worker on https://git.eeqj.de/sneak/AutistMask/issues/518.
**Where.** `src/popup/styles/main.css` (line 31) makes the popup body 396px wide. `README.md` (UI Design Philosophy, "360x600 popup"), two comments in `src/popup/index.html` (lines 93 and 215) and `POPUP_VIEWPORT` in `tests/e2e/run.js` (line 1833, used by the Chrome layout tests) all say 360x600.
**What happens.** The Chrome layout tests measure the popup at a width it never has, so a screen that overflows at 396 can pass them, and the README describes a size the popup is not. The Chrome suite's "scrolls sideways" check on the transaction approval passes whatever the page does, because it runs in a 1280x720 window.
**Should.** The README, the comments and the tests state and use the popup's real size.
Severity: minor
## Definition of done
- [ ] `README.md`, the two comments in `src/popup/index.html` and `POPUP_VIEWPORT` give the popup's real size.
- [ ] The Chrome layout tests pass at that size, and the transaction approval's sideways-scroll check runs at the approval window's real size, so it can fail.
Model: opus-5-5
clawbot
self-assigned this 2026-10-14 14:02:46 +02:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Found by the worker on #518.
Where.
src/popup/styles/main.css(line 31) makes the popup body 396px wide.README.md(UI Design Philosophy, "360x600 popup"), two comments insrc/popup/index.html(lines 93 and 215) andPOPUP_VIEWPORTintests/e2e/run.js(line 1833, used by the Chrome layout tests) all say 360x600.What happens. The Chrome layout tests measure the popup at a width it never has, so a screen that overflows at 396 can pass them, and the README describes a size the popup is not. The Chrome suite's "scrolls sideways" check on the transaction approval passes whatever the page does, because it runs in a 1280x720 window.
Should. The README, the comments and the tests state and use the popup's real size.
Severity: minor
Definition of done
README.md, the two comments insrc/popup/index.htmlandPOPUP_VIEWPORTgive the popup's real size.Model: opus-5-5