fix: hidden continue button on multisite setup wizard complete step (#842)#853
fix: hidden continue button on multisite setup wizard complete step (#842)#853superdav42 merged 2 commits intomainfrom
Conversation
…site setup wizard complete step The 'Continue to Multisite Ultimate Setup' and 'Refresh and Check Again' buttons on the wp-ultimo-multisite-setup page were invisible. Both used wu-bg-blue-600 / wu-bg-green-600 which are not compiled into framework.css. With wu-text-white present but no background colour, the result was white text on a white/transparent background — invisible. Replace both buttons with standard WordPress button-primary/button-large classes which are always styled by WP Admin CSS regardless of theme. Also fix related info-box CSS classes that reference -50 and -200 shade variants not present in framework.css (bg-blue-50, bg-green-50, bg-gray-50, border-blue-200, border-green-200, text-blue-800) with the closest available equivalents (-100/-500 shades). Fixes #842
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 28 minutes and 23 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 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 |
Merge SummaryIssue: #842 — Hidden continue button on the multisite setup wizard complete step Root cause: and in Fix: Replaced both invisible buttons with standard WP Files changed:
Verified: All existing tests pass (exit 0) locally. |
🔨 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 a884293 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:
|
🔨 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: |
Summary
Resolves #842
The Continue to Multisite Ultimate Setup and Refresh and Check Again buttons on the
wp-ultimo-multisite-setupwizard'sstep=completepage were completely invisible. Users could not proceed after enabling WordPress Multisite.Root Cause
section_complete()anddisplay_manual_instructions()inclass-multisite-setup-admin-page.phpused Tailwind utility classes that are not compiled intoframework.css:wu-bg-blue-600hover:wu-bg-blue-700wu-bg-green-600hover:wu-bg-green-700wu-text-whiteis in the CSS, so both buttons rendered white text on a transparent (white) background — invisible.Fix
button button-primary button-largeclasses, which are always styled by WP Admin CSS regardless of admin colour scheme.framework.csswith the nearest available alternatives:wu-bg-blue-50→wu-bg-blue-100wu-bg-green-50→wu-bg-green-100wu-bg-gray-50→wu-bg-gray-100wu-border-blue-200→wu-border-blue-500wu-border-green-200→wu-border-green-300wu-text-blue-800→wu-text-blue-700Testing
test_section_complete_button_uses_wp_button_classes_not_missing_tailwind— assertsbutton-primaryis present andwu-bg-blue-600/wu-bg-blue-700are absent.test_display_manual_instructions_button_uses_wp_button_classes— assertsbutton-primaryis present andwu-bg-green-600/wu-bg-green-700are absent.Multisite_Setup_Admin_Page_Testtests continue to pass (exit 0).Verification