Repository navigation
Land the judged redesign branch line: strips, content layer, 24-grid icons, dakit dea377c - #38
Merged
Merged
Conversation
Client identity meta and campaign summaries render as single segmented strips, audience breakdowns and tag clouds become colon-free segment facts with status dots, the Mailchimp mappings section joins the panel pattern, and the audience select gets a placeholder label.
The family contract (dakit family.md, 71a39d3) renders icons as 24-viewBox paths at 20px, stroke 1.8; the operator chrome still taught the retired 16-box, stroke-1.5 set. Redraw all twenty sidebar, toolbar, theme, and action glyphs on the 24 grid: shared controls copy the sibling apps byte for byte (dataqna's theme moon/sun, menu, sidebar chevrons, sliders; the showcase chevron-down, x, doc, plus, clock), and relay-specific concepts (grid, send, users, inbox, funnel, at-sign, briefcase, code, shield) move to the same 24-grid drawing style so no app mixes the two sets.
…mily/land-1003 # Conflicts: # templates/base.html # templates/mailing/dashboard.html
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
1 participant
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.
Merges
redesign/dataops-family(065247e, the continuously-judged line, 7 judge rounds recorded, latest PASS) intomain(793a84c) so main becomes the superset. Merge commit: 849666f.Branch commits carried (8 ahead of main)
.seg-fact, stat-link→stat-value)Conflict resolutions
viewBox="0 0 24 24", rendered 20px,stroke-width="1.8",fill="none", round caps/joins. ZeroviewBox="0 0 16 16"/stroke-width="1.5"anywhere in templates. Main's path-agnostic geometry-pinning test (test_chrome_icons_use_family_24grid_geometry) kept unchanged — it asserts attributes, not path data, and passes against the branch's icon set./admin/(Django admin) — and the branch never had the admin-API-keys feature at all. Restored main's "Admin API keys" nav entry (24-grid shield icon) pointing atmailing:admin_api_keys; the admin keys page/routes/PR Fix judge findings on the landed family redesign #33 fixes (stack-table + data-labels, banded panel, outlined.dangerRevoke, no Django-admin link) survive untouched as main-only files.Verification
uv run pytest: 755 passed (branch tip's count governs + main's geometry test)uv run ruff check .: clean