-
Notifications
You must be signed in to change notification settings - Fork 185
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
select dropdown changes capitalization when displaying choice values #247
Comments
That is because the display uses Line 92 in e7024bd
I think most times it's more aesthetically pleasing to have title case but for abbreviations it's not a good thing. I'm open to suggestions on how to get the best of both worlds. |
citing from scripture "Explicit is better than implicit." Wooey should not on it's own decide how to display values which were given by the user. |
I specify a "choice" in my argparser, which Wooey properly displays as a select dropdown.
However, it unexpectedly changes the capitalization of the displayed values.
parser.add_argument("--currency", default="USD", choices=['USD', 'EUR', 'CHF' ,'JPY'], required=False)
In wooey this now becomes:
Note how capitalization of the choice options is changing.
This is unexpected. I did expect that the dropdown displayed values are exactly as specified.
The script still works, because internally it still used the capitalized versions as values. Only the display seems to be affected.
The text was updated successfully, but these errors were encountered: