Admin Page: hide Related Posts preview when not active.

[jeherve]

Fixes #10691

Changes proposed in this Pull Request:

• We now hide the Related Posts preview when the module is not active.

Testing instructions:

• Go to Settings > Traffic
• Activate and deactivate the Related Posts module, and make sure the preview appearing below the toggles is updated accordingly.

Proposed changelog entry for your changes:

• Admin Page: hide Related Posts preview when not active.

[jetpackbot]

Thank you for the great PR description!

When this PR is ready for review, please apply the [Status] Needs Review label. If you are an a11n, please have someone from your team review the code if possible. The Jetpack team will also review this PR and merge it to be included in the next Jetpack release.

Scheduled Jetpack release: January 10, 2019.
Scheduled code freeze: January 3, 2019

Generated by 🚫 dangerJS against 9ffde11

[keoshi]

Works great!

Though I have to say that in the context of “early value” this leaves me a bit conflicted. Isn't showing a preview providing more value than hiding it behind a toggle? In that sense, shouldn't Highlight related content with a heading and Show a thumbnail image where available also be hidden? I can see the argument for simplifying the UI, but IMO this shows less value early on. cc @rickybanister @joanrho

[rickybanister]

I see that point @keoshi, but this is the only feature that did that. Nothing else hints at what will happen after it's turned on. I guess I'd rather consistency here and we work on systematically making it easier for customers to understand.

Perhaps we consider making the info bubbles much less useless and putting a case study inside them instead of a silly link to documentation. By 'case study' I mean a screenshot of the feature in use on the front-end where applicable and a pitch of the value proposition of the feature.

Or something similar.

[oskosk]

Works as expected!