Theming issues noted while porting Weather applet

2016-02-26 Thread Friedrich W. H. Kossebau
Hi, happy to have the weather applet almost revived and good enough to have your eyes on it (hopefully in time for 5.6), see parallel emails for https:// git.reviewboard.kde.org/r/127193/ Some issues I noticed which seem unrelated to the weather applet code, are these known or should be filed a

Re: Review Request 127193: Revival of Weather applet (for Plasma 5.6)

2016-02-26 Thread Friedrich W. H. Kossebau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127193/ --- (Updated Feb. 27, 2016, 1:18 a.m.) Review request for Plasma and Marco Ma

Review Request 127193: Revival of Weather applet (for Plasma 5.6)

2016-02-26 Thread Friedrich W. H. Kossebau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127193/ --- Review request for Plasma and Marco Martin. Repository: kdeplasma-addons

Re: Review Request 126593: [Task Manager] Add Jump List actions to grouped tasks

2016-02-26 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126593/ --- (Updated Feb. 26, 2016, 8 p.m.) Status -- This change has been marke

Jenkins-kde-ci: plasma-desktop Plasma-5.5 stable-kf5-qt5 » Linux,gcc - Build # 71 - Still Failing!

2016-02-26 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-desktop%20Plasma-5.5%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/71/ Project: PLATFORM=Linux,compiler=gcc Date of build: Fri, 26 Feb 2016 19:45:37 + Build duration: 2 min 10 sec CHANGE SET Revision 1b5307e78dacea388c18c

[Differential] [Closed] D1027: [FolderView] HIde tooltip when beginning drag

2016-02-26 Thread broulik (Kai Uwe Broulik)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOP05de074748ab: [FolderView] Hide tooltip when beginning drag (authored by broulik). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1027?vs=

Re: Review Request 127102: Use fixed width for digital clock applet

2016-02-26 Thread Daniel Faust
> On Feb. 18, 2016, 1:05 vorm., David Edmundson wrote: > > applets/digital-clock/package/contents/ui/DigitalClock.qml, line 555 > > > > > > rather than looping, can we use FontMetric's maximumCharacterWidth > >

[Differential] [Accepted] D1027: [FolderView] HIde tooltip when beginning drag

2016-02-26 Thread hein (Eike Hein)
hein accepted this revision. This revision is now accepted and ready to land. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D1027 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, hein Cc: plasma-devel __

[Differential] [Request, 1 line] D1027: [FolderView] HIde tooltip when beginning drag

2016-02-26 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added a reviewer: hein. broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop. broulik added a project: Plasma. Herald added a subscriber: plasma-devel. REVISION SUMMARY This prevents the tooltip from shooting to the top left corner

[Differential] [Closed] D1026: [FolderModel] Simplify deleting existing drag image

2016-02-26 Thread broulik (Kai Uwe Broulik)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOPd59af0bff7b2: [FolderModel] Simplify deleting existing drag image (authored by broulik). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D10

[Differential] [Closed] D1021: [FolderView] Allow chosing list or icon view in popup

2016-02-26 Thread broulik (Kai Uwe Broulik)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOP28aae4bc188c: [FolderView] Allow chosing list or icon view in popup (authored by broulik). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D

[Differential] [Accepted] D1021: [FolderView] Allow chosing list or icon view in popup

2016-02-26 Thread hein (Eike Hein)
hein accepted this revision. This revision is now accepted and ready to land. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D1021 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, Plasma, hein Cc: plasma-deve

[Differential] [Accepted] D1026: [FolderModel] Simplify deleting existing drag image

2016-02-26 Thread hein (Eike Hein)
hein accepted this revision. This revision is now accepted and ready to land. REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D1026 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, Plasma, hein Cc: plasma-deve

[Differential] [Updated, 188 lines] D1021: [FolderView] Allow chosing list or icon view in popup

2016-02-26 Thread broulik (Kai Uwe Broulik)
broulik updated this revision to Diff 2473. broulik added a comment. - Fix switching between list view and icon view at runtime Using States {} I can achieve atomic anchor changes; it seems when you use regular bindings and you have horizontal/verticalCenter and a left/right/top/bottom a

[Differential] [Request, 10 lines] D1026: [FolderModel] Simplify deleting existing drag image

2016-02-26 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, hein. broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop. broulik added a project: Plasma. Herald added a subscriber: plasma-devel. REVISION SUMMARY Instead of looking up three times (contains, value, remo

[Differential] [Updated] D1025: [DrKonqi] Avoid duplicate "Help" button

2016-02-26 Thread broulik (Kai Uwe Broulik)
broulik updated the summary for this revision. REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D1025 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, Plasma, dfaure Cc: plasma-devel ___

[Differential] [Request, 1 line] D1025: [DrKonqi] Avoid duplicate "Help" button

2016-02-26 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, dfaure. broulik set the repository for this revision to rPLASMAWORKSPACE Plasma Workspace. broulik added a project: Plasma. Herald added a subscriber: plasma-devel. REVISION SUMMARY For some reason I got two "Help" buttons in the c

Jenkins-kde-ci: plasma-workspace Plasma-5.5 stable-kf5-qt5 » Linux,gcc - Build # 62 - Still Failing!

2016-02-26 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-workspace%20Plasma-5.5%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/62/ Project: PLATFORM=Linux,compiler=gcc Date of build: Fri, 26 Feb 2016 13:25:06 + Build duration: 1 min 55 sec CHANGE SET Revision e722b1e5f214ec1d007

[Differential] [Commented On] D1021: [FolderView] Allow chosing list or icon view in popup

2016-02-26 Thread hein (Eike Hein)
hein added a comment. So basically to get this moving along I'd like investigation of why those delegates don't fix themselves -- let's see if we can't avoid adding a new FIXME? REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D1021 EMAIL PREFERENCES

Re: Plasma Mobile Components -> Kirigami (?)

2016-02-26 Thread Thomas Pfeiffer
On Mittwoch, 24. Februar 2016 23:33:45 CET Dirk Hohndel wrote: > On Wed, Feb 24, 2016 at 10:52:08PM +0100, Thomas Pfeiffer wrote: > > > > It is physical, it is creative, kinda playful without being childish, > > > > and > > > > it's far less common than Origami, which helps with searchability. > >

[Differential] [Accepted] D1022: [server] Properly initialize Cursor

2016-02-26 Thread bshah (Bhushan Shah)
bshah accepted this revision. bshah added a reviewer: bshah. This revision is now accepted and ready to land. REPOSITORY rKWAYLAND KWayland BRANCH server-fix-cursor-damage REVISION DETAIL https://phabricator.kde.org/D1022 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emai

[Differential] [Request, 75 lines] D1022: [server] Properly initialize Cursor

2016-02-26 Thread Martin Gräßlin
graesslin created this revision. graesslin added a reviewer: Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. REVISION SUMMARY The Cursor wasn't properly initialized. E.g. the damage signal didn't get connected resulting in the server not noticing when the curso