Skip to content
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

PR: Update the "About GWHAT" window and reduce startup time. #120

Merged
merged 3 commits into from
Dec 11, 2017

Conversation

jnsebgosselin
Copy link
Owner

The objective of this PR is to update the text in the About GWHAT window to reflect the changes that were made for the copyrights of the software.

Also, in this PR the creation of the About GWHAT window is delayed when it is actually required, instead of generating it on startup. The objective is to reduce the startup time of GWHAT as discussed in Issue #56.

image

Instead of creating the GWHAT window on startup, it is created only when
the button about gwhat is clicked.
@pep8speaks
Copy link

pep8speaks commented Dec 11, 2017

Hello @jnsebgosselin! Thanks for updating the PR.

Line 15:1: E402 module level import not at top of file
Line 16:1: E402 module level import not at top of file
Line 17:1: E402 module level import not at top of file

Line 129:80: E501 line too long (87 > 79 characters)
Line 133:80: E501 line too long (86 > 79 characters)

Comment last updated on December 11, 2017 at 21:51 Hours UTC

@jnsebgosselin
Copy link
Owner Author

For the record, here is a screenshot of the About GWHAT window before this PR:

image

@jnsebgosselin jnsebgosselin merged commit d231d07 into master Dec 11, 2017
@jnsebgosselin jnsebgosselin deleted the about_what_update branch December 11, 2017 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants