D5850: Fix duplicate Discover entry on the Application Menu

2017-05-14 Thread Antonio Rojas
This revision was automatically updated to reflect the committed changes. Closed by commit R134:13d96d15a177: Fix duplicate Discover entry on the Application Menu (authored by arojas). REPOSITORY R134 Discover Software Store CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5850?vs=1450

D5850: Fix duplicate Discover entry on the Application Menu

2017-05-14 Thread Aleix Pol Gonzalez
apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D5850 To: arojas, #discover_software_store, apol Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreut

Re: Build issue of KWin in TumbleWeed

2017-05-14 Thread Michail Vourlakos
On 14/05/2017 09:23 μμ, Luca Beltrame wrote: In data domenica 14 maggio 2017 20:06:37 CEST, Michail Vourlakos ha scritto: Even though the xkbcommon (0.7.1) packages are installed (and devel package also) from main repo You will need libxkbcommon-x11-devel and libxkbcommon-devel installed. A

[kio-extras] [Bug 376344] cant write to smb shares which has write access

2017-05-14 Thread Sergey
https://bugs.kde.org/show_bug.cgi?id=376344 Sergey changed: What|Removed |Added CC||rion...@gmail.com --- Comment #22 from Sergey --- Jus

Re: Build issue of KWin in TumbleWeed

2017-05-14 Thread Luca Beltrame
In data domenica 14 maggio 2017 20:06:37 CEST, Michail Vourlakos ha scritto: > Even though the xkbcommon (0.7.1) packages are installed (and devel > package also) from main repo You will need libxkbcommon-x11-devel and libxkbcommon-devel installed. Are they both present? You can also use "zypp

Build issue of KWin in TumbleWeed

2017-05-14 Thread Michail Vourlakos
Hello, I am trying to build plasma-desktop with kdesrc-build in Tumbleweed. In general, most of the dependencies needed was found but I have the following issue with KWin. Even though the xkbcommon (0.7.1) packages are installed (and devel package also) from main repo kwin stops in the cma

Plasma Vault

2017-05-14 Thread Jonathan Riddell
How do I get Plasma Vault to do anything? I've compiled it and the Plasmoid has a button saying Create a new vault and nothing else. Clicking it does nothing. I was told I had to install encfs but installing that doesn't help Jonathan

D5851: Fix the PID updated in PlasmaWindowedModel

2017-05-14 Thread Martin Flöser
graesslin added a comment. In https://phabricator.kde.org/D5851#109478, @davidedmundson wrote: > It makes absolutely no sense to have a pidChanged signal and then say clients shouldn't be using it. > > Either window has no reason to have a pidChanged signal or we should connect to it

D5853: Remove gtk+-3 build dep, install versioned gtk.css dirs unconditionally

2017-05-14 Thread Andreas Sturmlechner
asturmlechner created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY GTK will search /usr/share/themes//gtk- and then earlier versions all the way back to 3.0. See also: https://developer.gnome.or

Re: Review Request 129851: JpegCreator: Use Qt's support for fast downscaling and auto-rotation

2017-05-14 Thread Alex Bikadorov
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129851/#review103212 --- FYI: https://bugs.kde.org/show_bug.cgi?id=379817 - Alex Bik

[kio-extras] [Bug 379817] New: Behaviour change: JPEG image preview scaled to widget size

2017-05-14 Thread Alex Bikadorov
https://bugs.kde.org/show_bug.cgi?id=379817 Bug ID: 379817 Summary: Behaviour change: JPEG image preview scaled to widget size Product: kio-extras Version: unspecified Platform: Archlinux Packages OS: Linux

D5851: Fix the PID updated in PlasmaWindowedModel

2017-05-14 Thread David Edmundson
davidedmundson added a comment. It makes absolutely no sense to have a pidChanged signal and then say clients shouldn't be using it. Either window has no reason to have a pidChanged signal or we should connect to it. It has to be one or the other. REPOSITORY R127 KWayland REVISION DE

D5851: Fix the PID updated in PlasmaWindowedModel

2017-05-14 Thread Martin Flöser
graesslin added inline comments. INLINE COMMENTS > plasmawindowmodel.cpp:84 > > +QObject::connect(window, &PlasmaWindow::pidChanged, q, > +[window, this] { this->dataChanged(window, PlasmaWindowModel::Pid); } In the original report I critized that we connect to a pid changed signal

D5851: Fix the PID updated in PlasmaWindowedModel

2017-05-14 Thread David Edmundson
davidedmundson created this revision. Restricted Application added projects: Plasma on Wayland, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY Fixes https://phabricator.kde.org/D5747 TEST PLAN Tests now pass REPOSITORY R127 KWayland BRANCH

D5850: Fix duplicate Discover entry on the Application Menu

2017-05-14 Thread Antonio Rojas
arojas created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. TEST PLAN Only one Discover entry is shown on the menu REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D5850 AFFECTED

D5747: add pid to plasma window management protocol

2017-05-14 Thread Eike Hein
hein added a comment. FWIW I did run make test on an earlier revision of the patch during the sprint while doing work on top of it; it must have regressed after. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D5747 To: sebas, #plasma, hein, graesslin Cc: apol, davi