[Powerdevil] [Bug 382105] Power devil crash dès la sortie de veille

2017-07-07 Thread Christoph Feck
https://bugs.kde.org/show_bug.cgi?id=382105 Christoph Feck changed: What|Removed |Added Severity|normal |crash -- You are receiving

[Powerdevil] [Bug 382105] Power devil crash dès la sortie de veille

2017-07-07 Thread kristof
https://bugs.kde.org/show_bug.cgi?id=382105 --- Comment #1 from kristof --- Application: org_kde_powerdevil () Qt Version: 5.6.1 Frameworks Version: 5.35.0 Operating System: Linux 4.8.0-58-generic x86_64 Distribution: Linux Mint 18.2 Sonya -- Information about the crash:

[Powerdevil] [Bug 382105] New: Power devil crash dès la sortie de veille

2017-07-07 Thread kristof
https://bugs.kde.org/show_bug.cgi?id=382105 Bug ID: 382105 Summary: Power devil crash dès la sortie de veille Product: Powerdevil Version: unspecified Platform: Other OS: Linux Status: UNCONFIRMED

D1231: Add Remote Access interface to KWayland

2017-07-07 Thread Oleg Chernovskiy
Kanedias added a comment. I'll get rid of the fakeinput-related changes and test it with KRfb tomorrow. @davidedmundson , can you test this with multiple outputs? REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D1231 To: Kanedias, graesslin, davidedmundson Cc:

D1231: Add Remote Access interface to KWayland

2017-07-07 Thread Oleg Chernovskiy
Kanedias updated this revision to Diff 16318. Kanedias added a comment. Added wl_output reference for buffers as requested by @davidedmundson and @graesslin REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1231?vs=15553=16318 REVISION DETAIL

[Breeze] [Bug 381407] SDDM's Breeze theme should have a prettier background than just a flat blue color

2017-07-07 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=381407 --- Comment #3 from Nate Graham --- Ah, you;re right. I still think it's weird to have just an ugly plain blue background for both, but I guess my lone opinion doesn't carry much weight, and it *is* customizable. -- You are

D6554: [Plasma Dialog] Fix flag condition

2017-07-07 Thread David Edmundson
davidedmundson added a comment. Your description doesn't match the code > This didn't work when other flags,, such as WindowStaysOnTopHint were present. & Flag is the same as TestFlag() From what I can tell, you've changed: if (!tooltipFlag && !notification) into if(!

D6554: [Plasma Dialog] Fix flag condition

2017-07-07 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D6554 To: broulik, #plasma, mart Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,

D6555: [Applet Alternatives] Set applet status to RequiresAttentionStatus while open

2017-07-07 Thread Kai Uwe Broulik
broulik edited the test plan for this revision. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D6555 To: broulik, #plasma Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D6555: [Applet Alternatives] Set applet status to RequiresAttentionStatus while open

2017-07-07 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY This ensures the panel stays visible while applet alternatives dialog is open. It also works around a quirk in Plasma Dialog which won't

D6554: [Plasma Dialog] Fix flag condition

2017-07-07 Thread Kai Uwe Broulik
broulik edited the summary of this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D6554 To: broulik, #plasma Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D6554: [Plasma Dialog] Fix flag condition

2017-07-07 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY The code wants to check whether we're not a tooltip. This didn't work. TEST PLAN My Alternatives popup now slides in

Re: KWin 5.10.3.1 update

2017-07-07 Thread Maximiliano Curia
¡Hola Jonathan! El 2017-07-03 a las 17:26 +0100, Jonathan Riddell escribió: KWin has received an update to the 5.10.3 release. 5.10.3.1 contains a fix for https://bugs.kde.org/show_bug.cgi?id=381870 Freeze in KWin::checkGLError on startup which affects machines with an NVidia card. The

D6552: RFC: [Folder View] Reply mouse click so left button containment actions work

2017-07-07 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY When we don't initiate a rubber band selection, replay the mouse event so containment actions assigned to the left mouse button work.

[Powerdevil] [Bug 378498] Power settings missing scrollbar

2017-07-07 Thread Christoph Feck
https://bugs.kde.org/show_bug.cgi?id=378498 Christoph Feck changed: What|Removed |Added CC||domenico.iezzi@gmail.co

[Powerdevil] [Bug 382097] Powerdevil config window too big and unresizable

2017-07-07 Thread Christoph Feck
https://bugs.kde.org/show_bug.cgi?id=382097 Christoph Feck changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[Powerdevil] [Bug 382097] New: Powerdevil config window too big and unresizable

2017-07-07 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=382097 Bug ID: 382097 Summary: Powerdevil config window too big and unresizable Product: Powerdevil Version: 5.10.3 Platform: Archlinux Packages OS: Linux Status: UNCONFIRMED

D6544: Don't draw focus line on selected list item

