Skip to content
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

Admin Page: add the new Apps and Plans landing pages #4345

Merged
merged 6 commits into from
Jul 19, 2016

Conversation

eliorivero
Copy link
Contributor

@eliorivero eliorivero commented Jul 8, 2016

Fixes #4303.

Changes proposed in this Pull Request:

  • adds the new Apps landing page
  • adds the new Plans landing page

@eliorivero eliorivero added [Status] In Progress Admin Page React-powered dashboard under the Jetpack menu labels Jul 8, 2016
@eliorivero eliorivero added this to the 4.3-internal-beta milestone Jul 8, 2016
@eliorivero eliorivero force-pushed the add/admin-new-landing-pages branch from aebf679 to d8899dd Compare July 8, 2016 17:28
@eliorivero eliorivero force-pushed the add/admin-new-landing-pages branch from 04eb8ac to 0c0302d Compare July 8, 2016 21:55
@richardmuscat richardmuscat modified the milestones: 4.3-public-beta, 4.3-internal-beta Jul 11, 2016
@eliorivero eliorivero changed the title Admin Page: add the new Apps landing page Admin Page: add the new Apps and Plans landing pages Jul 14, 2016
@eliorivero eliorivero added [Status] Needs Review This PR is ready for review. and removed [Status] In Progress labels Jul 14, 2016
@eliorivero eliorivero force-pushed the add/admin-new-landing-pages branch from 7c1941e to 4a9f1c6 Compare July 14, 2016 13:33
…ortCard only when user is in jetpack_premium or jetpack_business plans.
<div className="jp-jetpack-landing__img-text">
<div className="jp-jetpack-landing__column">
<h2>{ __( 'Feel the performance' ) }</h2>
<p>{ __( "All the WordPress apps are built for spped. You'll notice the difference in performance immediately, with near-instant page-loads and less waiting around." ) }</p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in the word speed.

@zinigor
Copy link
Member

zinigor commented Jul 15, 2016

Code-wise looks great except for several nitpicks, well done, thank you!
And one more thing - images do not load for me without the extension, I have tried both Firefox and Chrome. When I edit the source and add .png at the end, they appear as they should.

@zinigor zinigor added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Jul 15, 2016
@eliorivero
Copy link
Contributor Author

Solved issues pointed by @zinigor.

@eliorivero eliorivero force-pushed the add/admin-new-landing-pages branch from 0ba9fb4 to 9fffb70 Compare July 19, 2016 12:32
@eliorivero eliorivero merged commit 2176cff into master Jul 19, 2016
@eliorivero eliorivero deleted the add/admin-new-landing-pages branch July 19, 2016 14:05
@eliorivero eliorivero removed the [Status] Ready to Merge Go ahead, you can push that green button! label Jul 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Page React-powered dashboard under the Jetpack menu
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants