Skip to content

chore: two dead .admin-workspace-content rules in admin.css #495

Description

@FlyM1ss

admin.css carries two rules scoped to .admin-workspace-content:

471  .admin-workspace-content .admin-stats{margin:18px 0 22px}
472  .admin-workspace-content .admin-credits-status{margin:0 0 14px}

That wrapper exists in app.html:2144 but was never ported into admin.html, so neither rule matches anything on /admin. They were retyped from styles.css during the console port; the ancestor element did not come with them.

Not a missing-style bug — admin-console.css supplies both unscoped, at different values:

/app (styles.css, via wrapper) /admin (admin-console.css, unscoped)
.admin-stats margin:18px 0 22px margin:0 0 16px (:2617)
.admin-credits-status margin-top:14px margin-top:14px (:3319)

So the effect is a small vertical-spacing divergence on the site-stats row between the two pages, plus two rules in admin.css that can never fire. Either delete them or port the wrapper — but they should not stay as-is, because they read as governing /admin and do not.

Found while fixing #494 (which corrects a code comment that cited the dead admin.css rule instead of the live admin-console.css one). No overlap with that PR's changes.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NUSgbaFPuckwMHnTgLmFw6

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions