[Differential] [Commented On] D1816: Switch to Noto Mono as default monospace font

2016-06-10 Thread jriddell (Jonathan Riddell)
jriddell added a comment. Maintainers answered on https://github.com/googlei18n/noto-fonts/issues/711 and they seem to recommend git versions over tar versions REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D1816

Re: Review Request 126689: Change minimum Size of Calendar

2016-06-10 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126689/ --- (Updated Juni 10, 2016, 10:42 nachm.) Status -- This change has

Re: Review Request 126356: change minimum size of clipboard

2016-06-10 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126356/ --- (Updated Juni 10, 2016, 10:40 nachm.) Status -- This change has

Re: Review Request 128108: update weather applet ui

2016-06-10 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128108/ --- (Updated June 10, 2016, 10:37 p.m.) Review request for Plasma and

SDDM Theme update

2016-06-10 Thread kainz.a
Hi David, kde neon dev ship the new breeze sddm theme (thanks jonathan). It look really good, but I have also some remarks: 1. the avatar icon is to small (not the same size than the plymount icon) 2. can we use the ? and generic avatar I prepare for you 3. the search for user avatar is higher

Re: What happened to Dolphin's transfer dialog?

2016-06-10 Thread Mark Gaiser
This is funny. It's related to this, but i didn't want to keep this information from you folks :) Now that i have this issue i'm looking at some old mailing list archives of 2008. At one point there apparently was a proposal for a "JobViewServer" specification [1] that supported file transfer

Re: What happened to Dolphin's transfer dialog?

2016-06-10 Thread Mark Gaiser
On Fri, Jun 10, 2016 at 9:04 PM, Kai Uwe Broulik wrote: > Hi, > > > Could you please reconsider that implementation detail? > > Why? You still have the job progress in the notification area with time > and controls by default. It's just that you need to disable them both

Re: What happened to Dolphin's transfer dialog?

2016-06-10 Thread Kai Uwe Broulik
Hi, > Could you please reconsider that implementation detail? Why? You still have the job progress in the notification area with time and controls by default. It's just that you need to disable them both for the legacy dialog to show up. I was thinking of providing the full UI in task manager

Re: What happened to Dolphin's transfer dialog?

2016-06-10 Thread Thomas Pfeiffer
On 10.06.2016 20:53, Mark Gaiser wrote: On Fri, Jun 10, 2016 at 6:55 PM, Kai Uwe Broulik > wrote: Hi, oh, heh. Yeah I'm using the job dataengine and if there's any user of it the job view server will be started and job

Re: What happened to Dolphin's transfer dialog?

2016-06-10 Thread Mark Gaiser
On Fri, Jun 10, 2016 at 6:55 PM, Kai Uwe Broulik wrote: > Hi, > > oh, heh. Yeah I'm using the job dataengine and if there's any user of it > the job view server will be started and job progress is sent there instead. > > You can disable the "show application status" thing

Re: Review Request 128108: update weather applet ui

2016-06-10 Thread Friedrich W. H. Kossebau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128108/#review96349 --- With the latest diff I get these runtime errors on the

Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 135 - Still Unstable!

2016-06-10 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/135/ Project: PLATFORM=Linux,compiler=gcc Date of build: Fri, 10 Jun 2016 18:14:13 + Build duration: 9 min 39 sec CHANGE SET Revision

Re: New idea for Plasma's logo

2016-06-10 Thread Thomas Pfeiffer
On 10.06.2016 11:44, Alessandro Longo wrote: I like a lot the idea. I suggest to invite people to post their logos to visual-design mailing list, to to let them know that it exists. Sending an email with an attachment should be enough accessible for everyone and we will have a transparent

Re: What happened to Dolphin's transfer dialog?

