fix(dashboard): enqueue wu-styling on network admin dashboard for activity-stream widget#768
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe pull request adds an explicit enqueue of the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
🔨 Build Complete - Ready for Testing!📦 Download Build Artifact (Recommended)Download the zip build, upload to WordPress and test:
🌐 Test in WordPress Playground (Very Experimental)Click the link below to instantly test this PR in your browser - no installation needed! Login credentials: |
…ivity-stream widget
The activity-stream widget wraps output in <div class="wu-styling">, which
requires framework.css (registered as 'wu-styling'). The network admin dashboard
is not a wp-ultimo page so enqueue_default_admin_styles() skips it — add an
explicit wp_enqueue_style('wu-styling') call in enqueue_scripts() which already
guards on $pagenow === 'index.php'.
Fixes #767
7d2306c to
09cc78b
Compare
🔨 Build Complete - Ready for Testing!📦 Download Build Artifact (Recommended)Download the zip build, upload to WordPress and test:
🌐 Test in WordPress Playground (Very Experimental)Click the link below to instantly test this PR in your browser - no installation needed! Login credentials: |
|
Performance Test Results Performance test results for a70f4a3 are in 🛎️! Note: the numbers in parentheses show the difference to the previous (baseline) test run. Differences below 2% or 0.5 in absolute values are not shown. URL:
|
Summary
The activity-stream dashboard widget wraps its output in
<div class="wu-styling">, which requiresframework.css(registered as thewu-stylinghandle). The network admin dashboard is not a wp-ultimo admin page, soenqueue_default_admin_styles()skips it — the stylesheet is never enqueued and the widget renders unstyled.Fix
Add
wp_enqueue_style('wu-styling')inDashboard_Widgets::enqueue_scripts()before the existing script enqueues. The method already guards on$pagenow === 'index.php', so the enqueue only fires on the dashboard.Files Changed
inc/class-dashboard-widgets.php— addwp_enqueue_style('wu-styling')inenqueue_scripts()Testing
PHPStan level 0: no errors on changed file.
Change is a single
wp_enqueue_style()call in an already-guarded method — no logic changes.Resolves #767
aidevops.sh v3.6.167 plugin for OpenCode v1.3.16 with claude-sonnet-4-6 spent 4m and 7,613 tokens on this as a headless worker.
Summary by CodeRabbit