Skip to content

Fix: UI: AI Request Logs: page background inconsistent with other admin screens and DataView border radius clipped#676

Merged
dkotter merged 2 commits into
WordPress:developfrom
hbhalodia:fix/issue-670
Jun 5, 2026
Merged

Fix: UI: AI Request Logs: page background inconsistent with other admin screens and DataView border radius clipped#676
dkotter merged 2 commits into
WordPress:developfrom
hbhalodia:fix/issue-670

Conversation

@hbhalodia

@hbhalodia hbhalodia commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

What?

Closes #670

Why?

  • PR aims to add the consistency accross the pages and it resolves the border and border radius issues.

How?

  • Added custom background color, add overflow hidden to container so it does not overflows.

Use of AI Tools

Testing Instructions

  1. Open the page here - wp-admin/tools.php?page=ai-request-logs
  2. Check the background color, it should be consistent with other pages.
  3. Check for the dataviews layout border, the inner container should not overflow outside.

Screenshots or screencast

Screenshot 2026-06-05 at 1 56 45 PM Screenshot 2026-06-05 at 1 56 52 PM Screenshot 2026-06-05 at 1 56 59 PM

Changelog Entry

Fixed - UI inconsistency on AI Request Logs page.

Open WordPress Playground Preview

@codecov

codecov Bot commented Jun 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.68%. Comparing base (4f27b52) to head (7423fb4).

Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #676   +/-   ##
==========================================
  Coverage      74.68%   74.68%           
  Complexity      1754     1754           
==========================================
  Files             85       85           
  Lines           7549     7549           
==========================================
  Hits            5638     5638           
  Misses          1911     1911           
Flag Coverage Δ
unit 74.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: hbhalodia <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@dkotter dkotter left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks nicer so going to approve and merge, though I think there's additional things we can do here, like making the header full width to more closely match other pages

@dkotter dkotter merged commit 9383c9f into WordPress:develop Jun 5, 2026
25 of 43 checks passed
@dkotter dkotter added this to the 1.1.0 milestone Jun 5, 2026
@simison

simison commented Jun 5, 2026

Copy link
Copy Markdown
Member

This looks nicer so going to approve and merge, though I think there's additional things we can do here, like making the header full width to more closely match other pages

This would be great indeed :-) You can then set max-width on the contents instead of full container.

A good step would be bringing this page to routes/* and built with WP Build: that'll make it consistent with all the other new WP core pages, rather than lean to older settings pages visually.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI: AI Request Logs: page background inconsistent with other admin screens and DataView border radius clipped

3 participants