-
Notifications
You must be signed in to change notification settings - Fork 338
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: guidance for UX related PRs #4950
Conversation
.github/PULL_REQUEST_TEMPLATE.md
Outdated
@@ -12,6 +12,20 @@ | |||
- Screenshots |
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.
Screenshots are already mentioned here. Adding more text is unlikely to be read by contributors, we want to keep the instructions as minimal and relevant as possible.
Is dark/light theme a common enough topic that it needs to be a permanent part of the PR template?
22d2473
to
7e70539
Compare
7e70539
to
8497ff0
Compare
.github/PULL_REQUEST_TEMPLATE.md
Outdated
@@ -9,7 +9,8 @@ | |||
- Update the changelog using `npm run newChange`. | |||
- Link to related issues/commits. | |||
- Testing: how did you test your changes? | |||
- Screenshots | |||
- Screenshots (if the pull request is related to UI/UX then please include light and dark theme screenshots) | |||
- UX Reviews and Sign-off |
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.
UX review/signoff is more of a project-level requirement which would be tracked in our project system.
Update PR template to include guidance for UI/UX related PRs
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.