-
Notifications
You must be signed in to change notification settings - Fork 12
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
R30 Release Announcement #722
Conversation
Visit the preview URL for this PR (updated for commit f460156): https://blui-doc-it--pr722-feature-5150-r30-rel-m0ygx7up.web.app (expires Fri, 26 Jan 2024 05:23:11 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 590ed6fb71e127776beff0a05819e22fd4307b8f |
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.
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 wonder if this should be BLUI 4.0.
- The phrase "Material Design 3" is repeated three times in the summary. Consider making it more succinct.
- Avoid saying
@brightlayer-ui/colors/branding package
, as this is confusing. - Your "package summary" section is not used correctly. We divided it by frameworks so that React devs will read only react-related stuff. Some shared things, like templates, colors, and icons, are grouped together. Though it would be beneficial to call out the two deprecated packages.
- For the "5 new components have been added" line, the components need to be linked to dev doc.
- I am personally not a fan of "as per". I think there's only "per" or "according to" in modern American English.
- Changelogs need to link to the master branch.
Co-authored-by: huayunh <[email protected]>
Fixes #BLUI-5150 .
Changes proposed in this Pull Request:
Screenshots / Screen Recording (if applicable)
To Test:
Any specific feedback you are looking for?