D24932: [WIP]: Add Button to open the folder in filelight for more details
Nathaniel Graham
noreply at phabricator.kde.org
Sat Oct 26 03:36:35 BST 2019
ngraham added a comment.
Much better, thanks.
INLINE COMMENTS
> kpropertiesdialog.cpp:1097
> +
> + if (QStandardPaths::findExecutable(QStringLiteral("filelight")) != QString()) {
> + d->m_sizeDetailsButton = new QPushButton(i18n("Explore in Filelight?"), d->m_frame);
`isEmpty()` is preferred over comparing to `QString()`
> kpropertiesdialog.cpp:1100
> + d->m_sizeDetailsButton->setIcon(QIcon::fromTheme(QStringLiteral("filelight")));
> + connect(d->m_sizeDetailsButton, &QAbstractButton::clicked, this, &KFilePropsPlugin::slotSizeDetails);
> + }
This is a `QPushButton`, so connect to `&QPushButton::clicked`
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D24932
To: shubham, ngraham, #frameworks
Cc: kde-frameworks-devel, #frameworks, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191026/333de617/attachment.html>
More information about the Kde-frameworks-devel
mailing list