KDE CI: Frameworks kcoreaddons kf5-qt5 WindowsMSVCQt5.9 - Build # 22 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kcoreaddons%20kf5-qt5%20WindowsMSVCQt5.9/22/ Project: Frameworks kcoreaddons kf5-qt5 WindowsMSVCQt5.9 Date of build: Fri, 24 Nov 2017 01:58:10 + Build duration: 2 hr 51 min and counting

KDE CI: Frameworks plasma-framework kf5-qt5 WindowsMSVCQt5.9 - Build # 65 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20WindowsMSVCQt5.9/65/ Project: Frameworks plasma-framework kf5-qt5 WindowsMSVCQt5.9 Date of build: Thu, 23 Nov 2017 23:39:06 + Build duration: 3 hr 14 min and counting

KDE CI: Frameworks kcoreaddons kf5-qt5 FreeBSDQt5.7 - Build # 64 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kcoreaddons%20kf5-qt5%20FreeBSDQt5.7/64/ Project: Frameworks kcoreaddons kf5-qt5 FreeBSDQt5.7 Date of build: Fri, 24 Nov 2017 01:58:10 + Build duration: 4 min 56 sec and counting JUnit

KDE CI: Frameworks kcoreaddons kf5-qt5 SUSEQt5.10 - Build # 5 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kcoreaddons%20kf5-qt5%20SUSEQt5.10/5/ Project: Frameworks kcoreaddons kf5-qt5 SUSEQt5.10 Date of build: Fri, 24 Nov 2017 01:58:10 + Build duration: 3 min 24 sec and counting JUnit Tests

KDE CI: Frameworks kcoreaddons kf5-qt5 SUSEQt5.7 - Build # 3 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kcoreaddons%20kf5-qt5%20SUSEQt5.7/3/ Project: Frameworks kcoreaddons kf5-qt5 SUSEQt5.7 Date of build: Fri, 24 Nov 2017 01:58:10 + Build duration: 3 min 14 sec and counting JUnit Tests

D8958: Fix unintentional breadcrumb menu item activation

2017-11-23 Thread Алексей Шилин
aleksejshilin updated this revision to Diff 22866. aleksejshilin added a comment. - Don't consider mouse moves which are smaller than drag distance - Don't pass the ignored mouse release event to the base class REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE

KDE CI: Frameworks plasma-framework kf5-qt5 WindowsMSVCQt5.9 - Build # 64 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20WindowsMSVCQt5.9/64/ Project: Frameworks plasma-framework kf5-qt5 WindowsMSVCQt5.9 Date of build: Thu, 23 Nov 2017 19:15:05 + Build duration: 4 hr 23 min and counting

D8979: ecm_add_test: Use proper path sep on Windows

2017-11-23 Thread David Faure
dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Ah, oops, indeed. REPOSITORY R240 Extra CMake Modules BRANCH master REVISION DETAIL https://phabricator.kde.org/D8979 To: kfunk, dfaure, cgiboudeaux, bcooksley Cc: #windows,

D8979: ecm_add_test: Use proper path sep on Windows

2017-11-23 Thread Kevin Funk
kfunk added a reviewer: cgiboudeaux. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D8979 To: kfunk, dfaure, cgiboudeaux Cc: #frameworks, #build_system

D8979: ecm_add_test: Use proper path sep on Windows

2017-11-23 Thread Kevin Funk
kfunk added a reviewer: bcooksley. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D8979 To: kfunk, dfaure, cgiboudeaux, bcooksley Cc: #windows, #frameworks, #build_system

D8979: ecm_add_test: Use proper path sep on Windows

2017-11-23 Thread Kevin Funk
kfunk added a subscriber: Windows. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D8979 To: kfunk, dfaure, cgiboudeaux Cc: #windows, #frameworks, #build_system

D8979: ecm_add_test: Use proper path sep on Windows

2017-11-23 Thread Kevin Funk
kfunk created this revision. kfunk added a reviewer: dfaure. Restricted Application added projects: Frameworks, Build System. Restricted Application added subscribers: Build System, Frameworks. REVISION SUMMARY We need to separate paths in environment variables with a semincolon (;) on

