Skip to content

Conversation

@danieldudzic
Copy link
Collaborator

Description

This PR fixes the broken settings notifications.

  • Add Notifications component using WordPress SnackbarList
  • Add SuccessIcon and ErrorIcon SVG components replacing emoji icons
  • Update useNotices hook with auto-dismiss (5 seconds) and manual dismiss button
  • Style notifications to match WooCommerce notifications (dark background, blur effect, proper spacing)
  • Refactor existing components to use the centralized useNotices hook

Screenshots

WooCommerce notifications PayPal Settings notifications
WooCommerce_settings_‹_WooCommerce_PayPal_Payments_—_WordPress WooCommerce_settings_‹_WooCommerce_PayPal_Payments_—_WordPress

Steps to Test

  1. Go to 'PayPal Payments settings'
  2. Click on 'Restore dismissed Things To Do'
  3. Verify success notification appears with green checkmark icon
  4. Verify notification auto-dismisses after 5 seconds
  5. Click the '✕' button to manually dismiss

@github-actions
Copy link

Test using WordPress Playground

The changes in this pull request can be previewed and tested using a WordPress Playground instance.
WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser.

🔗 Test this pull request with WordPress Playground

What's included:

  • ✅ WordPress (latest)
  • ✅ WooCommerce (latest)
  • ✅ PayPal Payments plugin v3.3.2-pr4034-21183242905-g612e44a (built from this PR)

Login credentials:

  • Username: admin
  • Password: password

Plugin Details:

  • Version: 3.3.2-pr4034-21183242905-g612e44a
  • Commit: 612e44a
  • Artifact: woocommerce-paypal-payments-3.3.2-pr4034-21183242905-g612e44a

💡 The demo environment resets each time you refresh. Perfect for testing!

🔄 This link updates automatically with each new commit to the PR.

⚠️ This URL is valid for 30 days from when this comment was last updated.


🤖 Auto-generated for commit 612e44a • Last updated: 2026-01-20T18:45:02.770Z

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.

1 participant