1453276 - Improve mma documentation for "Pocket in Top Sites"

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Metrics, enhancement, P1)

Description

[Petru-Mugurel Lingurar [:petru]]

As per :liuche and because the patch for 1452173 has already been merged

> +    "Pocket in Top Sites" -> true       // If Top Sites home panel contain websites recommended by Pocket
>
> Nit: this sounds kind of passive, maybe something like "If Pocket recommendations are enabled (by default or through user action)"

I think a more accurate description for the "Pocket in Top Sites" attribute would be "If Pocket recommendations for Top Sites home panel are enabled (by default or through user action)"

Comment 1

[Petru-Mugurel Lingurar [:petru]]

Attachment: Bug 1453276 - Improve mma documentation for "Pocket in Top Sites";

Review commit: https://reviewboard.mozilla.org/r/235602/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/235602/

Comment 2

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Comment on attachment 8966923 [details]
Bug 1453276 - Improve mma documentation for "Pocket in Top Sites";

https://reviewboard.mozilla.org/r/235602/#review241944

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ba90e25685c6
Improve mma documentation for "Pocket in Top Sites"; r=mcomella

Comment 4

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/ba90e25685c6