Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-03 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128680/#review98855 --- Andreas, any idea which icon the action could use? - Christo

Re: Review Request 122551: New feature: Open all recent files

2016-09-03 Thread Thomas Murach
> On Feb. 13, 2015, 12:06 vorm., Kevin Funk wrote: > > You're aware that Kate (and any other decent editor) has session management > > which probably solves your issue? It saves the session when you close it, > > and re-opens all the files that had been opened. > > Thomas Murach wrote: > S

[Differential] [Commented On] D2545: Cleanup KSharedUiServerProxy before qApp exists

2016-09-03 Thread dfaure (David Faure)
dfaure added a comment. So what is the actual deadlock you're experiencing? I showed this to Thiago and he said he needs more info on what is actually happening, i.e. where exactly it deadlocks in QtDBus. BRANCH master REVISION DETAIL https://phabricator.kde.org/D2545 EMAIL PREFERE

Re: Review Request 128818: Revert "Remove unused KF5 dependencies"

2016-09-03 Thread Alexander Potashev
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128818/ --- (Updated Сен. 3, 2016, 12:13 п.п.) Status -- This change has been ma

Re: Review Request 128818: Revert "Remove unused KF5 dependencies"

2016-09-03 Thread Alexander Potashev
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128818/#review98839 --- Ship it! Doing a self-ship-it because it's trivial. - Ale

Re: Review Request 128582: Remove unused KF5 dependencies

2016-09-03 Thread Alexander Potashev
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128582/ --- (Updated Сен. 3, 2016, 12:11 п.п.) Status -- This change has been ma

Re: Review Request 128582: Remove unused KF5 dependencies

2016-09-03 Thread Alexander Potashev
> On Авг. 25, 2016, 10:02 п.п., Andreas Sturmlechner wrote: > > lddtree /usr/lib64/libKF5KrossUi.so is listing all these three dependencies > > still in use. > > Alexander Potashev wrote: > Oops... You're right: kross/src/ui/actioncollectionview.cpp uses > 1. KComboBox from KCompletion,

Review Request 128818: Revert "Remove unused KF5 dependencies"

2016-09-03 Thread Alexander Potashev
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128818/ --- Review request for KDE Frameworks, Aleix Pol Gonzalez and Andreas Sturmlech

Re: Review Request 128694: Fix all edit-clear-location-ltr icons

2016-09-03 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128694/ --- (Updated Sept. 3, 2016, 8:48 a.m.) Status -- This change has been ma

Re: Review Request 128694: Fix all edit-clear-location-ltr icons

2016-09-03 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128694/#review98835 --- I submit the right icons thanks for reporting (and fix) the

Re: Review Request 128763: WindowThumbnail: Do GL calls in the correct thread

2016-09-03 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128763/ --- (Updated Sept. 3, 2016, 4:33 a.m.) Status -- This change has been ma

Re: Review Request 128694: Fix all edit-clear-location-ltr icons

2016-09-03 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128694/#review98836 --- the patch isn't needed any more - Andreas Kainz On Aug. 15

Re: Review Request 128763: WindowThumbnail: Do GL calls in the correct thread

2016-09-03 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128763/#review98834 --- Ship it! Ship It! - Martin Gräßlin On Sept. 2, 2016, 6: