[Differential] [Updated, 70 lines] D4628: Add ability to disable saving in `KTextEditor::Document`

2017-02-15 Thread Russell Greene
russellg removed R39 KTextEditor as the repository for this revision. russellg updated this revision to Diff 11390. russellg added a comment. Updated the test to be slightly better. Use `QVERIFY(...)` instead of `QCOMPARE(..., true)` in one case, and make sure that `saveEnabled()` is false

[Differential] [Request, 69 lines] D4628: Add ability to disable saving in `KTextEditor::Document`

2017-02-15 Thread Russell Greene
russellg created this revision. russellg set the repository for this revision to R39 KTextEditor. Restricted Application added subscribers: Frameworks, kwrite-devel. Restricted Application added a project: Frameworks. REVISION SUMMARY I'm currently writing an application where the user is

[Differential] [Commented On] D4626: Renamed icons for encrypted and decripted folders

2017-02-15 Thread Ivan Čukić
ivan added a comment. I will use them, but don't do that just yet - I want to propose some changes to them. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4626 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: ivan, alex-l,

[Differential] [Closed] D4626: Renamed icons for encrypted and decripted folders

2017-02-15 Thread Ivan Čukić
This revision was automatically updated to reflect the committed changes. Closed by commit R266:b79085c027c0: Renamed icons for encrypted and decripted folders (authored by ivan). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D4626?vs=11385=11386#toc REPOSITORY R266 Breeze Icons

[Differential] [Accepted] D4626: Renamed icons for encrypted and decripted folders

2017-02-15 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a comment. This revision is now accepted and ready to land. good point. If you want to use this icons I'll make them for the other places size. REPOSITORY R266 Breeze Icons BRANCH ivan/rename-icons REVISION DETAIL

[Differential] [Request, 958 lines] D4626: Renamed icons for encrypted and decripted folders

2017-02-15 Thread Ivan Čukić
ivan created this revision. ivan added reviewers: alex-l, andreaska. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY Since the icons are in places, not in actions, changed the names to imply state, not action.

[Differential] [Request, 799 lines] D4625: Icons for Plasma Vault

2017-02-15 Thread Ivan Čukić
ivan created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY Merge branch 'master' into ivan/plasmavault-icons REPOSITORY R266 Breeze Icons BRANCH ivan/plasmavault-icons REVISION DETAIL

[Differential] [Closed] D4624: Added arc config file

2017-02-15 Thread Ivan Čukić
This revision was automatically updated to reflect the committed changes. Closed by commit R266:bc60e3c2f85f: Added arc config file (authored by ivan). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4624?vs=11382=11383 REVISION DETAIL

[Differential] [Request, 3 lines] D4624: Added arc config file

2017-02-15 Thread Ivan Čukić
ivan created this revision. ivan added a reviewer: Frameworks. Restricted Application added a project: Frameworks. REVISION SUMMARY Added arc configuration file REPOSITORY R266 Breeze Icons BRANCH master REVISION DETAIL https://phabricator.kde.org/D4624 AFFECTED FILES .arcconfig

[Differential] [Commented On] D4584: KDirWatch: replace QList by std::vector to save on new/delete.

2017-02-15 Thread Sergio Martins
smartins added inline comments. INLINE COMMENTS > dfaure wrote in kdirwatch.cpp:1379 > Detaches? a std::vector? doh! REPOSITORY R244 KCoreAddons BRANCH master REVISION DETAIL https://phabricator.kde.org/D4584 EMAIL PREFERENCES

[Differential] [Changed Subscribers] D4620: allow to add application actions on an open menu

2017-02-15 Thread David Faure
dfaure added inline comments. INLINE COMMENTS > hein wrote in dropjob.cpp:324 > Should this be && instead of ||? "Add actions if either of those is not > empty" seems weird, what's the reasoning? It's a copy of the if() on line 311. But there it makes sense. Here not so much ;) What if there

[Differential] [Commented On] D4584: KDirWatch: replace QList by std::vector to save on new/delete.

2017-02-15 Thread David Faure
dfaure added inline comments. INLINE COMMENTS > smartins wrote in kdirwatch.cpp:1379 > can m_clients be shared ? If yes, htis detaches Detaches? a std::vector? > mwolff wrote in kdirwatch_p.h:84 > right, but that's what you want to state here, no? anyhow, I'm also OK with > leaving it as-is

[Differential] [Changed Subscribers] D4584: KDirWatch: replace QList by std::vector to save on new/delete.

