-
Notifications
You must be signed in to change notification settings - Fork 851
My plan: update free plan page #11418
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Thank you for the great PR description! When this PR is ready for review, please apply the Scheduled Jetpack release: April 2, 2019. |
Making Security & Performance the two top focuses
|
@keoshi thanks for starting the build on this! So far, it works great on testing. I'm not sure about the last card that condenses the 2-col wide paid plan upsell card @jeffgolenski originally had designed in his mockups in the Issue...so, let's discuss more on Monday. Unrelated to this PR, but worth mentioning here while I'm testing: the many notifications that pop up on my first visit to this page (some new Coupon code thing?), Jumpstart, and everything below the Free plan features are distracting. I've made a note to address this as part of Improved Onboarding. |
Related: #10752 |
|
@jeffgolenski I believe I removed all up-sells — which ones are you referring to? |
|
@keoshi I'm still seeing the prompt to download the app in the footer. I suppose that's not technically an up-sell, but a cross-sell. Can we work that into one of the cards you think?... actually, do we have metrics on that CTAs success? Also, we should take a look at fixing up the plans grid page as well. First off, just removing the giant themes up-sell banner (after we look at CR metrics, of course) |
|
@jeffgolenski, thanks for the feedback!
Yep, I think that's a good idea, but take a look at the data I sent you on Slack. The amount of clicks on that banner is non negligible, and the card seems to be pretty effective (as annoying as it might be).
Yep, let's do it on a separate PR though since this is for the My Plan page. Can you open a new issue for that, please? |
|
@keoshi Per our chat, we can bump the Plans Page modifications to our Project 10. Jetpack Plans Marketing Improvements |
|
I've updated this PR @jeffgolenski @joanrho — would appreciate another quick design review. |
jeherve
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like that change, and I don't see any issues with it. The page is so much more useful to me that way, I can finally go to "My Plan" and actually know what plan I am on! 🎉
🚢
joanrho
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just tested and looks/works as expected. Really nice work @keoshi, thank you! 🚢
* Initial Changelog for 7.2 * Testing list: add mention of IE11 testing * Initial Changelog for 7.2 * Testing list: add mention of IE11 testing * Add CL for #11224 * Add CL for #11426 * Add CL for #11442 * Add testing instructions for #11224 * Add CL for #11451 * Reclassify CL item * Add testing instructions for #11451 * Add CL for #11486 * Add CL for #11418 * Add CL for #11524 * Add CL and testing instructions for #11449 * Add CL for #11460 * Add CL for #11520 and #11582 * Add CL for #11531 * Add CL #11644 * Add testing instructions for #11644 * Add testing instructions for #11644 * Add CL for #11618 * Uniform changelog lines * CL #11679 * CL #11661 * CL #11654 * CL #11645 * CL #11643 * CL #11636 * CL #11635 and for other PHPCS commits * CL #11627 * CL #11626 * CL #11598 * CL #11596 * Remove nested items for shortcopy. I don't believe the detailed list is helpful * CL #11570 * CL #11569 * CL #11560 * CL #11558 * CL #11555 * CL #6704 * CL #11298 * CL #11324 * CL #11443 * CL #11484 * CL #11516 * CL #11529 * Expand Ads block enhancement CL item


Fixes #10995
Changes proposed in this Pull Request:
Testing instructions:
My Planpage.Before
After
Proposed changelog entry for your changes: