Make notice on Privacy Policy page to 'Check out our guide' dismissable

[garrett-eclipse]

This ticket branches from #44669 with this comment from @norcross;
https://core.trac.wordpress.org/ticket/44669?replyto=4#comment:4

The notice reference can be found here;
​https://github.com/WordPress/WordPress/blob/f7ba175491b725a5f3d636072c8b432774d38ae3/wp-admin/includes/misc.php#L1613-L1629

With this notice the is-dismissable class could be added;
​https://codex.wordpress.org/Plugin_API/Action_Reference/admin_notices

If a new Privacy Policy page is created this notice should re-appear until dismissed.

[garrett-eclipse]

Notice to make dismissable

[garrett-eclipse]

Make notice dismissable

[garrett-eclipse]

Added the .is-dismissable class to the privacy policy notice.

[audrasjb]

This ticket is triaged in milestone 5.0.3. The patch looks good, small and self-contained enough to land in the next minor. Adding commit keyword to see if it can land in 5.0.3.

[desrosj]

This falls outside the 5.0.3 scope (block editor related updates, major bugs, regressions), so let's tackle it in 5.1.

[pento]

This patch doesn't add the close button for me. @garrett-eclipse, could you investigate, please?

[garrett-eclipse]

Notice w/ Close Button

[garrett-eclipse]

Thanks @pento,

Sorry about that, had it working in /build and hadn't applied to /src before making that original diff.

Latest patch is correct with proper spelling of is-dismissible class and calls makeNoticesDismissible on wp-pp-notice to invoke the function to inject the button.

Minor notes;
Upon reload the notice will re-appear which is expected.
As well just this was started pre-5.0 so this notice currently only displays with Classic Editor enabled.

Please apply the latest and let me know if there's any issues.

[pento]

Thank you for refreshing the patch, @garrett-eclipse.

I think it needs to remember that the notice has been dismissed. As you noted, refreshing the page makes it reappear, which includes when you click "Save Draft". A dismissed notice that keeps coming back is a more frustrating experience than a non-dismissable notice. 😉

[garrett-eclipse]

Thanks for the feedback @pento I tend to agree would be nice to save the dismissed state of the notice there.

I can definitely work on that functionality but am hoping for a little direction if you don't mind.
To keep the notice dismissed I would be adding a flag either as a Cookie, Session data or User Meta and am wondering if there's an existing standard (maybe an example) for this in core?

As well whatever storage method is used I feel should get cleared when the policy text has changed so as to re-introduce the guide link.
If I do that it would make sense to be done here along side the policy_text_changed_notice notice which re-appears on the privacy.php screen.
And by here I mean - ​https://github.com/WordPress/WordPress/blob/51155f3989c29bbae5322b6cba58952a7c58b0ba/wp-admin/includes/misc.php#L1428

I'm leaning towards using User Meta over Cookie/Session data as it pertains across browsers/devices and is a more permanent dismissal but would love your thoughts.

Thanks

[garrett-eclipse]

#46098 introduces the notice to the block editor. The persistent dismiss will need to be hooked into both notices.

[garrett-eclipse]

Some notes from #46098 to help with persistence on Gutenberg version of the notice.

As the dismiss isn't persistent in that a reload will revive the notice we should remove the dismiss from this notice and address it along w/ the dismissal for the classic editor version in #44981.

Yeah, I agree. I think it should also be a feature built-in to the notices module, and could probably leverage existing persistence functionality offered to data stores (of which notices extends) via the data persistence plugin.

​https://github.com/WordPress/gutenberg/tree/master/packages/data/src/plugins/persistence