-
Notifications
You must be signed in to change notification settings - Fork 163
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
Initial improvements to upgrade guide #4119
Conversation
Demo starting at https://vanilla-framework-4119.demos.haus |
82a9744
to
2ceeab0
Compare
|
2ceeab0
to
cc23243
Compare
The most recent commit is with some fixes from the QA I did. The text-max-width section still needs rewording but we can update it once we add the class |
FYI - |
Also, the grid looks broken in this demo. Just noting it here so I don't forget to mention it |
cc23243
to
2b92e0c
Compare
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.
LGTM 👍🏼 Had a look through the epic and it's got everything I can think of apart from rewording the text max width section to make it clearer - I'll add an issue for that
Co-authored-by: Beth Collins <beth.collins@canonical.com>
Initial improvements around 3.0 upgrade guide.
To be continues in canonical-web-and-design/vanilla-squad#1024 epic
QA
Check upgrade guide: https://vanilla-framework-4119.demos.haus/docs/upgrade-guide-v3
Make sure it reads ok
Make sure any missing things are listed as part of canonical-web-and-design/vanilla-squad#1024 epic