-
-
Notifications
You must be signed in to change notification settings - Fork 86
Tracking Branch for the Ember Website Redesign RFC #145
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
0e0626e
to
88c0566
Compare
88c0566
to
0109d72
Compare
feee86e
to
d577c76
Compare
Remove tabs implementation
remove es-ulist from next version of styleguide
Resolve accessibility issues with the navbar
Optimise Ember Logo
implementing basic background shapes
improve typography and color docs
Set body and text color
Upgrade sparkles components to glimmer
This PR is now ready to be reviewed and merged 🎉 I have updated the description with more context for people and a comment about what should or should not be reviewed 👍 Feel free to ask me any questions you might have |
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.
This looks great ✨ Thank you for making this awesome initiative happen!
Since this is already in production, I think we should merge it even if changes are needed. SemVer will help us track the work and do updates. Thanks everyone for your work on this! |
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.
Ship it! 👍
This PR has been tracking the work for the Ember Styleguide needed for the Website Redesign RFC. This branch has already been released as pre-releases and is in use on the Ember Website right now.
As we are moving into the next phase of the redesign project it is time to release this work as a stable release 🎉 😱
This PR is too big to review the code directly, but every change was made using a PR to this branch and each of those PRs were reviewed individually. If you would like to review this PR before we merge it then I would recommend reviewing the output by looking at the new documentation: https://deploy-preview-145--ember-styleguide.netlify.app/
After this PR is merged to master I will create a new stable release from the master branch 👍