Skip to content

Replace alerts with WP notices; UI/SCSS/tests - #860

Open
murshed wants to merge 1 commit into
WordPress:trunkfrom
murshed:trunk
Open

Replace alerts with WP notices; UI/SCSS/tests#860
murshed wants to merge 1 commit into
WordPress:trunkfrom
murshed:trunk

Conversation

@murshed

@murshed murshed commented Aug 1, 2026

Copy link
Copy Markdown

Replace window.alert calls with WordPress notices (createSuccessNotice/createErrorNotice) and delay navigation/reload by 1s so snackbars are visible. Import notices store where needed. Minor UI tweaks: replace edit icon with pencil and add aria-hidden to icons. Migrate SCSS @import usages to @use for @wordpress/base-styles (mixins and colors). Add unit tests for generateWpVersions covering normal and RC version inputs. These changes improve UX by showing non-blocking notifications and modernize styles imports and tests.

Replace window.alert calls with WordPress notices (createSuccessNotice/createErrorNotice) and delay navigation/reload by 1s so snackbars are visible. Import notices store where needed. Minor UI tweaks: replace edit icon with pencil and add aria-hidden to icons. Migrate SCSS @import usages to @use for @wordpress/base-styles (mixins and colors). Add unit tests for generateWpVersions covering normal and RC version inputs. These changes improve UX by showing non-blocking notifications and modernize styles imports and tests.
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