-
Notifications
You must be signed in to change notification settings - Fork 8
Docs/elevation #85
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
Docs/elevation #85
Conversation
- moved drawer markdowns to legacy - fixed links - added migration guide to sidebar - fixed wrong quotation marks - fixed grammar mistakes
✅ Deploy Preview for industrial-experience ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
@flxlst09 I cannot request your review since you created the PR, but please feel free to review anyways 😊 |
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.
Very understandable, some comments left for improvement. TBD in a meeting.
Improved the structure and clarity of the elevation documentation by renaming elevation levels, updating tips, and refining usage guidance. Added new illustrative images for elevation and interaction states. Clarified recommendations for color token usage for better readability.
@flxlst09 applied your suggestions, feel free to re-review 😊 |
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.
Only a few small things to change - therefore I've already approved.
Co-authored-by: tokyojen <[email protected]>
IX-3076
💡 What is the current behavior?
Elevation not described in docu
🆕 What is the new behavior?
Elevation principle described and documented
-> also fixed some broken links from the drawer deprecation where the pipeline complained 😋