fix: update audit logs page layout classes and add newline at EOF

[impoiler]

Summary

Fixes the layout of the Audit Logs page to correctly fill the viewport and apply the appropriate background and border styles.

Changes

• Replaced h-[calc(100dvh-1rem)] with h-[calc(100vh-16px)] for consistent viewport height calculation
• Swapped mx-auto flex flex-col p-4 utility classes for no-border-parent bg-background flex to align with the layout conventions used elsewhere in the app
• Added missing newline at end of file

Type of change

• Bug fix
• Feature
• Refactor
• Documentation
• Chore/CI

Affected areas

• Core (Go)
• Transports (HTTP)
• Providers/Integrations
• Plugins
• UI (React)
• Docs

How to test

Navigate to the Audit Logs page and verify:

• The page fills the full viewport height without overflow or clipping
• The background color and border styling match the rest of the workspace layout

cd ui
pnpm i || npm i
pnpm build || npm run build

Screenshots/Recordings

Add before/after screenshots showing the corrected Audit Logs page layout.

Breaking changes

• Yes
• No

Related issues

Security considerations

None.

Checklist

• I read docs/contributing/README.md and followed the guidelines
• I added/updated tests where appropriate
• I updated documentation where needed
• I verified builds succeed (Go and UI)
• I verified the CI pipeline passes locally if applicable

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 12eb52b1-1ffa-40fd-b212-027fb9e7b21d

📥 Commits

Reviewing files that changed from the base of the PR and between d478309 and 3ab32b4.

📒 Files selected for processing (1)

• ui/app/workspace/audit-logs/page.tsx

🚧 Files skipped from review as they are similar to previous changes (1)

• ui/app/workspace/audit-logs/page.tsx

---

📝 Walkthrough

Summary by CodeRabbit

• Style
  • Updated the Audit Logs page layout to a full-width, borderless row flex design with refreshed background styling.
  • Adjusted the page’s height/viewport fit for more consistent on-screen presentation while keeping the audit log view visible.

Walkthrough

The audit logs page wrapper styling was updated to use a full-width flex layout and a different viewport height calculation. The AuditLogsView component remains rendered in the same place.

Changes

Audit logs page layout

Layer / File(s)	Summary
Page wrapper layout ui/app/workspace/audit-logs/page.tsx	The outer wrapper around AuditLogsView changes Tailwind classes, including width, background, flex direction, and height calculation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I hop through layouts, light and free,
With bg-background around me.
A wider page, a taller sky,
Yet AuditLogsView still waves hi.
🐇✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: updating Audit Logs page layout classes and the EOF newline.
Description check	✅ Passed	The description follows the template and covers summary, changes, type, affected area, testing, screenshots, breaking changes, issues, security, and checklist.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch 06-26-fix_audit_logs_page_width

---

_{Comment @coderabbitai help to get the list of available commands.}

[impoiler]

• feat: add collapsible tag limit to TagInput #4730
• fix: update audit logs page layout classes and add newline at EOF #4719 👈 (View in Graphite)
• refactor: extract TruncatedLabel component and replace inline truncation logic #4716
• feat: add TruncatedLabel component with tooltip on overflow #4715
• fix: use parseAsSafeArrayOf for logs page array query params to handle special characters in URLs #4714
• dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

[greptile-apps]

Confidence Score: 5/5

Safe to merge — the change is a one-line CSS class swap with no logic, data flow, or component structure changes.

The only touched file is a thin page wrapper that passes directly to AuditLogsView. The class changes align with the established full-viewport pattern visible in dashboard, mcp-logs, and logs pages, and no data-testid attributes, state, or API calls are involved.

No files require special attention.

Important Files Changed

Filename	Overview
ui/app/workspace/audit-logs/page.tsx	Layout wrapper classes updated to match the full-viewport pattern used in dashboard and mcp-logs pages; newline added at EOF. No logic, data, or testid changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[AuditLogsPage wrapper div] -->|before| B["no-padding-parent mx-auto flex h-[calc(100dvh-1rem)] w-full flex-col p-4"]
    A -->|after| C["no-padding-parent no-border-parent bg-background flex h-[calc(100vh-16px)] w-full"]
    C --> D[AuditLogsView]
    B --> D

Loading

%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A[AuditLogsPage wrapper div] -->|before| B["no-padding-parent mx-auto flex h-[calc(100dvh-1rem)] w-full flex-col p-4"]
    A -->|after| C["no-padding-parent no-border-parent bg-background flex h-[calc(100vh-16px)] w-full"]
    C --> D[AuditLogsView]
    B --> D

Loading

_{Reviews (4): Last reviewed commit: "fix: audit logs page width" | Re-trigger Greptile}

[akshaydeo]

Merge activity

• Jun 26, 1:47 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jun 26, 1:51 PM UTC: @akshaydeo merged this pull request with Graphite.

The base branch was changed.