D8976: Remove redundant map look-ups

2017-11-23 Thread Kevin Funk
kfunk accepted this revision. This revision is now accepted and ready to land. REPOSITORY R282 NetworkManagerQt BRANCH master REVISION DETAIL https://phabricator.kde.org/D8976 To: apol, #frameworks, jgrulich, kfunk

KDE CI: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.7 - Build # 150 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20FreeBSDQt5.7/150/ Project: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.7 Date of build: Thu, 23 Nov 2017 20:49:37 + Build duration: 12 min and counting

KDE CI: Frameworks plasma-framework kf5-qt5 SUSEQt5.7 - Build # 7 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.7/7/ Project: Frameworks plasma-framework kf5-qt5 SUSEQt5.7 Date of build: Thu, 23 Nov 2017 20:49:37 + Build duration: 10 min and counting JUnit Tests

KDE CI: Frameworks plasma-framework kf5-qt5 SUSEQt5.10 - Build # 4 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.10/4/ Project: Frameworks plasma-framework kf5-qt5 SUSEQt5.10 Date of build: Thu, 23 Nov 2017 20:49:37 + Build duration: 10 min and counting JUnit

KDE CI: Frameworks kirigami kf5-qt5 WindowsMSVCQt5.9 - Build # 116 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20WindowsMSVCQt5.9/116/ Project: Frameworks kirigami kf5-qt5 WindowsMSVCQt5.9 Date of build: Thu, 23 Nov 2017 13:53:03 + Build duration: 6 hr 45 min and counting JUnit

D8098: Strip down and re-write the baloo tags KIO slave

2017-11-23 Thread James Smith
smithjd added inline comments. INLINE COMMENTS > anthonyfieroni wrote in kio_tags.h:101 > metaData(QString{}) Doesn't work. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D8098 To: smithjd, #frameworks, vhanda, #dolphin, ngraham, dfaure Cc: anthonyfieroni, dfaure,

D8098: Strip down and re-write the baloo tags KIO slave

2017-11-23 Thread James Smith
smithjd updated this revision to Diff 22857. smithjd marked 3 inline comments as done. smithjd added a comment. - Review suggestions. REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8098?vs=22792=22857 BRANCH master-nestedTags (branched from master)

KDE CI: Frameworks plasma-framework kf5-qt5 WindowsMSVCQt5.9 - Build # 63 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20WindowsMSVCQt5.9/63/ Project: Frameworks plasma-framework kf5-qt5 WindowsMSVCQt5.9 Date of build: Thu, 23 Nov 2017 13:26:03 + Build duration: 5 hr 48 min and counting

D8971: Port from QDom to QXmlStreamReader

2017-11-23 Thread Aleix Pol Gonzalez
apol added a comment. +1 REPOSITORY R292 KUnitConversion REVISION DETAIL https://phabricator.kde.org/D8971 To: broulik, #frameworks Cc: apol, anthonyfieroni

D8976: Remove redundant map look-ups

2017-11-23 Thread Aleix Pol Gonzalez
apol created this revision. apol added reviewers: Frameworks, jgrulich. Restricted Application added a project: Frameworks. REPOSITORY R282 NetworkManagerQt BRANCH master REVISION DETAIL https://phabricator.kde.org/D8976 AFFECTED FILES src/manager.cpp To: apol, #frameworks, jgrulich

D8971: Port from QDom to QXmlStreamReader

2017-11-23 Thread Kai Uwe Broulik
broulik added a comment. `qFuzzyIsNull` rather. The original code didn't even check .. REPOSITORY R292 KUnitConversion REVISION DETAIL https://phabricator.kde.org/D8971 To: broulik, #frameworks Cc: anthonyfieroni

D8971: Port from QDom to QXmlStreamReader