2016-06-10 Thread Kai Uwe Broulik
Hi, oh, heh. Yeah I'm using the job dataengine and if there's any user of it the job view server will be started and job progress is sent there instead.  You can disable the "show application status" thing in task manager settings. Might need two separate settings (one for unity launchers, one

Re: What happened to Dolphin's transfer dialog?

2016-06-10 Thread Mark Gaiser
Thank you, Alexander! I was also suspecting that change to have caused it, but i don't see that change disabling (or removing) the progress window. Actually, i;m not even sure which part is responsible for putting the progress window in screen. It could be dolphin specific, but it could also be

[Differential] [Request, 4 lines] D1820: Properly call Toplevel::setSurface for wl_surface_id X11 client message event

2016-06-10 Thread Martin Gräßlin
graesslin created this revision. graesslin added reviewers: KWin, Plasma on Wayland. Restricted Application added subscribers: kwin, plasma-devel. Restricted Application added projects: Plasma on Wayland, KWin. REVISION SUMMARY For an X11 window through Xwayland KWin gets a client message with

[Differential] [Commented On] D1816: Switch to Noto Mono as default monospace font

2016-06-10 Thread jriddell (Jonathan Riddell)
jriddell added a comment. Release management of Noto is curious, depending on where you get your release from it is released. I've filed an issue to ask why it's not in the download tar https://github.com/googlei18n/noto-fonts/issues/711 Our current default is Oxygen Mono which is

[Differential] [Commented On] D1816: Switch to Noto Mono as default monospace font

2016-06-10 Thread davidedmundson (David Edmundson)
davidedmundson added a comment. If it's not released, we can't use it. I'm sure you've personally told me that when I've needed Kubuntu to do something in the past. REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETAIL

[Differential] [Request, 11 lines] D1816: Switch to Noto Mono as default monospace font

2016-06-10 Thread jriddell (Jonathan Riddell)
jriddell created this revision. jriddell added reviewers: Plasma, Plasma: Design. jriddell set the repository for this revision to rPLASMAINTEGRATION Integration for Qt applications in Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.

Re: Review Request 127781: Remove clearly broken code

2016-06-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127781/ --- (Updated June 10, 2016, 1:47 p.m.) Status -- This change has been

[Differential] [Commented On] D1814: Add TaskManager requestActivities

2016-06-10 Thread mart (Marco Martin)
mart added a comment. +1 REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D1814 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: davidedmundson, #plasma Cc: mart, plasma-devel, sebas

[Differential] [Request, 97 lines] D1815: Add an Activities list into the context menu

2016-06-10 Thread davidedmundson (David Edmundson)
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY This makes the task manager's context menu match KWin's titlebar context menu. BUG:

[Differential] [Request, 156 lines] D1814: Add TaskManager requestActivities

2016-06-10 Thread davidedmundson (David Edmundson)
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Acts like requestVirtualDesktops only with a QStringList make runningActivities invokable

[Differential] [Commented On] D1744: Fix the ignore struts multi-screen handling

2016-06-10 Thread Martin Gräßlin
graesslin added a comment. In https://phabricator.kde.org/D1744#33668, @apol wrote: > I just tested the patch, the issue isn't solved here. can you please xprop the panel and provide the xrandr. I just want to be sure that you are not hitting a similar problem like @lbeltrame

[Differential] [Commented On] D1779: Add a parent_window event to Plasma Window interface

2016-06-10 Thread Martin Gräßlin
graesslin added a comment. In https://phabricator.kde.org/D1779#33666, @hein wrote: > Now if the latter is the semantics we want to bake into the protocol, we should make that clear in the docs and consider misuse a bug (i.e. document that it's wrong to have a window with a parent if

[Differential] [Updated] D1744: Fix the ignore struts multi-screen handling

2016-06-10 Thread apol (Aleix Pol Gonzalez)
apol added a comment. I just tested the patch, the issue isn't solved here. REPOSITORY rKWIN KWin REVISION DETAIL https://phabricator.kde.org/D1744 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graesslin, #plasma, lbeltrame, apol Cc: plasma-devel,

[Differential] [Updated] D1779: Add a parent_window event to Plasma Window interface

2016-06-10 Thread hein (Eike Hein)
hein added a comment. Hmm I'm concerned about using "transient" in the API docs and code. "Transient" is a descriptory adjective that means "temporary", i.e. if a window is a transient to some other window, it communicates two things: The association to the other window, and the expectation

Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 134 - Still Unstable!

2016-06-10 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/134/ Project: PLATFORM=Linux,compiler=gcc Date of build: Fri, 10 Jun 2016 10:55:53 + Build duration: 10 min CHANGE SET Revision

Re: Review Request 128148: Fix check to keep QMenu on screen for multiscreen layouts

2016-06-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128148/ --- (Updated June 10, 2016, 1:57 p.m.) Status -- This change has been

[Differential] [Closed] D1811: UpdateClientArea from ShellClient::doSetGeometry if the ShellClient has a strut

2016-06-10 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWIN2343a90aa517: UpdateClientArea from ShellClient::doSetGeometry if the ShellClient has a strut (authored by graesslin). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D1811?vs=4320=4325#toc

[Differential] [Closed] D1799: Add missing hasChildren() implementation.

2016-06-10 Thread hein (Eike Hein)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAWORKSPACE2d7b6692d364: Add missing hasChildren() implementation. (authored by hein). REPOSITORY rPLASMAWORKSPACE Plasma Workspace CHANGES SINCE LAST UPDATE

[Differential] [Commented On] D1811: UpdateClientArea from ShellClient::doSetGeometry if the ShellClient has a strut

2016-06-10 Thread Martin Gräßlin
graesslin added a comment. Just as an fyi: I added a test case to expose the actual problem to the change which I'll now push. REPOSITORY rKWIN KWin BRANCH update-struts-on-geometry-change REVISION DETAIL https://phabricator.kde.org/D1811 EMAIL PREFERENCES

Re: Review Request 128148: Fix check to keep QMenu on screen for multiscreen layouts

2016-06-10 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128148/#review96343 --- Ship it! Ship It! - Eike Hein On June 10, 2016, 9:56

[Differential] [Commented On] D1813: Fix selected name filter with multiple mimetypes

2016-06-10 Thread elvisangelaccio (Elvis Angelaccio)
elvisangelaccio added a comment. In https://phabricator.kde.org/D1813#33652, @mart wrote: > shouldn't be kde2QtFilter() fixed instead? I guess I can try, but I'm not sure what exactly the function does... Btw this is the commit where it was introduced:

[Differential] [Commented On] D1813: Fix selected name filter with multiple mimetypes

2016-06-10 Thread mart (Marco Martin)
mart added a comment. shouldn't be kde2QtFilter() fixed instead? REPOSITORY rPLASMAINTEGRATION Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D1813 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To:

[Differential] [Accepted] D1793: [Device Notifier] Be more declarative

2016-06-10 Thread mart (Marco Martin)
mart accepted this revision. mart added a reviewer: mart. This revision is now accepted and ready to land. REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D1793 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To:

[Differential] [Closed] D1803: Fix the strut handling for wayland clients

2016-06-10 Thread Martin Gräßlin
This revision was automatically updated to reflect the committed changes. Closed by commit rKWIN58db4777966e: Fix the strut handling for wayland clients (authored by graesslin). REPOSITORY rKWIN KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1803?vs=4299=4323 REVISION DETAIL

[Differential] [Request, 25 lines] D1813: Fix selected name filter with multiple mimetypes

2016-06-10 Thread elvisangelaccio (Elvis Angelaccio)
elvisangelaccio created this revision. elvisangelaccio added a reviewer: Plasma. elvisangelaccio added a subscriber: Plasma. elvisangelaccio set the repository for this revision to rPLASMAINTEGRATION Integration for Qt applications in Plasma. elvisangelaccio added a project: Plasma. Restricted

[Differential] [Commented On] D1803: Fix the strut handling for wayland clients

2016-06-10 Thread bshah (Bhushan Shah)
bshah added a comment. Sounds sane, but not completely confident to Accept it. REPOSITORY rKWIN KWin REVISION DETAIL https://phabricator.kde.org/D1803 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graesslin, #plasma_on_wayland, #kwin Cc: bshah,

Review Request 128148: Fix check to keep QMenu on screen for multiscreen layouts

2016-06-10 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128148/ --- Review request for KDE Frameworks and Plasma. Repository:

[Differential] [Accepted] D1811: UpdateClientArea from ShellClient::doSetGeometry if the ShellClient has a strut

2016-06-10 Thread bshah (Bhushan Shah)
bshah accepted this revision. bshah added a reviewer: bshah. This revision is now accepted and ready to land. REPOSITORY rKWIN KWin BRANCH update-struts-on-geometry-change REVISION DETAIL https://phabricator.kde.org/D1811 EMAIL PREFERENCES

Re: New idea for Plasma's logo

2016-06-10 Thread Alessandro Longo
I like a lot the idea. I suggest to invite people to post their logos to visual-design mailing list, to to let them know that it exists. Sending an email with an attachment should be enough accessible for everyone and we will have a transparent process and a log of discussions. -- Alessandro

Re: New idea for Plasma's logo

2016-06-10 Thread Alessandro Longo
Il 09-06-2016 11:49 Thomas Pfeiffer ha scritto: You posted a screenshot from your system on Google+ [1] where you use your proposed logo (although not in monochrome) as a launcher icon. This might have added to the confusion. [1]

[Differential] [Request, 3 lines] D1812: Update window geometry when creating the decoration in ShellClient

2016-06-10 Thread Martin Gräßlin
graesslin created this revision. graesslin added reviewers: KWin, Plasma on Wayland. Restricted Application added subscribers: kwin, plasma-devel. Restricted Application added projects: Plasma on Wayland, KWin. REVISION SUMMARY When managing a new decorated ShellClient we observed that the

Re: Review Request 128144: Windecoration KCM theme color

2016-06-10 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128144/#review96340 --- \-1 Using Plasma theme in widgets-dialogs is a no-go.