NUX: Make disabling tips instance specific. #12633
Labels
[Feature] NUX
Anything that impacts the new user experience
[Type] Enhancement
A suggestion for improvement.
Is your feature request related to a problem? Please describe.
If I disable tips in the Block Editor, I don't want that to hide tips for all other apps in my WordPress install.
Describe the solution you'd like
Something like
context
that would function similarly to thenotices
package.areTipsEnabled
,enableTips
anddisableTips
would all take an optionalcontext
parameter that defaults toglobal
. Perhaps that'd include modifying the<DotTip />
component to include acontext
prop, it could be inferred from thetipId
but that is almost certainly not correct for "shared spaces" like the editor.Alternately, the context could be set for the entire page load and would just effect the reducer ( which would in turn mean they are persisted differently in
localStorage
. Likewp.nux.setContext()
.For clarity, I mean a reducer shape something like this.
Or if was important to distinguish the
null
state.Describe alternatives you've considered
Heavily monkey patching
isTipVisible()
. #12632 would alleviate this need a bit since it'd be less likely that users would completely disable tips.The text was updated successfully, but these errors were encountered: