-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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 form to use drop downs #34458
Conversation
This is a minor change that removes the checkboxes since that causes a todo list functionality to be triggered in the UI, making it hard for contributors to keep track of what's happening.
Messed up the second dropdown item
Any idea how this looks like in the end? |
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 tried this in a private repo, it looks like this:
Huzzah! Thank you @priethor for the review, the commit, and the screenshot :) |
It seems a bit inconvenient to need to open the dropdown, then select the option. It's two clicks that I doubt people will do to select Given there is no option to set default values to preselect |
Based on feedback, this updates the form to make the dropdowns required.
Great idea - just made the required! |
Description
This is a minor change that removes the checkboxes since that causes a todo list functionality to be triggered in the UI, making it hard for contributors to keep track of what's happening.
How has this been tested?
N/A
Screenshots
Types of changes
Enhancement
Checklist:
*.native.js
files for terms that need renaming or removal).