-
Notifications
You must be signed in to change notification settings - Fork 26.9k
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
Update bug_report.yml #12991
Update bug_report.yml #12991
Conversation
Those placeholders look weird... Can we instead put those texts into the description above the edit box and explicitly mention that the are examples? |
which placeholder you think it's weird specifically, or do you mean all of them as we have seen in practice, people don't read descriptions
I think placeholder makes it more obvious and can give actual demonstration of what we want if you think some placeholder is too much feel free to change them |
People who don't read descriptions will not read placeholders. The way it is in the PR is confusing even for me. I feel that all placeholders are too much and all examples should be in descriptions, not in placeholders. |
less placeholder version only on test report not pushed to this branch yet |
Looks great! |
pushed less placeholder version |
Description
update bug report template
add a checklist with simple debug instruction
use
placehoder
for example of what should be filled inso that people have to actually fill in and not just leave the template in
and give actual examples of what we actually expect
perfect example of a type of issue post that I'm trying to avoid
https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/12996
demo of how it looks when filling the form
https://github.com/w-e-w/ext-temp-test/issues/new?assignees=&labels=bug-report&projects=&template=bug_report.yml&title=%5BBug%5D%3A+
demo of how it looks after it's posted
w-e-w/ext-temp-test#31
Checklist: