D8756: Forbid more syscalls which could modify the filesystem

2017-11-10 Thread David Kahles
This revision was automatically updated to reflect the committed changes.
Closed by commit R133:7720e8486337: Forbid more syscalls which could modify the 
filesystem (authored by davidk).

REPOSITORY
  R133 KScreenLocker

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8756?vs=22174=22176

REVISION DETAIL
  https://phabricator.kde.org/D8756

AFFECTED FILES
  greeter/autotests/seccomp_test.cpp
  greeter/seccomp_filter.cpp

To: davidk, graesslin
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8754: Baloo KCM text improvements

2017-11-10 Thread Burkhard Lück
lueck added inline comments.

INLINE COMMENTS

> index.docbook:18-19
>  
>  2017-07-28
>  Plasma 5.10
>  

Please bump date + releaseinfo

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8754

To: ngraham, #vdg, #plasma, abetts
Cc: lueck, apol, plasma-devel, abetts, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, sebas, mart


D8756: Forbid more syscalls which could modify the filesystem

2017-11-10 Thread Martin Flöser
graesslin accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R133 KScreenLocker

BRANCH
  seccomp

REVISION DETAIL
  https://phabricator.kde.org/D8756

To: davidk, graesslin
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8748: [libinput] Find screen for touch screen devices and calibrate accordingly

2017-11-10 Thread Martin Flöser
graesslin updated this revision to Diff 22175.
graesslin added a comment.
Restricted Application edited projects, added Plasma; removed KWin.


  Use screens()->geometry() instead of size

REPOSITORY
  R108 KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8748?vs=22152=22175

BRANCH
  libinput-calibrate-touchscreen2

REVISION DETAIL
  https://phabricator.kde.org/D8748

AFFECTED FILES
  autotests/libinput/device_test.cpp
  autotests/libinput/mock_libinput.cpp
  autotests/libinput/mock_libinput.h
  input.cpp
  libinput/connection.cpp
  libinput/connection.h
  libinput/device.cpp
  libinput/device.h
  plugins/platforms/drm/drm_output.h
  plugins/platforms/drm/screens_drm.cpp
  plugins/platforms/drm/screens_drm.h
  screens.cpp
  screens.h

To: graesslin, #kwin, #plasma
Cc: davidedmundson, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D8756: Forbid more syscalls which could modify the filesystem

2017-11-10 Thread David Kahles
davidk added a reviewer: graesslin.

REPOSITORY
  R133 KScreenLocker

REVISION DETAIL
  https://phabricator.kde.org/D8756

To: davidk, graesslin
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8756: Forbid more syscalls which could modify the filesystem

2017-11-10 Thread David Kahles
davidk created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Forbid more syscalls. An malicious theme could create directories with the
  password as name, or encode the password in chmod bits. Also, prevent
  deleting anything, so a theme can't delete the users files.

TEST PLAN
  - Autotests run fine
  - Started sceenlocker, unlocked, created a new session. Got no seccomp 
violations in dmesg and everything worked fine.
  - Didn't test it with the nvidia driver

REPOSITORY
  R133 KScreenLocker

BRANCH
  seccomp

REVISION DETAIL
  https://phabricator.kde.org/D8756

AFFECTED FILES
  greeter/autotests/seccomp_test.cpp
  greeter/seccomp_filter.cpp

To: davidk
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8754: Baloo KCM text improvements

2017-11-10 Thread Nathaniel Graham
ngraham added a comment.


  I kind of think all of those are worse than what we already have, to be 
honest. "Do not search in these locations" is a pretty clear and user-friendly 
text string IMHO.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8754

To: ngraham, #vdg, #plasma, abetts
Cc: apol, plasma-devel, abetts, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, sebas, mart


D8754: Baloo KCM text improvements

2017-11-10 Thread Andres Betts
abetts added a comment.


  I think that people these days understand that indexing your files is a 
component of searching. If the word indexing is not liked, maybe it could be
  
  Capture file content data
  Read file content data
  Record file content data
  
  Etc.
  
  I would also suggest that you start each checkbox with a verb.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8754

To: ngraham, #vdg, #plasma, abetts
Cc: apol, plasma-devel, abetts, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, sebas, mart


D8755: User Manager KCM text changes and UI modernization

2017-11-10 Thread Nathaniel Graham
This revision was automatically updated to reflect the committed changes.
Closed by commit R128:5c072eab10bd: User Manager KCM text changes and UI 
modernization (authored by ngraham).

REPOSITORY
  R128 User Manager

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8755?vs=22167=22168

REVISION DETAIL
  https://phabricator.kde.org/D8755

AFFECTED FILES
  src/account.ui

To: ngraham, abetts, #vdg, #plasma, apol
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8755: User Manager KCM text changes and UI modernization

2017-11-10 Thread Aleix Pol Gonzalez
apol accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R128 User Manager

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D8755

To: ngraham, abetts, #vdg, #plasma, apol
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8754: Baloo KCM text improvements

2017-11-10 Thread Aleix Pol Gonzalez
apol added a comment.


  Why does the user need to know about the indexing? isn't it an implementation 
detail?

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8754

To: ngraham, #vdg, #plasma, abetts
Cc: apol, plasma-devel, abetts, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, sebas, mart


D8524: Add a signal notifying a change in the names of running activities.

2017-11-10 Thread Eike Hein
hein added a comment.


  Ping

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D8524

To: hein, #plasma
Cc: anthonyfieroni, ivan, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D8755: User Manager KCM text changes and UI modernization

2017-11-10 Thread Nathaniel Graham
ngraham edited the test plan for this revision.

REPOSITORY
  R128 User Manager

REVISION DETAIL
  https://phabricator.kde.org/D8755

To: ngraham, abetts, #vdg, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8755: User Manager KCM text changes and UI modernization

2017-11-10 Thread Nathaniel Graham
ngraham created this revision.
ngraham added reviewers: abetts, VDG, Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  BUG: 386635
  
  Improve some strings and use a more modern checkboxes-on-the-left-side design

TEST PLAN
  Tested in KDE Neon:

REPOSITORY
  R128 User Manager

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D8755

AFFECTED FILES
  src/account.ui

To: ngraham, abetts, #vdg, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8754: Baloo KCM text improvements

2017-11-10 Thread Nathaniel Graham
ngraham created this revision.
ngraham added reviewers: VDG, Plasma, abetts.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  BUG: 386634
  
  Implement @abetts' proposed text changes for the Baloo KCM

TEST PLAN
  Tested in KDE Neon:

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D8754

AFFECTED FILES
  doc/kcontrol/baloo/index.docbook
  kcms/baloo/configwidget.ui

To: ngraham, #vdg, #plasma, abetts
Cc: plasma-devel, abetts, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, sebas, apol, mart


D8692: QML mouse cursor KCM and components

2017-11-10 Thread Nathaniel Graham
ngraham edited the summary of this revision.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8692

To: mart, #plasma
Cc: januz, ngraham, subdiff, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D8748: [libinput] Find screen for touch screen devices and calibrate accordingly

2017-11-10 Thread Martin Flöser
graesslin marked an inline comment as done.
graesslin added inline comments.

INLINE COMMENTS

> graesslin wrote in connection.cpp:395
> It's failing to compile in tests/ directory. There I don't have the 
> mockscreens, so I didn't think of it. Will try with the mockscreens. Thanks 
> for the suggestion.

Didn't get it to compile with the mockscreens.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D8748

To: graesslin, #kwin, #plasma
Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart


D8748: [libinput] Find screen for touch screen devices and calibrate accordingly

2017-11-10 Thread Martin Flöser
graesslin added inline comments.

INLINE COMMENTS

> davidedmundson wrote in connection.cpp:395
> not that it's a problem, but why? 
> You have screens mocked already.

It's failing to compile in tests/ directory. There I don't have the 
mockscreens, so I didn't think of it. Will try with the mockscreens. Thanks for 
the suggestion.

> davidedmundson wrote in connection.cpp:488
> you're not updating in deviceAdded(), so hypothetically if one adds a touch 
> screen at runtime, and plugs in the HDMI cable before the USB cable it won't 
> have the right screen.
> 
> You could put this method (and the screens changed connect) in Device as 
> you're not doing anything with Connection in this method.

> You could put this method (and the screens changed connect) in Device as 
> you're not doing anything with Connection in this method.

I didn't want to add the screens dependency to Device. I need it in Connection 
for the touch point translation, so I added it there. Device itself doesn't 
have any dependencies of KWin yet, Connection is kind of the "firewall" between 
them.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D8748

To: graesslin, #kwin, #plasma
Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart


D8752: [platforms/drm] At safety checks for the properties

2017-11-10 Thread Martin Flöser
graesslin created this revision.
graesslin added reviewers: KWin, Plasma, fvogt, subdiff.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.

REVISION SUMMARY
  The AMS code accesses elements in a vector which might not be valid. This
  change refactors the code to be more robust, especially the DrmPlane,
  which started to crash after adding transformation support.
  
  BUG: 386490

REPOSITORY
  R108 KWin

BRANCH
  drm-property-safety

REVISION DETAIL
  https://phabricator.kde.org/D8752

