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

DQt2: About Dialog #3623

Merged
merged 1 commit into from
Mar 9, 2016
Merged

DQt2: About Dialog #3623

merged 1 commit into from
Mar 9, 2016

Conversation

rukai
Copy link
Contributor

@rukai rukai commented Feb 12, 2016

Comparison between wx bottom and Qt top.
prscreenshot_wx_vs_qt

The Lioncash new and improved edition.
thelioncashnewandimprovededition

Review on Reviewable

@delroth
Copy link
Member

delroth commented Feb 12, 2016

What's up with the links looking terrible?

@rukai
Copy link
Contributor Author

rukai commented Feb 12, 2016

-_- I mixed up the screenshots. wx is bottom and Qt is top.


Comments from the review on Reviewable.io

@rukai rukai force-pushed the dolphinQtAbout branch 2 times, most recently from 280b574 to 3c1b243 Compare February 12, 2016 12:31
@lioncash
Copy link
Member

Review status: 0 of 7 files reviewed at latest revision, 7 unresolved discussions.


a discussion (no related file):
The font size for the Dolphin title likely shouldn't that thick/bolded.


Source/Core/DolphinQt2/AboutDialog.cpp, line 31 [r1] (raw file):
I don't think adding an interface descriptor is necessary.


Source/Core/DolphinQt2/AboutDialog.cpp, line 40 [r1] (raw file):
Should probably resolve this TODO.


Source/Core/DolphinQt2/AboutDialog.cpp, line 56 [r1] (raw file):
main_layout


Source/Core/DolphinQt2/AboutDialog.cpp, line 68 [r1] (raw file):
missing newline


Source/Core/DolphinQt2/AboutDialog.h, line 14 [r1] (raw file):
missing newline


Source/Core/DolphinQt2/MenuBar.cpp, line 44 [r1] (raw file):

connect(documentation, &QAction::triggered, this, [=]() {    
    QDesktopServices::openUrl(QUrl(QStringLiteral("https://dolphin-emu.org/docs/guides")));
});

Comments from the review on Reviewable.io

@rukai rukai force-pushed the dolphinQtAbout branch 2 times, most recently from 3f96800 to 3ed5228 Compare February 12, 2016 12:55
@lioncash
Copy link
Member

Review status: 0 of 7 files reviewed at latest revision, 4 unresolved discussions.


Source/Core/DolphinQt2/AboutDialog.cpp, line 1 [r2] (raw file):
2016


Source/Core/DolphinQt2/AboutDialog.cpp, line 9 [r2] (raw file):
DolphinQt/AboutDialog.h


Source/Core/DolphinQt2/AboutDialog.cpp, line 10 [r2] (raw file):
DolphinQt/Resources.h


Source/Core/DolphinQt2/AboutDialog.h, line 1 [r2] (raw file):
2016


Comments from the review on Reviewable.io

@JosJuice
Copy link
Member

Is information like the revision hash selectable and copyable, like in the old DolphinQt (PR #2833)?

@rukai
Copy link
Contributor Author

rukai commented Feb 13, 2016

Yes it is.


Review status: 0 of 7 files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

lioncash added a commit that referenced this pull request Mar 9, 2016
@lioncash lioncash merged commit 1c7f5b1 into dolphin-emu:master Mar 9, 2016
@leoetlino leoetlino modified the milestone: Qt Sep 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants