Closed
Bug 1874949
Opened 9 months ago
Closed 8 months ago
Add triggers for Review Checker setting changes
Categories
(Firefox :: Messaging System, task, P1)
Firefox
Messaging System
Tracking
()
RESOLVED
WONTFIX
Iteration:
124.1 - Jan 22 - Feb 2
People
(Reporter: aminomancer, Unassigned)
References
(Blocks 1 open bug)
Details
Planned feature callouts 4 and 5 will trigger when a user:
- disables "automatically open Review Checker"
- disables Review Checker
This requires 2 new FxMS triggers. There are 2 possible ways to do this:
- Listen to the prefs controlled by these toggles and fire the trigger when the prefs changes.
- In the forthcoming UI code, in the click listeners' callbacks, directly invoke
ASRouter.sendTriggerMessage
with an appropriately named trigger.
The latter depends more on the UI, which doesn't exist yet, but it's also more precisely what we want. So the ideal approach depends on when we start work on this. We can start on 1 immediately since we can decide what the names of the prefs will be, and they will be used in the UI code, or else if the UI patch uses different pref names it can also change them in the trigger listener.
Comment 1•8 months ago
|
||
Potentially uplift candidate for Fx123, dependency adding review checker settings in about:preferences in progress in Fx124
Iteration: --- → 124.1 - Jan 22 - Feb 2
Points: --- → 5
Priority: -- → P1
Reporter | ||
Comment 2•8 months ago
|
||
preferenceObserver trigger might be good enough for this.
Reporter | ||
Updated•8 months ago
|
Points: 5 → ---
Reporter | ||
Comment 3•8 months ago
|
||
Not needed thanks to preferenceObserver
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•