AFFECTED FILES
  plugins/platforms/drm/drm_object.cpp
  plugins/platforms/drm/drm_object.h
  plugins/platforms/drm/drm_object_plane.cpp

To: graesslin, #kwin, #plasma, fvogt, subdiff
Cc: plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
hardening, jensreuterberg, abetts, sebas, apol, mart


D8748: [libinput] Find screen for touch screen devices and calibrate accordingly

2017-11-10 Thread David Edmundson
davidedmundson added a comment.


  One minor comment (the third one), otherwise all good.

INLINE COMMENTS

> connection.cpp:395
>  case LIBINPUT_EVENT_TOUCH_DOWN: {
> +#ifndef KWIN_BUILD_TESTING
>  TouchEvent *te = static_cast(event.data());

not that it's a problem, but why? 
You have screens mocked already.

> connection.cpp:397
>  TouchEvent *te = static_cast(event.data());
> -emit touchDown(te->id(), te->absolutePos(m_size), 
> te->time(), te->device());
> +const auto  = screens()->size(te->device()->screenId());
> +emit touchDown(te->id(), 
> screens()->geometry(te->device()->screenId()).topLeft() + 
> te->absolutePos(size), te->time(), te->device());

if you fetch the geometry here you'll get both the size and topLeft in an 
easier to read way.

> connection.cpp:488
>  
> +void Connection::updateScreens()
> +{

you're not updating in deviceAdded(), so hypothetically if one adds a touch 
screen at runtime, and plugs in the HDMI cable before the USB cable it won't 
have the right screen.

You could put this method (and the screens changed connect) in Device as you're 
not doing anything with Connection in this method.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D8748

To: graesslin, #kwin, #plasma
Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart


D8731: [x11] Add support for automatic screen rotation through XRandR

2017-11-10 Thread Martin Flöser
graesslin added a comment.


  > Yet we're putting this sensor stuff in kwin instead, then having to come up 
with extra interfaces to fix the problems we're creating.
  
  The main reason is that I wanted to support the phone case (hwcomposer) where 
we don't have the OutputConfigurationInterface hooked up and where it doesn't 
make sense to hook it up. The second reason is that KWin needs to support the 
full story including input adjustments. KScreen cannot do that, KWin must do 
it. Given that I would consider it weird to do everything related to the 
rotation just except detecting the orientation change. Also I think that KWin 
should be able to handle the most basic screen functionality (and yes I count 
rotation to that nowadays) on Wayland without the help of other processes. This 
makes it easier to deploy KWin outside of Plasma. E.g. for things we discussed 
like having KWin running in SDDM as the greeter compositor it means that KWin 
can provide automatic rotation without having to start half of our session in 
the login process.
  
  Of course that's all debatable and you might have noticed that the 
OrientationSensor class I added to KWin does not include anything else from 
KWin. It's a standalone component which can be split out and moved to KScreen.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D8731

To: graesslin, #kwin, #plasma, sebas, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart


D8731: [x11] Add support for automatic screen rotation through XRandR

2017-11-10 Thread Martin Flöser
graesslin updated this revision to Diff 22153.
graesslin added a comment.
Restricted Application edited projects, added KWin; removed Plasma.


  Fixup arc failure

REPOSITORY
  R108 KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8731?vs=22151=22153

BRANCH
  xrandr-automatic-tranformation

REVISION DETAIL
  https://phabricator.kde.org/D8731

AFFECTED FILES
  plugins/platforms/x11/standalone/screens_xrandr.cpp
  plugins/platforms/x11/standalone/screens_xrandr.h

To: graesslin, #kwin, #plasma, sebas, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart


D8748: [libinput] Find screen for touch screen devices and calibrate accordingly

2017-11-10 Thread Martin Flöser
graesslin created this revision.
graesslin added reviewers: KWin, Plasma.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.

REVISION SUMMARY
  This change finds the screen for a touch screen device based on:
  
  - number available screens
  - output name defined on the touch screen device
  - internal screen
  - and physical size
  
  The id of the screen is stored in the Device allowing to adjust the
  touch points accordingly. This means instead of transferring to the
  combined display size the touch points are transferred into the output
  space and the position of the output is added. Thus in a multi screen
  system the touch points are properly mapped to the output.
  
  Furthermore the screen orientation is passed to the Device and a
  calibration matrix is set accordingly. Thus a transformed screen has the
  touch screen transformed accordingly.
  
  Please note that this only affects libinput on Wayland and not on X11!
  The x11 standalone platform needs to gain similar code.

REPOSITORY
  R108 KWin

BRANCH
  libinput-calibrate-touchscreen2

REVISION DETAIL
  https://phabricator.kde.org/D8748

AFFECTED FILES
  autotests/libinput/device_test.cpp
  autotests/libinput/mock_libinput.cpp
  autotests/libinput/mock_libinput.h
  input.cpp
  libinput/connection.cpp
  libinput/connection.h
  libinput/device.cpp
  libinput/device.h
  plugins/platforms/drm/drm_output.h
  plugins/platforms/drm/screens_drm.cpp
  plugins/platforms/drm/screens_drm.h
  screens.cpp
  screens.h

To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
hardening, jensreuterberg, abetts, sebas, apol, mart


D8731: [x11] Add support for automatic screen rotation through XRandR

2017-11-10 Thread Martin Flöser
graesslin updated this revision to Diff 22151.
graesslin added a comment.
Restricted Application edited projects, added Plasma; removed KWin.


  - [libinput] Find screen for touch screen devices and calibrate accordingly

REPOSITORY
  R108 KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8731?vs=22105=22151

BRANCH
  libinput-calibrate-touchscreen

REVISION DETAIL
  https://phabricator.kde.org/D8731

AFFECTED FILES
  autotests/libinput/device_test.cpp
  autotests/libinput/mock_libinput.cpp
  autotests/libinput/mock_libinput.h
  input.cpp
  libinput/connection.cpp
  libinput/connection.h
  libinput/device.cpp
  libinput/device.h
  plugins/platforms/drm/drm_output.h
  plugins/platforms/drm/screens_drm.cpp
  plugins/platforms/drm/screens_drm.h
  plugins/platforms/x11/standalone/screens_xrandr.cpp
  plugins/platforms/x11/standalone/screens_xrandr.h
  screens.cpp
  screens.h

To: graesslin, #kwin, #plasma, sebas, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D8731: [x11] Add support for automatic screen rotation through XRandR

2017-11-10 Thread David Edmundson
davidedmundson added a comment.


  > I hope this addresses your concerns and can us allow to unblock this change.
  
  It could work, so I won't block it.
  
  But I still don't understand the design rationale.
  
  We have a component that is in charge of handling all screen management. It 
has all the backends already, it does the config tracking, it's got the 
communication all hooked up already with a simple single path in DRM.
  Yet we're putting this sensor stuff in kwin instead, then having to come up 
with extra interfaces to fix the problems we're creating.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D8731

To: graesslin, #kwin, #plasma, sebas, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart


D8744: Use Qt.callLater instead of creating a Timer

2017-11-10 Thread Kai Uwe Broulik
broulik created this revision.
broulik added reviewers: Plasma, hein.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

TEST PLAN
  This is new in Qt 5.8.
  The kicker dialogs are properly destroyed.
  I didn't actually see the one in FolderView being called.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8744

AFFECTED FILES
  applets/kicker/package/contents/ui/ItemListDialog.qml
  containments/desktop/package/contents/ui/FolderViewDialog.qml

To: broulik, #plasma, hein
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8441: Wallpaper: hide color or blur filling options for full filling mode

2017-11-10 Thread Nathaniel Graham
ngraham added a comment.


  A neat idea, but yeah, I think it probably wouldn't even occur to most people 
to try to use a transparent image! If you approve, would you mind formally 
accepting it? I'm willing to triage and handle any bugs that come in saying 
"Hey, you broke my transparent wallpaper!"

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D8441

To: guoyunhe, ngraham
Cc: jensreuterberg, broulik, ngraham, davidedmundson, plasma-devel, ZrenBot, 
progwolff, lesliezhai, ali-mohamed, abetts, sebas, apol, mart


D8743: Add dynamic QML visual results

2017-11-10 Thread Aditya Mehra
This revision was automatically updated to reflect the committed changes.
Closed by commit R846:b5e85c3e4b9f: Add dynamic QML visual results (authored by 
Aiix).

REPOSITORY
  R846 Mycroft Plasma integration

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8743?vs=22143=22144

REVISION DETAIL
  https://phabricator.kde.org/D8743

AFFECTED FILES
  plasmoid/contents/ui/LoaderType.qml
  plasmoid/contents/ui/main.qml

To: Aiix, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


D8743: Add dynamic QML visual results

2017-11-10 Thread Aditya Mehra
Aiix created this revision.
Aiix added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Ref https://phabricator.kde.org/T7433 Added LoaderType which displays 
dynamically generated QML template results from mycroft skills inside dynamic 
component

REPOSITORY
  R846 Mycroft Plasma integration

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D8743

AFFECTED FILES
  plasmoid/contents/ui/LoaderType.qml
  plasmoid/contents/ui/main.qml

To: Aiix, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


Re: Out-of-screen painting for QQuickView ?

2017-11-10 Thread David Edmundson
bug ​353983


D8692: QML mouse cursor KCM and components

2017-11-10 Thread Marco Martin
mart added a comment.


  In https://phabricator.kde.org/D8692#165286, @subdiff wrote:
  
  > - Open the kcm with kcmshell5 kcm_cursortheme opens it in a very small view 
with only one column
  
  
  can you try it now with the last commit?

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8692

To: mart, #plasma
Cc: ngraham, subdiff, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D8692: QML mouse cursor KCM and components

2017-11-10 Thread Marco Martin
mart updated this revision to Diff 22142.
mart added a comment.


  - ensure at least 2 columns are visible

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8692?vs=22026=22142

BRANCH
  kcm-redesign/cursorTheme

REVISION DETAIL
  https://phabricator.kde.org/D8692

AFFECTED FILES
  CMakeLists.txt
  kcmcontrols/GridDelegate.qml
  kcmcontrols/GridViewPage.qml
  kcmcontrols/qmldir
  kcms/cursortheme/CMakeLists.txt
  kcms/cursortheme/Messages.sh
  kcms/cursortheme/cursortheme.desktop
  kcms/cursortheme/kcm_cursortheme.desktop
  kcms/cursortheme/kcmcursortheme.cpp
  kcms/cursortheme/kcmcursortheme.h
  kcms/cursortheme/package/contents/ui/Delegate.qml
  kcms/cursortheme/package/contents/ui/main.qml
  kcms/cursortheme/package/metadata.desktop
  kcms/cursortheme/xcursor/cursortheme.cpp
  kcms/cursortheme/xcursor/cursortheme.h
  kcms/cursortheme/xcursor/itemdelegate.cpp
  kcms/cursortheme/xcursor/itemdelegate.h
  kcms/cursortheme/xcursor/previewwidget.cpp
  kcms/cursortheme/xcursor/previewwidget.h
  kcms/cursortheme/xcursor/sortproxymodel.cpp
  kcms/cursortheme/xcursor/sortproxymodel.h
  kcms/cursortheme/xcursor/thememodel.cpp
  kcms/cursortheme/xcursor/thememodel.h
  kcms/cursortheme/xcursor/themepage.cpp
  kcms/cursortheme/xcursor/themepage.h
  kcms/cursortheme/xcursor/themepage.ui
  kcms/cursortheme/xcursor/xcursor.knsrc
  kcms/cursortheme/xcursor/xcursortheme.cpp
  kcms/cursortheme/xcursor/xcursortheme.h

To: mart, #plasma
Cc: ngraham, subdiff, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D8441: Wallpaper: hide color or blur filling options for full filling mode

2017-11-10 Thread Jens Reuterberg
jensreuterberg added a comment.


  Heh I may be the one who used transparent images - the idea being that you 
could, in theory set colour of an entire wallpaper based on icon colour using 
the correct kind of transparent image (essentially making darker parts less 
transparent meaning the lighter the area the brighter the underlying colour, 
creating a monochromatic image). To be quite frank as an idea it demanded 
slightly more work than was feasible, so I support this.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D8441

To: guoyunhe, ngraham
Cc: jensreuterberg, broulik, ngraham, davidedmundson, plasma-devel, ZrenBot, 
progwolff, lesliezhai, ali-mohamed, abetts, sebas, apol, mart


D8692: QML mouse cursor KCM and components

2017-11-10 Thread Marco Martin
mart added a comment.


  In https://phabricator.kde.org/D8692#165286, @subdiff wrote:
  
  > - Open the kcm with kcmshell5 kcm_cursortheme opens it in a very small view 
with only one column
  
  
  is the case of just few pixels too little to do a second column, to fix it it 
would need for thumbnails to be resizable...
  
  perhaps i should make thumbnails flexible when the view is small enough and 
wouldn't let 2 columns
  
  > - There is no white background to the list view
  
  the white background depends from https://phabricator.kde.org/D8526

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8692

To: mart, #plasma
Cc: ngraham, subdiff, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart


D8691: add some metadata

2017-11-10 Thread Marco Martin
This revision was automatically updated to reflect the committed changes.
Closed by commit R296:ae8d97bb5274: add some metadata (authored by mart).

REPOSITORY
  R296 KDeclarative

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8691?vs=22022=22141

REVISION DETAIL
  https://phabricator.kde.org/D8691

AFFECTED FILES
  src/quickaddons/configmodule.cpp
  src/quickaddons/configmodule.h

To: mart, #plasma, davidedmundson
Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart