Skip to content

Enqueue WooCommerce settings-embed styles to more closely mimic released UI#11

Merged
vbelolapotkov merged 4 commits into
trunkfrom
try/enqueue-woocommerce-settings-embed-styles
Jul 7, 2026
Merged

Enqueue WooCommerce settings-embed styles to more closely mimic released UI#11
vbelolapotkov merged 4 commits into
trunkfrom
try/enqueue-woocommerce-settings-embed-styles

Conversation

@daledupreez

@daledupreez daledupreez commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

This PR explores manually enqueuing the settings-embed styles from WooCommerce admin to more closely replicate the visual styles used in the WooCommerce Subscriptions UI for managing global plans in WooCommerce -> Settings -> Subscriptions, and tweaking some of the CSS to adjust the default padding states.

This builds on the refactor in PR #10.

Screenshot

Screenshot 2026-07-02 at 20 55 39

@daledupreez daledupreez requested a review from vbelolapotkov July 3, 2026 00:48
@daledupreez daledupreez requested a review from a team as a code owner July 3, 2026 00:48
Base automatically changed from update/plans-ui-to-match-woocommerce-subscriptions-9-0-0 to trunk July 3, 2026 00:59
daledupreez and others added 2 commits July 3, 2026 03:18
Resolve by dropping the manual settings-embed style registration: since
the plans manager moved into the wc-settings screen (#12), WooCommerce
core already enqueues the settings-embed stylesheet there itself
(handle wc-admin-style, loaded after this plugin's styles), so the
manual copy only duplicated the same file. The actions-column padding
tweak is the remaining change.
@vbelolapotkov vbelolapotkov merged commit 5b6ff08 into trunk Jul 7, 2026
2 checks passed
@vbelolapotkov vbelolapotkov deleted the try/enqueue-woocommerce-settings-embed-styles branch July 7, 2026 10:56
@vbelolapotkov

Copy link
Copy Markdown
Contributor

Merged to continue iterations on plans UI in follow-up PRs.

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.

2 participants