You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Bug: "Enable Mobile Responsive" toggle resets to on after every notification update #125
The "Enable Mobile Responsive" toggle (Customize → Themes → Mobile in the notification builder) automatically resets to on every time an existing notification is updated, even if the user had explicitly turned it off and saved it that way. The user's saved preference is silently overwritten on every save — there is no UI indication this is happening.
Steps to reproduce
Create any NotificationX notification (e.g. WooCommerce Sales, WordPress reviews, etc.).
Go to Customize → Themes → Mobile, toggle "Enable Mobile Responsive" to off.
Save the notification.
Reopen the same notification from the list and click Update without changing anything else.
Reopen it once more.
Expected: the toggle stays off, as last saved. Actual: the toggle is back to on; mobile responsive is silently re-enabled.
NotificationX Pro: 3.1.2 (Pro is not involved — the bug is in Free)
WordPress: latest
Tested in current Chrome / Safari
Root cause
In nxdev/notificationx/components/WrapperWithLoader.tsx, a useEffect keyed on builderContext.values.type fires whenever the type changes — including when an existing notification's saved type populates the form on edit. Inside that effect, setFieldValue("is_mobile_responsive", nx_type !== "custom") is called unconditionally, overwriting the user's saved toggle before they even touch the form.
Pull request: #124 — guards the override behind an nx_id check so the type-change default only applies when creating a new notification; existing entries keep the user's saved value.
Activity