-
Notifications
You must be signed in to change notification settings - Fork 22
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
TV VariableTable max precision column option #1670
base: master
Are you sure you want to change the base?
Conversation
* isFractional function added to isolate conditional statement
* isFractional function added to isolate conditional statement
…by default) * precision column visibility functions added.
Thank you for the PR. We'll pull this ourselves and test here in our development environment for a bit, to make sure it flows with our established user interface experience and to determine if there are any lurking bugs to squash. If we feel like it's a good fit we'd be happy to merge into our codebase and give you due credit for your contribution. |
Is there any progress on this PR? @sharmeye |
Although I don't actively develop Trick anymore, I stumbled across this PR today, and, as the original Trick View author, was compelled to offer some feedback:
|
View Dropdown menu has Max Precision option which is not selected by default.
If user selects this option Variable Table will show the precision column that has the following features:
This feature is related to #1642
I find practical to use a comboBox for precision since there can only be limited number of significance.
Example images of the feature;