2017-11-23 Thread Anthony Fieroni
anthonyfieroni added inline comments. INLINE COMMENTS > currency.cpp:713 > +const auto multiplier = > attributes.value(QLatin1String("rate")).toDouble(); > +if (multiplier != 0) { > +

D8332: Added baloo urls into places model

2017-11-23 Thread Nathaniel Graham
ngraham added a comment. I'm going to land this on 11/25/17 unless I hear any more objections from anyone. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D8332 To: renatoo, #frameworks, #dolphin, #kde_applications, dvratil, #vdg, ngraham, ervin, mlaurent, dfaure,

D8332: Added baloo urls into places model

2017-11-23 Thread David Faure
dfaure accepted this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D8332 To: renatoo, #frameworks, #dolphin, #kde_applications, dvratil, #vdg, ngraham, ervin, mlaurent, dfaure, mwolff Cc: mwolff, dfaure, ervin, usta, mlaurent, dvratil, ngraham, #frameworks

D7660: Fix a regression caused by changing backspace key behavior

2017-11-23 Thread Safa Alfulaij
safaalfulaij added a comment. I can commit but it is showing as not accepted. So I'll just wait for the final confirmation. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D7660 To: safaalfulaij, #ktexteditor, jgrulich, hein, dhaumann, mwolff, cullmann Cc:

D8971: Port from QDom to QXmlStreamReader

2017-11-23 Thread Kai Uwe Broulik
broulik created this revision. broulik added a reviewer: Frameworks. Restricted Application added a project: Frameworks. REVISION SUMMARY The Qt Xml module is not actively maintained anymore and docs suggest to use QXmlStreamReader. Also removes the dependency on Qt Xml TEST PLAN I can

D8732: Fix race that starts kactivitymanagerd multiple times

2017-11-23 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R6:b4bdd796079c: Fix race that starts kactivitymanagerd multiple times (authored by davidedmundson). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D8732?vs=22112=22819#toc REPOSITORY R6

D8348: Add a section for removable devices

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22811. renatoo added a comment. Parent branch updated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8348?vs=22776=22811 REVISION DETAIL https://phabricator.kde.org/D8348 AFFECTED FILES

D8947: Expose KFilePlacesModel 'iconName' role

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22817. renatoo added a comment. Parent branch updated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8947?vs=22782=22817 REVISION DETAIL https://phabricator.kde.org/D8947 AFFECTED FILES

D8944: Create a 'KFilePlacesModel::refresh' function

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22814. renatoo added a comment. Parent branch updated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8944?vs=22779=22814 REVISION DETAIL https://phabricator.kde.org/D8944 AFFECTED FILES

D8434: Created 'remote' section

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22812. renatoo added a comment. Parent branch updated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8434?vs=22777=22812 REVISION DETAIL https://phabricator.kde.org/D8434 AFFECTED FILES

D8943: Create 'KFilePlacesModel::convertedUrl' static function

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22813. renatoo added a comment. Parent branch updated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8943?vs=22778=22813 REVISION DETAIL https://phabricator.kde.org/D8943 AFFECTED FILES

D8948: Created an auxiliary function 'KFilePlacesModel::movePlace'

2017-11-23 Thread David Faure
dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D8948 To: renatoo, dfaure Cc: #frameworks

D8946: Avoid unnecessary 'dataChanged' signal

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22816. renatoo added a comment. Parent branch updated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8946?vs=22781=22816 REVISION DETAIL https://phabricator.kde.org/D8946 AFFECTED FILES

D8948: Created an auxiliary function 'KFilePlacesModel::movePlace'

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22818. renatoo added a comment. Parent branch updated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8948?vs=22809=22818 REVISION DETAIL https://phabricator.kde.org/D8948 AFFECTED FILES

D8945: Return a valid bookmark object for any entry in KFilePlacesModel

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22815. renatoo added a comment. Parent branch updated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8945?vs=22780=22815 REVISION DETAIL https://phabricator.kde.org/D8945 AFFECTED FILES

D8970: Use https for downloading currency exchange rates

2017-11-23 Thread Kai Uwe Broulik
broulik created this revision. broulik added a reviewer: Frameworks. Restricted Application added a project: Frameworks. TEST PLAN Removed my `~/.local/share/libkunitconversion/currency.xml` file, restarted krunner, entered `5EUR` and got the file properly created. I verified with `curl` that

D8332: Added baloo urls into places model

2017-11-23 Thread Renato Oliveira Filho
renatoo added a comment. @dvratil, @ervin any comments? REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D8332 To: renatoo, #frameworks, #dolphin, #kde_applications, dvratil, #vdg, ngraham, ervin, mlaurent, dfaure, mwolff Cc: mwolff, dfaure, ervin, usta, mlaurent,

KDE CI: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.7 - Build # 149 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20FreeBSDQt5.7/149/ Project: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.7 Date of build: Thu, 23 Nov 2017 15:04:34 + Build duration: 22 min and counting

KDE CI: Frameworks plasma-framework kf5-qt5 SUSEQt5.7 - Build # 6 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.7/6/ Project: Frameworks plasma-framework kf5-qt5 SUSEQt5.7 Date of build: Thu, 23 Nov 2017 15:04:34 + Build duration: 7 min 14 sec and counting JUnit

KDE CI: Frameworks plasma-framework kf5-qt5 SUSEQt5.10 - Build # 3 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.10/3/ Project: Frameworks plasma-framework kf5-qt5 SUSEQt5.10 Date of build: Thu, 23 Nov 2017 15:04:34 + Build duration: 6 min 40 sec and counting

D8963: [Containment Interface] Set WA_TranslucentBackground before accessing winId()

2017-11-23 Thread Kai Uwe Broulik
broulik created this revision. broulik added reviewers: Plasma, fvogt. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY This would create the surface after which this flag has no effect causing menus

KDE CI: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.7 - Build # 148 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20FreeBSDQt5.7/148/ Project: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.7 Date of build: Thu, 23 Nov 2017 13:26:03 + Build duration: 11 min and counting

KDE CI: Frameworks kirigami kf5-qt5 FreeBSDQt5.7 - Build # 148 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20FreeBSDQt5.7/148/ Project: Frameworks kirigami kf5-qt5 FreeBSDQt5.7 Date of build: Thu, 23 Nov 2017 11:29:22 + Build duration: 51 min and counting JUnit Tests

KDE CI: Frameworks plasma-framework kf5-qt5 SUSEQt5.10 - Build # 2 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.10/2/ Project: Frameworks plasma-framework kf5-qt5 SUSEQt5.10 Date of build: Thu, 23 Nov 2017 13:26:03 + Build duration: 6 min 11 sec and counting

KDE CI: Frameworks plasma-framework kf5-qt5 SUSEQt5.7 - Build # 5 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.7/5/ Project: Frameworks plasma-framework kf5-qt5 SUSEQt5.7 Date of build: Thu, 23 Nov 2017 13:26:03 + Build duration: 6 min 6 sec and counting JUnit

D8965: [PlasmaComponents Menu] Set WA_TranslucentBackground on the underlying QMenu

2017-11-23 Thread Kai Uwe Broulik
broulik created this revision. broulik added reviewers: Plasma, fvogt. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY This ensures the menu can have nice round corners with Breeze and Oxygen. The

D8963: [Containment Interface] Set WA_TranslucentBackground before accessing winId()

2017-11-23 Thread Kai Uwe Broulik
broulik edited the summary of this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8963 To: broulik, #plasma, fvogt Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D8965: [PlasmaComponents Menu] Set WA_TranslucentBackground on the underlying QMenu

2017-11-23 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8965 To: broulik, #plasma, fvogt, davidedmundson Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai,

KDE CI: Frameworks kirigami kf5-qt5 SUSEQt5.10 - Build # 4 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20SUSEQt5.10/4/ Project: Frameworks kirigami kf5-qt5 SUSEQt5.10 Date of build: Thu, 23 Nov 2017 13:53:03 + Build duration: 1 min 16 sec and counting JUnit Tests

KDE CI: Frameworks kirigami kf5-qt5 FreeBSDQt5.7 - Build # 149 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20FreeBSDQt5.7/149/ Project: Frameworks kirigami kf5-qt5 FreeBSDQt5.7 Date of build: Thu, 23 Nov 2017 13:53:03 + Build duration: 1 min 39 sec and counting JUnit Tests

D8963: [Containment Interface] Set WA_TranslucentBackground before accessing winId()

2017-11-23 Thread Kai Uwe Broulik
broulik edited the summary of this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8963 To: broulik, #plasma, fvogt Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D8964: Fix two bugs in KMessageWidget

2017-11-23 Thread Albert Astals Cid
aacid updated this revision to Diff 22827. aacid added a comment. Small tweak to the test REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8964?vs=22825=22827 BRANCH master REVISION DETAIL https://phabricator.kde.org/D8964 AFFECTED FILES

D8964: Fix two bugs in KMessageWidget

2017-11-23 Thread Albert Astals Cid
aacid created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY Bug #1: If you call show() or setVisible() after animatedHide() nothing would happen This is because animatedHide() does

D8965: [PlasmaComponents Menu] Set WA_TranslucentBackground on the underlying QMenu

2017-11-23 Thread Kai Uwe Broulik
broulik edited the summary of this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8965 To: broulik, #plasma, fvogt Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D8964: Fix two bugs in KMessageWidget

2017-11-23 Thread Albert Astals Cid
aacid added a reviewer: Frameworks. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D8964 To: aacid, #frameworks Cc: #frameworks

D8963: [Containment Interface] Set WA_TranslucentBackground before accessing winId()

2017-11-23 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8963 To: broulik, #plasma, fvogt, davidedmundson Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai,

KDE CI: Frameworks kirigami kf5-qt5 WindowsMSVCQt5.9 - Build # 115 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20WindowsMSVCQt5.9/115/ Project: Frameworks kirigami kf5-qt5 WindowsMSVCQt5.9 Date of build: Thu, 23 Nov 2017 11:29:22 + Build duration: 25 min and counting JUnit Tests

D8332: Added baloo urls into places model

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22810. renatoo marked an inline comment as done. renatoo added a comment. Renamed test function REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8332?vs=22775=22810 REVISION DETAIL https://phabricator.kde.org/D8332

D8948: Created an auxiliary function 'KFilePlacesModel::movePlace'

2017-11-23 Thread Renato Oliveira Filho
renatoo updated this revision to Diff 22809. renatoo marked 7 inline comments as done. renatoo added a comment. Fixed typos Fixed code style REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8948?vs=22783=22809 REVISION DETAIL

KDE CI: Frameworks kirigami kf5-qt5 SUSEQt5.10 - Build # 3 - Fixed!

2017-11-23 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20kirigami%20kf5-qt5%20SUSEQt5.10/3/ Project: Frameworks kirigami kf5-qt5 SUSEQt5.10 Date of build: Thu, 23 Nov 2017 11:29:22 + Build duration: 8 min 9 sec and counting JUnit Tests

D8864: RFC: [MouseEventListener] Allow accepting mouse event

2017-11-23 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R296:e2795e947233: [MouseEventListener] Allow accepting mouse event (authored by broulik). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D8864?vs=22518=22803#toc REPOSITORY R296 KDeclarative

D8958: Fix unintentional breadcrumb menu item activation

2017-11-23 Thread Kai Uwe Broulik
broulik added a comment. I think you should store the mouse position and then check whether the `manhattanLength()` between new and old is `>= QApplication::startDragDisance()` to be consistent with regular click and drag drop handling – not everyone is capable of holding a mouse perfectly

D8098: Strip down and re-write the baloo tags KIO slave

2017-11-23 Thread Anthony Fieroni
anthonyfieroni added inline comments. INLINE COMMENTS > kio_tags.cpp:363 > + > +if (result.decodedUrl.contains(QStringLiteral("?//")) || > chopLastSection) { > +result.tag = result.tag.section(QDir::separator(), 0, -2); Better reverse conditions if (chopLastSection