Clarify distinction between Dashboard widgets

[coleshaw]

Fixes #13952

Changes proposed in this Pull Request:

• No functional changes, just visual changes to clarify role of each column in the Dashboard footer widget.

Is this a new feature or does it add/remove features to an existing part of Jetpack?

• If you're an Automattician, include a shortlink to the p2 discussion with Jetpack Product here.

N/A

Testing instructions:

• Go to the admin Dashboard
• View the Jetpack footer widget
• See headings for "Jetpack Protect" and "Akismet" (see screenshots for examples).

Proposed changelog entry for your changes:

• Add headings in Dashboard footer widget. Update copy for consistent verbiage.

[jetpackbot]

	Warnings
⚠️	pre-commit hook was skipped for one or more commits

This is an automated check which relies on PULL_REQUEST_TEMPLATE. We encourage you to follow that template as it helps Jetpack maintainers do their job. If you think 'Testing instructions' or 'Proposed changelog entry' are not needed for your PR - please explain why you think so. Thanks for cooperation 🤖

Generated by 🚫 dangerJS against b61df6e

[jeherve]

Thank you for your contribution. I left some comments below.

[coleshaw]

For reference, here is what the updated widget should look like.

[keoshi]

Thanks for doing this @coleshaw !

Does that last screenshot reflect the latest changes? If so, could we replace the first one on the original post, please?

This is looking good to me, but I'd like to see two changes before merging:

• Smaller font-size on the product name headings.
• Slightly larger margin-bottom on those same headings.

This should make the product names complementary to the valuable information below: the actual stats and descriptions. I've added a mock on #13952 (comment) :

Finally, should we be using the “Anti-spam” name instead of Akismet @jeherve ?

[jeherve]

should we be using the “Anti-spam” name instead of Akismet

That's a good point, let's do that!

We can take that opportunity to change "Protect" into "Brute force attack protection" as well, to stay consistent with the Jetpack Settings screen.

[coleshaw]

@keoshi @jeherve , thanks for the feedback. I've updated the code and posted new screenshots in the original PR description. I also updated a couple instances of "Protect" and "Akismet" in the widget, not just in the heading -- do those other changes make sense? Or do you only want the headings to use "Brute force attack protection" and "Anti-spam", and I should leave the other text strings alone?

[jeherve]

do those other changes make sense?

I think your changes make sense! 👍

[jeherve]

This works well, even with long strings:

[keoshi]

Looks great! Thanks again, @coleshaw !