Design/noid/hr visual refresh - #38
Merged
Merged
Conversation
Four subtly different card treatments had grown up across the views — background-hover with no border on the statistics tiles, main-background with a border on the chart panels beside them, a third combination on the exports cards, a fourth padding scale on My leave. Nothing was broken and nothing looked deliberate, which is the kind of drift that reads as "unfinished" long before anybody can say why. There is now one `.surface`, one `.tiles` grid and one `.page__hint` in the shared stylesheet, and the views use them instead of each rolling their own. The tables get the same treatment. `.tbl` gave them bottom rules and nothing else: no container, no hover, no header weight — so a long HR table rendered as rules floating on the page background, and the sick leave view had quietly hand-rolled its own row hover because the shared styles offered none. The wrap is now the table's own surface, headers sit on a tinted band, rows highlight under the pointer, and the last row's rule no longer doubles up with the border. StatTile and MeterBar are extracted from the one view each had grown up in. The tile carries an accent colour rather than painting every figure in the primary element: a row of identical blue numbers gives the eye nothing to hold on to, and the accent is free information. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Balances was seven numeric columns of equal weight, which meant the one column HR opens the page for — Available — looked exactly like the six around it, and questions as basic as "how many people are running low?" had to be totted up by eye. It now leads with four tiles answering the questions the table gets opened for: headcount, days taken, days still available, and how many people are nearly out or overdrawn. They are computed over the *filtered* rows, so narrowing to a search or a group re-answers them for that group. Only types that count against a balance are summed — adding unpaid leave, which has no entitlement, into "days available" would be meaningless. Each row gains a used-of-entitlement meter, and Available carries the only colour in the row: red when overdrawn, amber under a fifth left. The meter turns amber with it, so the warning is in the shape as well as in the number. Statistics gains a fourth tile, days in the busiest month, named. The average alone hides the August everybody disappears in, which is the one month worth staffing around. Its two charts now sit side by side, since they are read together, and the page uses the same measure as its siblings instead of being 200px narrower for no reason. Sick leave trades its summary sentence for the same tiles, and its average is taken across the whole workforce rather than the affected only — an average over the affected climbs as fewer people fall ill, which is the opposite of how the number gets read. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 AI (if applicable)