-
Notifications
You must be signed in to change notification settings - Fork 93
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
gcylc.rc enhancements #1849
gcylc.rc enhancements #1849
Conversation
print >> sys.stderr, ("WARNING: window size values must be " | ||
"positive. Using default.") | ||
fail = True | ||
# TODO: check for daft window sizes? (10, 5), (80000, 5000) ? |
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.
Worth doing or leave it up to GTK?
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.
Does GTK check for sensible sizes?
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.
On my desktop it would appear so, though I can't find anything in the gtk documentation to confirm this.
Review 1 - good. |
Yep, tests OK, looks good. |
Close #934 (bullet two)
Closes #817
Closes #1622
Added the following settings to gcylc.rc:
sort column
(gcylc.rc: configure which column should be sorted by default #1622)sort column ascending
(gcylc.rc: configure which column should be sorted by default #1622)transpose dot
(More customisation options in gcylc.rc #934)transpose graph
(More customisation options in gcylc.rc #934)window size
(cylc gui: gcylc.rc configuration for window size #817)Open to suggestions for better names for these settings.
@hjoliver Please Review
@benfitzpatrick Please Review