2017-02-15 Thread Sergio Martins
smartins added inline comments. INLINE COMMENTS > kdirwatch.cpp:1379 > > -Q_FOREACH (Client *c, e->m_clients) { > -if (c->instance == nullptr || c->count == 0) { > +for (Client : e->m_clients) { > +if (c.instance == nullptr || c.count == 0) { can m_clients be shared ?

[Differential] [Accepted] D4584: KDirWatch: replace QList by std::vector to save on new/delete.

2017-02-15 Thread Milian Wolff
mwolff accepted this revision. mwolff added a reviewer: mwolff. mwolff added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > dfaure wrote in kdirwatch_p.h:84 > Given that the same thing happens by default, what would be the reason to > make it explicit?

[Differential] [Commented On] D4414: don't regenerate frames when setting every property

2017-02-15 Thread David Rosca
drosca added a comment. I now get tons of binding loop errors from FrameSvgItem, it also breaks delegates in networkmanager and bluetooth applets. On the screenshot you can see that the last 3 delegates (HUAWEI, UPC and Internet) are slightly moved to the left and hovering over them will

[Differential] [Commented On] D4620: allow to add application actions on an open menu

2017-02-15 Thread Eike Hein
hein added a comment. Much nicer than the other approach, just one question. INLINE COMMENTS > dropjob.cpp:324 > +for (QMenu *menu : d->m_menus) { > +if (!d->m_appActions.isEmpty() || !d->m_pluginActions.isEmpty()) { > +menu->addSeparator(); Should this be && instead

[Differential] [Abandoned] D4575: add a popupmenuabouttoshow version that exposes the menu

2017-02-15 Thread Marco Martin
mart abandoned this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D4575 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mart, #plasma Cc: hein, plasma-devel, #frameworks, lesliezhai, ali-mohamed, jensreuterberg, abetts,

[Differential] [Updated, 11 lines] D4620: allow to add application actions on an open menu

2017-02-15 Thread Marco Martin
mart updated this revision to Diff 11371. mart added a comment. - remove useless signal REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4620?vs=11369=11371 BRANCH phab/setActionsLater REVISION DETAIL https://phabricator.kde.org/D4620 AFFECTED FILES

[Differential] [Request, 13 lines] D4620: allow to add application actions on an open menu

2017-02-15 Thread Marco Martin
mart created this revision. mart added a reviewer: Plasma. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY even if the menu has already been created, cause setapplicationActions to add actions in

[Differential] [Updated, 283 lines] D4576: [WIP]: integrate drop menu and filecopy drop menu

2017-02-15 Thread Marco Martin
mart updated this revision to Diff 11368. mart added a comment. - use dropjobs instead of directly accessing its qmenu REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4576?vs=11307=11368 BRANCH arcpatch-D4576 REVISION DETAIL

[Differential] [Request, 14 lines] D4619: Printing: Respect footer font, fix footer vertical position, make header/footer separator line visually lighter

2017-02-15 Thread Jan Ziak
atomsymbol created this revision. atomsymbol added a reviewer: kfunk. atomsymbol set the repository for this revision to R39 KTextEditor. Restricted Application added subscribers: Frameworks, kwrite-devel. Restricted Application added a project: Frameworks. REVISION SUMMARY Fixes bug 376060

[Differential] [Updated, 259 lines] D4537: EditorConfig support

2017-02-15 Thread Grzegorz Szymaszek
gszymaszek updated this revision to Diff 11366. gszymaszek added a comment. key and value are const. REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4537?vs=11365=11366 REVISION DETAIL https://phabricator.kde.org/D4537 AFFECTED FILES CMakeLists.txt

[Differential] [Updated, 259 lines] D4537: EditorConfig support

2017-02-15 Thread Grzegorz Szymaszek
gszymaszek updated this revision to Diff 11365. gszymaszek marked 2 inline comments as done. gszymaszek added a comment. Pulled check-variable-functions out of EditorConfig class, moved some variable definitions into main loop of parser. REPOSITORY R39 KTextEditor CHANGES SINCE LAST

[Differential] [Commented On] D4537: EditorConfig support

2017-02-15 Thread Dominik Haumann
dhaumann added a comment. Patch looks already pretty good, I think we're soon there. INLINE COMMENTS > gszymaszek wrote in editorconfig.cpp:23 > Is it OK to initialize `m_handle` in the constructor? If so, is `m_handle(0)` > necessary? Yes, this is good now. > editorconfig.cpp:34-56 >

[Differential] [Commented On] D4248: Add Boost_INCLUDE_DIR to kactivities-stats autotests.

2017-02-15 Thread Adriaan de Groot
adridg added a comment. I'm going to claim feedback timeout, plus all it does is add some -I flags when compiling the tests. REPOSITORY R159 KActivities Statistics REVISION DETAIL https://phabricator.kde.org/D4248 EMAIL PREFERENCES

[Differential] [Closed] D4248: Add Boost_INCLUDE_DIR to kactivities-stats autotests.

2017-02-15 Thread Adriaan de Groot
This revision was automatically updated to reflect the committed changes. Closed by commit R159:d2cbef5d94f3: Add Boost_INCLUDE_DIR to kactivities-stats autotests. (authored by adridg). REPOSITORY R159 KActivities Statistics CHANGES SINCE LAST UPDATE

[Differential] [Closed] D3826: Detect inotify.

2017-02-15 Thread Adriaan de Groot
This revision was automatically updated to reflect the committed changes. adridg marked 2 inline comments as done. Closed by commit R240:a02c4d0152a7: Detect inotify. (authored by adridg). REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE