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
Qt: Support versions < 5.6 #6008
Conversation
a4e5f9d
to
cbdd649
Compare
c2b1a47
to
db91a67
Compare
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.
Looks good, one main thing:
- What does initializing a QKeySequence with zero mean? Should it be default-initialized instead?
and two small things:
addMenuAction
andaddToolAction
could probably be combined and extracted into some QtUtils header- Dolphin style guide says function namss should start with an uppercase letter
ea0e1b2
to
bdbbdb9
Compare
Source/Core/CMakeLists.txt
Outdated
if(ENABLE_QT2) | ||
add_subdirectory(DolphinQt2) | ||
endif() | ||
add_subdirectory(DolphinQt2) |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
For the record: All buildbots have passed when forced to compile DolphinQt2:
|
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.
Nice!
No description provided.