git-ents.gitmain
⌘K
foforge
commit 4e79735
fix: stop styling unwired controls as interactive

The topic pill used the link color and a hover color shift though it is a plain span with no click handler, and the issues page subtabs had no explicit cursor, both inviting a click that does nothing. The topic pill now reads as a static badge like the issue filter chips, and subtabs get the same cursor:default those chips already use.

Assisted-by: Claude:claude-sonnet-5

Joseph D. Carpinelli · 1 month ago

Reviews

No reviews of this commit yet — record a verdict below.

Start a review

verdict

crates/git-ents-server/src/web/style.css @@ -111,8 +111,7 @@ .pill-public { font-family: var(--font-mono); font-size: .7rem; font-weight: 500; color: var(--color-text-muted); background: var(--color-code-bg); border-radius: var(--radius-pill); padding: .1rem .55rem; } .repo-desc { font-size: .98rem; color: var(--color-text); max-width: 40rem; margin-top: .65rem; } .topics { display: flex; flex-wrap: wrap; gap: .4rem; margin-top: .75rem; } -.topic { font-family: var(--font-mono); font-size: .72rem; color: var(--color-link); background: var(--color-accent-subtle); border-radius: var(--radius-pill); padding: .12rem .6rem; text-decoration: none; } -.topic:hover { color: var(--color-link-hover); } +.topic { font-family: var(--font-mono); font-size: .72rem; color: var(--color-text-muted); background: var(--color-accent-subtle); border-radius: var(--radius-pill); padding: .12rem .6rem; cursor: default; } .tabs { display: flex; gap: .15rem; border-bottom: 1px solid var(--color-border); margin-bottom: 1.75rem; overflow-x: auto; } .tab { display: inline-flex; align-items: center; gap: .4rem; padding: 10px 14px; font-size: .88rem; color: var(--color-text-muted); text-decoration: none; white-space: nowrap; position: relative; transition: color .15s; } .tab:hover { color: var(--color-text); } @@ -335,7 +334,7 @@ .chip { font-family: var(--font-mono); font-size: .76rem; color: var(--color-text-muted); background: var(--color-surface); border: 1px solid var(--color-border); border-radius: var(--radius-pill); padding: .2rem .7rem; cursor: default; } .chip.active { color: var(--color-bg); background: var(--color-accent); border-color: var(--color-accent); } .card-header.subtabs { display: flex; gap: 1.25rem; text-transform: none; letter-spacing: 0; } -.subtab { display: inline-flex; align-items: center; gap: .35rem; color: var(--color-text-muted); } +.subtab { display: inline-flex; align-items: center; gap: .35rem; color: var(--color-text-muted); cursor: default; } .subtab.active { color: var(--color-text); font-weight: 600; } .subtab .icon { width: 13px; height: 13px; } .subtab.active .icon { color: var(--s-func); }