-
Notifications
You must be signed in to change notification settings - Fork 722
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
CV64: Add option groups #3360
CV64: Add option groups #3360
Conversation
A couple of these option descriptions would probably be better as single longer lines to avoid having awkward line breaks in the middle of sentences whose exact display varies by browser settings. |
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.
This looks fine to me. Up to you if you want to further adjust the docstring formatting.
Got rid of all mid-sentence line breaks and I think I'm pretty happy with it now. I also decided to decrease the default value for |
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.
LGTM
* Add the option groups. * Get rid of all mid-sentence line breaks.
* Add the option groups. * Get rid of all mid-sentence line breaks.
* Add the option groups. * Get rid of all mid-sentence line breaks.
What is this fixing or adding?
Adds option groups for CV64 and changes the formatting of the option descriptions to remove indentations whenever there's a manual line break.
How was this tested?
Ran the WebHost and made sure it all looked good there.
If this makes graphical changes, please attach screenshots.