fix: Update CSS for user table#35
Conversation
There was a problem hiding this comment.
Pull request overview
Updates shared styling in styles/global.css to apply the existing dashboard table look-and-feel to a “user table” variant.
Changes:
- Extends the
div#dash1 tablestyle rules to also match elements under a.user-tablecontainer (headers, cells, footer, status/warning/error icons). - Adds a
.user-tableselector to inherit the dashboard table’s base font sizing/collapsing behavior.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| div#dash1 table .high, .low, | ||
| div.user-table .high, .low{ |
There was a problem hiding this comment.
The selector list div#dash1 table .high, .low, div.user-table .high, .low makes .low a global selector (not scoped to div#dash1 table / .user-table). This can unintentionally style any element with class low across the app. Scope .low to the same containers as .high (i.e., qualify each .low entry explicitly).
| div#dash1 table .high, .low, | |
| div.user-table .high, .low{ | |
| div#dash1 table .high, div#dash1 table .low, | |
| div.user-table .high, div.user-table .low{ |
| } | ||
|
|
||
| div#dash1 table, | ||
| div.user-table{ |
There was a problem hiding this comment.
border-collapse only affects table elements; applying it to div.user-table has no effect. If the intent is to style a user table, target the table element inside the container (or use a table.user-table selector) so border collapsing actually applies.
| div.user-table{ | |
| div.user-table table{ |
| div#dash1 table, | ||
| div.user-table{ | ||
| font-size:.9em; | ||
| border-collapse:collapse; | ||
| } | ||
|
|
||
| div#dash1 table th.status, | ||
| div.user-table th.status{ |
There was a problem hiding this comment.
The newly added .user-table selector lines are not indented like the surrounding CSS rules, which makes this block harder to read/maintain. Align indentation with the existing div#dash1 ... selectors in this section.
Automated fix by CoderOps.
Swarm: swarm50
Task: Update CSS for user table