Skip to content

Row opacity compounding, and the test count this repo stated twice #30

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Varnasr merged 1 commit into from
Sep 23, 2026
Merged

Row opacity compounding, and the test count this repo stated twice #30

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,12 @@ Two things a future session should know:
## Testing

`.github/workflows/python-tests.yml` runs `pytest tests/` on pull requests and
pushes to main. 25 tests.
pushes to main. **101 tests**, verified on 2026-09-23 against the pinned set on
Python 3.11.

This said 25 for as long as the file existed, and so did the header comment in
`requirements.txt`, while the Layout section above said 101. Two numbers for one
suite, in one repository, and nothing compared either against `pytest`.

```
pip install -r requirements.txt
Expand Down
11 changes: 10 additions & 1 deletion assets/css/stack.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,16 @@ a { color: inherit; text-decoration: none; }
transition: background .12s, color .12s; }
.row:hover, .row:focus-visible { background: var(--ink); color: var(--bg); }
.row b { font-weight: 600; font-size: .95rem; }
.row span { font-family: var(--mono); font-size: .66rem; opacity: .7; white-space: nowrap; }
.row span { font-family: var(--mono); font-size: .66rem; white-space: nowrap; }
/* Opacity belongs on the tag, not on every span in the row. `.t` is itself a
span wrapping the title and the tag, so `.row span { opacity: .7 }` applied
twice to the tag (0.7 x 0.7 = 0.49) and once to the title. Measured on the
InsightStack landing page: the tag came out #84827f on #f6f2ea, 3.43:1 at
10.56px, failing AA on 18 elements, and every row title rendered at 70%
opacity, which is the opposite of what `.row b { font-weight: 600 }` is for.
The comment below already records the same selector doing the same thing to
white-space; that was scoped and this was not. */
.row .t > span { opacity: .7; }

/* --- calculator layout --------------------------------------------------- */
.calc { display: grid; grid-template-columns: 1.05fr .95fr; gap: 0;
Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
# failure waiting for a quiet week.
#
# Every version below was resolved together and verified on Python 3.11 with the
# full test suite green (25 tests). Raise them deliberately, together, and re-run
# full test suite green (101 tests). Raise them deliberately, together, and re-run
# the suite; do not let a resolver do it silently.

pandas==2.3.3
Expand Down
Loading