Skip to content
This repository has been archived by the owner on Jul 1, 2019. It is now read-only.

Alters checkbox label to clarify that it’s optional #520

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

matthewpaulthomas
Copy link

Done

  • Changes “I agree to receive information about Canonical’s products and services.” to “Also notify me of events and new products”

In detail:

  • “Also”: clarifies that the checkbox is distinct from what else you’re subscribing to. It also avoids the need to label the checkbox as “(optional)”.
  • “notify me of”: Shorter, and more assertive, than “I agree to receive information about”.
  • “events and new products”: a summary of @caldav’s “an event, a webinar, a product launch, etc.”.
  • “.”: A checkbox label should not end with a full stop.

QA

  • Check out this feature branch
  • Run the site using the command ./run serve
  • View the site locally in your web browser at: http://0.0.0.0:8023/
  • Scroll to the bottom of the “Newsletter signup” form.

Issue / Card

Fixes #437.

@webteam-app
Copy link

@pmahnke
Copy link
Contributor

pmahnke commented Apr 18, 2019

@matthewpaulthomas can you run this by Katherine Ollerhead as well?

@matthewpaulthomas
Copy link
Author

@pmahnke Ok, I’ve mailed Katherine.

@matthewpaulthomas
Copy link
Author

@pmahnke I’ve forwarded you Allegra’s reply.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can subscribe to newsletter without checking “I agree to receive information”
3 participants