dfaure accepted this revision.
dfaure added a comment.
Thanks :-)
BRANCH
recentfilemenu
REVISION DETAIL
https://phabricator.kde.org/D26448
To: nicolasfella, #frameworks, dfaure
Cc: broulik, elvisangelaccio, cfeck, kde-frameworks-devel, LeGast00n, cblack,
michaelh, ngraham, bruns
nicolasfella updated this revision to Diff 83348.
nicolasfella added a comment.
- Use menus font metric
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D26448?vs=83346=83348
BRANCH
recentfilemenu
REVISION DETAIL
https://phabricator.kde.org/D26448
AFFECTED FILES
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.15/99/
Project:
kf5-qt5 FreeBSDQt5.15
Date of build:
Thu, 13 Aug 2020 10:46:45 +
Build duration:
6 min 10 sec and counting
JUnit Tests
Name:
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.15/98/
Project:
kf5-qt5 FreeBSDQt5.15
Date of build:
Thu, 13 Aug 2020 10:24:57 +
Build duration:
21 min and counting
JUnit Tests
Name: projectroot
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks/job/kparts/job/kf5-qt5%20WindowsMSVCQt5.14/57/
Project:
kf5-qt5 WindowsMSVCQt5.14
Date of build:
Thu, 13 Aug 2020 10:03:45 +
Build duration:
2 min 32 sec and counting
JUnit Tests
dfaure added inline comments.
INLINE COMMENTS
> krecentfilesmenu.cpp:74
> +{
> +action = new QAction(titleWithSensibleWidth(qobject_cast *>(menu->parent(;
> +QObject::connect(action, ::triggered, action, [this, menu]()
> {
QMenu is a QWidget. Why not just pass `menu` as