2017-07-07 Thread Kai Uwe Broulik
broulik added a comment. Updated, thanks. https://cgit.kde.org/breeze.git/commit/?id=6906919371ff92ff0dc7f6ae4e4e14310c5e1e53 REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D6544 To: broulik, #plasma, hpereiradacosta, fvogt Cc: cfeck, plasma-devel, ZrenBot,

D6551: [Task Manager] Also respect visible property of configure action

2017-07-07 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY When Plasma is locked down, configure actions are hidden. In Task Manager this was not taken into account and the action was only disabled

D6544: Don't draw focus line on selected list item

2017-07-07 Thread Christoph Feck
cfeck added inline comments. INLINE COMMENTS > cfeck wrote in breezestyle.cpp:3082 > Of course inherits(QString) is slower than qobject_cast<>. > > The only reason to use inherits() is when you do not link to the library that > the class defines. Also order to make the fastest test come

D6544: Don't draw focus line on selected list item

2017-07-07 Thread Christoph Feck
cfeck added inline comments. INLINE COMMENTS > broulik wrote in breezestyle.cpp:3082 > From what I can tell the difference is negligible. `qobject_cast` goes > through `QMetaObject::cast` which then calls `inherits` internally, so I'd > say `inherits` is the better choice. > > I didn't

D6550: [Task Manager] Set status to RequiresAttentionStatus while context menu is open

2017-07-07 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY This ensures the panel does not auto-hide then. There can only be one context menu at a time, so I moved the property to the root item so

D6527: [Containment Interface] Keep containment in RequiresAttentionStatus while context menu is open

2017-07-07 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R242:f3dcff28b8fb: [Containment Interface] Keep containment in RequiresAttentionStatus while… (authored by broulik). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE

D6544: Don't draw focus line on selected list item

2017-07-07 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R31:89b916039387: Don't draw focus line on selected list item (authored by broulik). REPOSITORY R31 Breeze CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6544?vs=16287=16297 REVISION DETAIL

D6549: Look for QtGraphicalEffects, so packagers don't forget it

2017-07-07 Thread Aleix Pol Gonzalez
apol added a comment. In https://phabricator.kde.org/D6549#122629, @sitter wrote: > Probably should go to ECM instead of being copy pasted. Yes, right... xD REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D6549 To: apol, #kirigami, sitter, mart Cc:

D6549: Look for QtGraphicalEffects, so packagers don't forget it

2017-07-07 Thread Harald Sitter
sitter added a comment. Probably should go to ECM instead of being copy pasted. REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D6549 To: apol, #kirigami, sitter, mart Cc: plasma-devel, apol, mart

D6549: Look for QtGraphicalEffects, so packagers don't forget it

2017-07-07 Thread Aleix Pol Gonzalez
apol created this revision. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. REPOSITORY R169 Kirigami BRANCH master REVISION DETAIL https://phabricator.kde.org/D6549 AFFECTED FILES CMakeLists.txt cmake/FindQMLModule.cmake.in

D6544: Don't draw focus line on selected list item

2017-07-07 Thread Hugo Pereira Da Costa
hpereiradacosta accepted this revision. hpereiradacosta added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > broulik wrote in breezestyle.cpp:3082 > From what I can tell the difference is negligible. `qobject_cast` goes > through `QMetaObject::cast` which

D6544: Don't draw focus line on selected list item

2017-07-07 Thread Kai Uwe Broulik
broulik added inline comments. INLINE COMMENTS > hpereiradacosta wrote in breezestyle.cpp:3082 > Not sure which one is the most efficient: > Widget->inherits or qobject_cast ? > (My guess would need the second, but ...) > Can you double check ? >From what I can tell the difference is

D6544: Don't draw focus line on selected list item

2017-07-07 Thread Hugo Pereira Da Costa
hpereiradacosta added inline comments. INLINE COMMENTS > breezestyle.cpp:3082 > +// no focus indicator on selected list items > +if (widget && widget->inherits("QAbstractItemView") && (state & > State_Selected)) > +{ return true; } Not sure which one is the most

[Breeze] [Bug 381407] SDDM's Breeze theme should have a prettier background than just a flat blue color

2017-07-07 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=381407 David Edmundson changed: What|Removed |Added CC|

D6544: Don't draw focus line on selected list item

2017-07-07 Thread Kai Uwe Broulik
broulik edited the test plan for this revision. REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D6544 To: broulik, #plasma, hpereiradacosta, fvogt Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D6544: Don't draw focus line on selected list item

2017-07-07 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY A fix I did for the focus line being drawn outside item boundaries resulted in the item selection rectangle being "cut off" by the focus

[Powerdevil] [Bug 382088] New: power on battery: Does not consider mutiple batteries

2017-07-07 Thread Pascal d'Hermilly
https://bugs.kde.org/show_bug.cgi?id=382088 Bug ID: 382088 Summary: power on battery: Does not consider mutiple batteries Product: Powerdevil Version: 5.10.3 Platform: Other OS: Linux Status: UNCONFIRMED