Skip to content

ui(server-logs,server-settings): redesign via /ui-expert - #862

Closed
LucasSantana-Dev wants to merge 2 commits into
release/v2.11.0from
ui/server-tools-2026-05-15
Closed

LucasSantana-Dev wants to merge 2 commits into
release/v2.11.0from
ui/server-tools-2026-05-15

Conversation

@LucasSantana-Dev

Copy link
Copy Markdown
Owner

Round-2 /ui-expert. Anchors: Linear table density (ServerLogs, b2b-data-dense) + Carbon admin grid + Polaris (ServerSettings, enterprise-admin). Queued behind PR #856.

Edits anchored to Linear table density (36px rows, gap-3) and Stripe asymmetric KPI card layout (ServerLogs), Carbon admin grid density and Polaris structured action bars (ServerSettings).

ServerLogs.tsx:
- Log-level summary grid: asymmetric layout with lg:col-span-2 on lead cards (top 3 levels), smaller cards for bottom 3
- LogEntry component: py-3.5 padding, improved hover state (bg-tertiary/40), border-bottom dividers, log type as code element
- Filter section: consolidated label, improved placeholder text

ServerSettings.tsx:
- RBAC grant grid: surface-card utility class, improved grid layout md:grid-cols-[1.5fr_1.2fr_1fr_48px]
- Manager role badges: brand tinting (bg-lucky-brand/15, border-lucky-brand/40), Shield icon branded
- Criativaria Baseline: border-l-4 border-l-lucky-warning accent, icon background box
- Access Control header: icon background box (p-2 rounded-lg bg-lucky-brand/10), Shield icon tinted

Verification: tsc --noEmit passes with zero errors. Slop audit passed (hard bans: no gradients, bento grids, glassmorphism, pure black/white). Em dashes in comments fixed per spec.

No TypeScript errors. No prop signature changes — all callers compatible.
@vercel

vercel Bot commented May 15, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lucky Ready Ready Preview, Comment May 15, 2026 2:29pm

Request Review

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LucasSantana-Dev has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@coderabbitai

coderabbitai Bot commented May 15, 2026 •

Copy link
Copy Markdown

Warning

Rate limit exceeded

@LucasSantana-Dev has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 33 minutes and 6 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c2ffcfd2-8ffd-41cf-bfee-089c37ac61f5

📥 Commits

Reviewing files that changed from the base of the PR and between db7a364 and 1af28f5.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json, !**/package-lock.json
📒 Files selected for processing (3)
  • packages/frontend/src/pages/ServerLogs.tsx
  • packages/frontend/src/pages/ServerSettings.test.tsx
  • packages/frontend/src/pages/ServerSettings.tsx
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ui/server-tools-2026-05-15

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added size/m dependencies Pull requests that update a dependency file frontend labels May 15, 2026
@github-actions

Copy link
Copy Markdown

Failed to generate code suggestions for PR

@github-actions

github-actions Bot commented May 15, 2026 •

Copy link
Copy Markdown

Size Change: +401 B (+0.11%)

Total Size: 370 kB

📦 View Changed
Filename Size Change
packages/frontend/dist/assets/Admin-DjixvvFA.js 0 B -1.99 kB (removed) 🏆
packages/frontend/dist/assets/Admin-N_VwEXxa.js 1.99 kB +1.99 kB (new file) 🆕
packages/frontend/dist/assets/AutoMessages-D5wtoLPc.js 2.68 kB +2.68 kB (new file) 🆕
packages/frontend/dist/assets/AutoMessages-Dau9sozK.js 0 B -2.68 kB (removed) 🏆
packages/frontend/dist/assets/AutoMod-D0E3u1sa.js 4.11 kB +4.11 kB (new file) 🆕
packages/frontend/dist/assets/AutoMod-DuxBdyPz.js 0 B -4.11 kB (removed) 🏆
packages/frontend/dist/assets/CommandsConfig-Cl2rQJsx.js 0 B -1.47 kB (removed) 🏆
packages/frontend/dist/assets/CommandsConfig-De2_ZjOx.js 1.47 kB +1.47 kB (new file) 🆕
packages/frontend/dist/assets/Config-CboQgtx-.js 0 B -1.72 kB (removed) 🏆
packages/frontend/dist/assets/Config-DiKNvSov.js 1.72 kB +1.72 kB (new file) 🆕
packages/frontend/dist/assets/DashboardOverview-BGcGJqRh.js 3.94 kB +3.94 kB (new file) 🆕
packages/frontend/dist/assets/DashboardOverview-C1vk4GaX.js 0 B -3.93 kB (removed) 🏆
packages/frontend/dist/assets/EmbedBuilder-BcA9nvC9.js 3.34 kB +3.34 kB (new file) 🆕
packages/frontend/dist/assets/EmbedBuilder-BK1tC5N4.js 0 B -3.34 kB (removed) 🏆
packages/frontend/dist/assets/Features-CxXYPCqu.js 756 B +756 B (new file) 🆕
packages/frontend/dist/assets/Features-LSvMTGqX.js 0 B -756 B (removed) 🏆
packages/frontend/dist/assets/GuildAutomation-0PtoFquK.js 0 B -2.92 kB (removed) 🏆
packages/frontend/dist/assets/GuildAutomation-OJoFjfzU.js 2.92 kB +2.92 kB (new file) 🆕
packages/frontend/dist/assets/index-3TAp7zAr.css 0 B -16.4 kB (removed) 🏆
packages/frontend/dist/assets/index-CoESPKGo.js 52.1 kB +52.1 kB (new file) 🆕
packages/frontend/dist/assets/index-CtkVPTLb.css 16.6 kB +16.6 kB (new file) 🆕
packages/frontend/dist/assets/index-D7SNjvan.js 0 B -52.1 kB (removed) 🏆
packages/frontend/dist/assets/label-BBE2Rlq-.js 0 B -490 B (removed) 🏆
packages/frontend/dist/assets/label-CoJCCqlx.js 490 B +490 B (new file) 🆕
packages/frontend/dist/assets/Landing-C8uy3RMu.js 3.76 kB +3.76 kB (new file) 🆕
packages/frontend/dist/assets/Landing-DeNWe_-z.js 0 B -3.76 kB (removed) 🏆
packages/frontend/dist/assets/Levels-C0MT7c6y.js 2.63 kB +2.63 kB (new file) 🆕
packages/frontend/dist/assets/Levels-Dendrxzs.js 0 B -2.63 kB (removed) 🏆
packages/frontend/dist/assets/Login-DgNiXWrD.js 2.5 kB +2.5 kB (new file) 🆕
packages/frontend/dist/assets/Login-v5Pbm4yt.js 0 B -2.5 kB (removed) 🏆
packages/frontend/dist/assets/Lyrics-C2pezXQi.js 0 B -1.33 kB (removed) 🏆
packages/frontend/dist/assets/Lyrics-Dc0KYJAs.js 1.33 kB +1.33 kB (new file) 🆕
packages/frontend/dist/assets/Moderation-9LzQhUZg.js 3.85 kB +3.85 kB (new file) 🆕
packages/frontend/dist/assets/Moderation-BrXo-iGs.js 0 B -3.85 kB (removed) 🏆
packages/frontend/dist/assets/Music-BcwtKc-x.js 0 B -7.12 kB (removed) 🏆
packages/frontend/dist/assets/Music-CfY5B2z-.js 7.12 kB +7.12 kB (new file) 🆕
packages/frontend/dist/assets/MusicConfig-C_u7ZmPx.js 0 B -1.61 kB (removed) 🏆
packages/frontend/dist/assets/MusicConfig-CxZ5KE8j.js 1.61 kB +1.61 kB (new file) 🆕
packages/frontend/dist/assets/PreferredArtists-B0B1momR.js 3.7 kB +3.7 kB (new file) 🆕
packages/frontend/dist/assets/PreferredArtists-Ba5OtAgS.js 0 B -3.7 kB (removed) 🏆
packages/frontend/dist/assets/ReactionRoles-CHyqKKjH.js 0 B -1.89 kB (removed) 🏆
packages/frontend/dist/assets/ReactionRoles-DJqY15J7.js 1.89 kB +1.89 kB (new file) 🆕
packages/frontend/dist/assets/ServerLogs-CQvZVZQI.js 0 B -2.91 kB (removed) 🏆
packages/frontend/dist/assets/ServerLogs-JM3Ihr6_.js 3.04 kB +3.04 kB (new file) 🆕
packages/frontend/dist/assets/ServerSettings-DCGNfS98.js 0 B -4.22 kB (removed) 🏆
packages/frontend/dist/assets/ServerSettings-DGD0OZoZ.js 4.34 kB +4.34 kB (new file) 🆕
packages/frontend/dist/assets/ServersPage-CvKUWH_V.js 2.92 kB +2.92 kB (new file) 🆕
packages/frontend/dist/assets/ServersPage-DORDnsBf.js 0 B -2.92 kB (removed) 🏆
packages/frontend/dist/assets/Starboard-xMQagbjU.js 0 B -2.08 kB (removed) 🏆
packages/frontend/dist/assets/Starboard-Y308f3Xq.js 2.08 kB +2.08 kB (new file) 🆕
packages/frontend/dist/assets/TrackHistory-CTr8qqIG.js 2.17 kB +2.17 kB (new file) 🆕
packages/frontend/dist/assets/TrackHistory-DYt7teyQ.js 0 B -2.17 kB (removed) 🏆
packages/frontend/dist/assets/TwitchNotifications-BLjhsfmf.js 0 B -2.44 kB (removed) 🏆
packages/frontend/dist/assets/TwitchNotifications-bNlBduQp.js 2.44 kB +2.44 kB (new file) 🆕
packages/frontend/dist/assets/useFeatures-BejlShmW.js 0 B -2.03 kB (removed) 🏆
packages/frontend/dist/assets/useFeatures-mdDFIoid.js 2.02 kB +2.02 kB (new file) 🆕
ℹ️ View Unchanged
Filename Size
packages/frontend/dist/assets/ActionPanel-BtdtlQ9o.js 403 B
packages/frontend/dist/assets/api-AUPUM2Qa.js 3.37 kB
packages/frontend/dist/assets/authStore-Cu1BkXvq.js 556 B
packages/frontend/dist/assets/badge-vPK-qUOM.js 501 B
packages/frontend/dist/assets/Card-C_uHUkFW.js 509 B
packages/frontend/dist/assets/CustomCommands-CU1vBphA.js 2.16 kB
packages/frontend/dist/assets/dialog-DSHSwnxX.js 951 B
packages/frontend/dist/assets/guildStore-il-poOxg.js 791 B
packages/frontend/dist/assets/input-Yz475ZAJ.js 469 B
packages/frontend/dist/assets/LastFm-DiX1-rJk.js 1.74 kB
packages/frontend/dist/assets/PrivacyPolicy-D3x5lQf7.js 1.38 kB
packages/frontend/dist/assets/rolldown-runtime-BYbx6iT9.js 471 B
packages/frontend/dist/assets/SectionHeader-B-f_yjhM.js 895 B
packages/frontend/dist/assets/select-DM_YBjDr.js 1.23 kB
packages/frontend/dist/assets/shim-Qmczh487.js 510 B
packages/frontend/dist/assets/Skeleton-WB3VdUho.js 238 B
packages/frontend/dist/assets/Spotify-DMj24LIB.js 1.75 kB
packages/frontend/dist/assets/StatTile-B79rBPkY.js 639 B
packages/frontend/dist/assets/switch-HLSMi6pZ.js 546 B
packages/frontend/dist/assets/TermsOfService-B_oLkpxV.js 1.37 kB
packages/frontend/dist/assets/usePageMetadata-DVjH87aq.js 329 B
packages/frontend/dist/assets/utils-TE4_8V5I.js 149 B
packages/frontend/dist/assets/vendor-forms-VJTTJemd.js 25.6 kB
packages/frontend/dist/assets/vendor-radix-DKbKEiWx.js 38.9 kB
packages/frontend/dist/assets/vendor-react-DNaNL-Gj.js 55.6 kB
packages/frontend/dist/assets/vendor-state-B_fYO9zl.js 24.1 kB
packages/frontend/dist/assets/vendor-ui-D9aFDhli.js 65 kB

compressed-size-action

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LucasSantana-Dev has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@sonarqubecloud

Copy link
Copy Markdown

@LucasSantana-Dev

Copy link
Copy Markdown
Owner Author

Closing per ADR docs/decisions/2026-05-21-close-stale-ui-pr-stack.md — 8-PR stack drained to free solo-dev attention. Branch ui/server-tools-2026-05-15 is preserved on origin; this redesign for the server-logs + server-settings surface is recoverable via rebase + reopen. Tracking issue will list all 8 surfaces + a revisit gate. Closes a stack flagged by audit-deep as PR-queue work, not backlog.

This branch was successfully deployed

1 active deployment
Preview — 1af28f50 Deployed May 15, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file frontend size/m

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant