Re: [PATCH] [kcms/phonon] Don't overwrite audio profile entries with same priority

2015-06-16 Thread Sebastian Kügler
On Monday, June 15, 2015 23:45:46 Bernie Innocenti wrote:
> On 15/06/15 22:01, Sebastian Kügler wrote:
> > Easily: https://identity.kde.org/index.php?r=registration/index
> 
>  >
>  > [...]
>  >
> 
> > Ah, thanks for the note. I've updated the documentation about sending
> > patches on Techbase.
> 
> Awesome, thanks for getting me started, I posted the review.
> 
> A few more suggestions to help streamline the new contributor onboarding 
> process:
> 
>   - Add a note about reviewboard in HACKING, that's where I looked first
> 
>   - Move the "Reviewboard" paragraph near the top of 
> https://techbase.kde.org/Contribute/Send_Patches
> 
>   - Add a link from https://techbase.kde.org/Development/Review_Board to 
> https://techbase.kde.org/Contribute/Get_a_Contributor_Account#Apply_for_an_a
> ccount
> 
>   - The paragraph "Who Can Apply For a KDE Contributor Account?" led me 
> to believe that a casual contribution did not qualify for an account. 
> Now I realize that I was confusing the basic identity with a full 
> developer account, but it would be good to clarify it near the top of 
> the page.
> 
>   - The Reviewboard page talks about a cli tool called "post-review", 
> but I had to use "rbt post" on Fedora (the rpm is called RBTools, like 
> on Suse).

Sounds all sensible. It's a wiki, though, and I'm sure nobody will mind (but 
indeed anybody will be thankful) if you went ahead and changed these things. I 
just did a rough go-over to update the most glaring anachronisms. If you're 
unsure about detailed changes, I can review the document once done.

Cheers,
-- 
sebas

Sebastian Kügler|http://vizZzion.org| http://kde.org

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Repeating item title in context menu

2015-06-16 Thread Sebastian Kügler
On Tuesday, June 16, 2015 15:02:19 Kai Uwe Broulik wrote:
> I discovered that Kickoff repeats the item title in its context menu, so
> when  you right click an application, the application title will be shown
> at the top of the menu. I've seen this kind of behavior only in one other
> place, the systray.
> 
> Imho I don't see why the context menu for the item you just right clicked,
> and  that is visually positioned next to it, should have the item shown in
> the menu. Moreover, Kickoff misuses a disabled entry for that and Hugo is
> not fond of adding support for icons in menu sections because that'll look
> ugly.
> 
> Any suggestions?

Kill it? I may have added it (don't remember, happend during a porting spree, 
maybe it was someone else), but I fully agree with your assessment.

Cheers,
-- 
sebas

Sebastian Kügler|http://vizZzion.org| http://kde.org

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Lockscreen multiscreen behavior

2015-06-16 Thread Martin Gräßlin

Am 2015-06-16 16:58, schrieb David Kahles:

Hello,
I want to hear some opinions about the multiscreen behavior of the 
lockscreen.


Currently the lockscreen is displayed on all connected monitors. I
would prefer if the
lockscreen is displayed only on the primary monitor, as it's done at
the loginscreen,
because there is no need for multiple instances.
I think this also matches the behavior of other operating
systems/desktop environments.

What do you think?


When we redid the lockscreen to be in QML with the lock always shown we 
did it that way on purpose. The main problem is that we don't know which 
screen the user is expecting it to be on. Even more we don't know 
whether the primary screen is actually the user's primary screen. We 
cannot expect users to configure it as they use it.


Easy example: notebook, external screen connected, internal screen is 
obviously the primary screen as it's a notebook. Now user closes the 
lid, screen stays configured to be on and locks the screen: at that 
point user has a broken system because lock screen is not shown any 
more. User might even try to hard turn off the system at that point, 
because user doesn't even think of the idea the closed notebook screen 
could have the locker.


So overall: if we want to be sure that users always can lock in again, 
we need to show on all screens.


Cheers
Martin
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124114: Add "Lock Screen" to desktop toolbox

2015-06-16 Thread David Edmundson

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124114/#review81518
---

Ship it!


> This does not check KAuthorized before to hide the entry but neither does the 
> context menu.

Well that's a bit rubbish. 
Ship this, but lets remember to fix both, poke me if it gets near release and I 
haven't.

BTW, is it possible to power this menu from the QActions we have in Plasma, I 
haven't looked but it should reduce some code?

- David Edmundson


On June 16, 2015, 8:18 p.m., Kai Uwe Broulik wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124114/
> ---
> 
> (Updated June 16, 2015, 8:18 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> It used to be present in 4.x and makes it consistent with what the context 
> menu provides. This does not check KAuthorized before to hide the entry but 
> neither does the context menu. As suggested in 
> https://forum.kde.org/viewtopic.php?f=289&t=126770
> 
> 
> Diffs
> -
> 
>   toolboxes/desktoptoolbox/contents/ui/ToolBoxItem.qml c2a36e9 
> 
> Diff: https://git.reviewboard.kde.org/r/124114/diff/
> 
> 
> Testing
> ---
> 
> Works
> 
> 
> File Attachments
> 
> 
> Lock screen entry
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/06/16/1c037bef-4280-4da8-bf9c-7cdc3d0e37b6__lockscreentoolbox.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124113: Ignore invalid X-Plasma-ConfigPlugins

2015-06-16 Thread David Edmundson

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124113/#review81517
---

Ship it!


Ship It!

- David Edmundson


On June 16, 2015, 6:33 p.m., Kai Uwe Broulik wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124113/
> ---
> 
> (Updated June 16, 2015, 6:33 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> ---
> 
> Prevents issues like Bug 349250
> 
> 
> Diffs
> -
> 
>   src/plasmaquick/configview.cpp e6abdf6 
> 
> Diff: https://git.reviewboard.kde.org/r/124113/diff/
> 
> 
> Testing
> ---
> 
> No longer get empty pages without title and icon
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 124114: Add "Lock Screen" to desktop toolbox

2015-06-16 Thread Kai Uwe Broulik

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124114/
---

Review request for Plasma.


Repository: plasma-desktop


Description
---

It used to be present in 4.x and makes it consistent with what the context menu 
provides. This does not check KAuthorized before to hide the entry but neither 
does the context menu. As suggested in 
https://forum.kde.org/viewtopic.php?f=289&t=126770


Diffs
-

  toolboxes/desktoptoolbox/contents/ui/ToolBoxItem.qml c2a36e9 

Diff: https://git.reviewboard.kde.org/r/124114/diff/


Testing
---

Works


File Attachments


Lock screen entry
  
https://git.reviewboard.kde.org/media/uploaded/files/2015/06/16/1c037bef-4280-4da8-bf9c-7cdc3d0e37b6__lockscreentoolbox.png


Thanks,

Kai Uwe Broulik

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 348882] KDE cannot set backlight anymore, kernel can

2015-06-16 Thread azrdev
https://bugs.kde.org/show_bug.cgi?id=348882

--- Comment #2 from azr...@googlemail.com ---
@Kai: maybe this stems from the KDE4 → KF5 transition of archlinux, which
started in december(?) by moving applications each as soon as it was ported -
at least that's how I understood it. plasma was not yet migrated at that point,
I'm not sure if it is ready now - the arch wiki [1][2] mostly says it's an
exclusive decision, i.e. I cannot install plasma4 and 5 simultaneously.

Having said that, my system currently consists of the plasma4 desktop, and some
of the applications from KF5 (gwenview, konsole, ...), the others still on KDE4
(systemsettings, dolphin, ...). Both kded4 and kded5 are running.

[1] 
[2] 

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 124113: Ignore invalid X-Plasma-ConfigPlugins

2015-06-16 Thread Kai Uwe Broulik

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124113/
---

Review request for Plasma.


Repository: plasma-framework


Description
---

Prevents issues like Bug 349250


Diffs
-

  src/plasmaquick/configview.cpp e6abdf6 

Diff: https://git.reviewboard.kde.org/r/124113/diff/


Testing
---

No longer get empty pages without title and icon


Thanks,

Kai Uwe Broulik

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124112: Implement drag and drop for Trash applet

2015-06-16 Thread Kai Uwe Broulik

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124112/
---

(Updated Juni 16, 2015, 6:32 nachm.)


Review request for Plasma.


Summary (updated)
-

Implement drag and drop for Trash applet


Bugs: 348378
https://bugs.kde.org/show_bug.cgi?id=348378


Repository: plasma-desktop


Description
---

This allows to drag files onto the trash applet to trash them.
Also, the code is cleaned up a bit and it is given a sensible minimum size (so 
the label always fits) as well as the query dialog has been replaced by the 
standard KIO dialog.


Diffs
-

  applets/trash/package/contents/ui/main.qml 7715439 
  applets/trash/plugin/CMakeLists.txt 9012e66 
  applets/trash/plugin/trash.h PRE-CREATION 
  applets/trash/plugin/trash.cpp PRE-CREATION 
  applets/trash/plugin/trashplugin.cpp 7e866a9 

Diff: https://git.reviewboard.kde.org/r/124112/diff/


Testing
---

Trashed one or multiple files on both desktop and panel.
When I dismiss the message box, however, Plasma will create an applet for the 
file dragged. How can I tell it I accepted the drop? Also, it crashes in 
Client::~Client() () from /usr/lib/x86_64-linux-gnu/libQPulseAudioPrivate.so at 
times after dismissing the dialog.


Thanks,

Kai Uwe Broulik

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 348882] KDE cannot set backlight anymore, kernel can

2015-06-16 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=348882

Kai Uwe Broulik  changed:

   What|Removed |Added

 CC||k...@privat.broulik.de

--- Comment #1 from Kai Uwe Broulik  ---
Those packages confuse me. You have workspace 4.11.19 installed as well as
plasma 15.04?! There's no plasma 15.04, that's application versioning scheme.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 348882] KDE cannot set backlight anymore, kernel can

2015-06-16 Thread Christoph Feck
https://bugs.kde.org/show_bug.cgi?id=348882

Christoph Feck  changed:

   What|Removed |Added

Version|4.11.5  |unspecified
   Assignee|unassigned-b...@kde.org |plasma-devel@kde.org
  Component|general |general
Product|kde |Powerdevil

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 124112: Implement drag and drop for Tash applet

2015-06-16 Thread Kai Uwe Broulik

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124112/
---

Review request for Plasma.


Bugs: 348378
https://bugs.kde.org/show_bug.cgi?id=348378


Repository: plasma-desktop


Description
---

This allows to drag files onto the trash applet to trash them.
Also, the code is cleaned up a bit and it is given a sensible minimum size (so 
the label always fits) as well as the query dialog has been replaced by the 
standard KIO dialog.


Diffs
-

  applets/trash/package/contents/ui/main.qml 7715439 
  applets/trash/plugin/CMakeLists.txt 9012e66 
  applets/trash/plugin/trash.h PRE-CREATION 
  applets/trash/plugin/trash.cpp PRE-CREATION 
  applets/trash/plugin/trashplugin.cpp 7e866a9 

Diff: https://git.reviewboard.kde.org/r/124112/diff/


Testing
---

Trashed one or multiple files on both desktop and panel.
When I dismiss the message box, however, Plasma will create an applet for the 
file dragged. How can I tell it I accepted the drop? Also, it crashes in 
Client::~Client() () from /usr/lib/x86_64-linux-gnu/libQPulseAudioPrivate.so at 
times after dismissing the dialog.


Thanks,

Kai Uwe Broulik

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Lockscreen multiscreen behavior

2015-06-16 Thread David Kahles

Hello,
I want to hear some opinions about the multiscreen behavior of the 
lockscreen.


Currently the lockscreen is displayed on all connected monitors. I would 
prefer if the
lockscreen is displayed only on the primary monitor, as it's done at the 
loginscreen,

because there is no need for multiple instances.
I think this also matches the behavior of other operating 
systems/desktop environments.


What do you think?

David
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124110: SystemLoadViewer opens KSysGuard on click

2015-06-16 Thread Bhushan Shah


> On June 16, 2015, 4:40 p.m., Bhushan Shah wrote:
> > Thanks for patch, do you have commit access or I can commit that for you.
> 
> Joshua Worth wrote:
> I don't think I do, so that would be great thanks. This is the first 
> patch I submitted by myself.

Done, submitted with 
http://commits.kde.org/kdeplasma-addons/6ecc5fc44e9ffc1535bfff1157be1b3efef875e2


- Bhushan


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124110/#review81510
---


On June 16, 2015, 6:54 p.m., Joshua Worth wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124110/
> ---
> 
> (Updated June 16, 2015, 6:54 p.m.)
> 
> 
> Review request for Plasma, David Edmundson and Martin Yrjölä.
> 
> 
> Repository: kdeplasma-addons
> 
> 
> Description
> ---
> 
> The SystemLoadViewer widget should open KSysGuard when clicked, but the 
> DataSource was set to the wrong source
> 
> 
> Diffs
> -
> 
>   applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 43acab4 
> 
> Diff: https://git.reviewboard.kde.org/r/124110/diff/
> 
> 
> Testing
> ---
> 
> Tested in panel and on desktop
> 
> 
> Thanks,
> 
> Joshua Worth
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124110: SystemLoadViewer opens KSysGuard on click

2015-06-16 Thread Joshua Worth

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124110/
---

(Updated June 16, 2015, 1:24 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma, David Edmundson and Martin Yrjölä.


Changes
---

Submitted with commit 6ecc5fc44e9ffc1535bfff1157be1b3efef875e2 by Bhushan Shah 
on behalf of Joshua Worth to branch Plasma/5.3.


Repository: kdeplasma-addons


Description
---

The SystemLoadViewer widget should open KSysGuard when clicked, but the 
DataSource was set to the wrong source


Diffs
-

  applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 43acab4 

Diff: https://git.reviewboard.kde.org/r/124110/diff/


Testing
---

Tested in panel and on desktop


Thanks,

Joshua Worth

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124110: SystemLoadViewer opens KSysGuard on click

2015-06-16 Thread Joshua Worth


> On June 16, 2015, 9:10 p.m., Bhushan Shah wrote:
> > Thanks for patch, do you have commit access or I can commit that for you.

I don't think I do, so that would be great thanks. This is the first patch I 
submitted by myself.


- Joshua


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124110/#review81510
---


On June 16, 2015, 9:05 p.m., Joshua Worth wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124110/
> ---
> 
> (Updated June 16, 2015, 9:05 p.m.)
> 
> 
> Review request for Plasma, David Edmundson and Martin Yrjölä.
> 
> 
> Repository: kdeplasma-addons
> 
> 
> Description
> ---
> 
> The SystemLoadViewer widget should open KSysGuard when clicked, but the 
> DataSource was set to the wrong source
> 
> 
> Diffs
> -
> 
>   applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 43acab4 
> 
> Diff: https://git.reviewboard.kde.org/r/124110/diff/
> 
> 
> Testing
> ---
> 
> Tested in panel and on desktop
> 
> 
> Thanks,
> 
> Joshua Worth
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Repeating item title in context menu

2015-06-16 Thread Kai Uwe Broulik
Hi all,

I discovered that Kickoff repeats the item title in its context menu, so when 
you right click an application, the application title will be shown at the top 
of the menu. I've seen this kind of behavior only in one other place, the 
systray.

Imho I don't see why the context menu for the item you just right clicked, and 
that is visually positioned next to it, should have the item shown in the 
menu. Moreover, Kickoff misuses a disabled entry for that and Hugo is not fond 
of adding support for icons in menu sections because that'll look ugly.

Any suggestions?

Cheers,
Kai Uwe
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 349226] battery updated not updated after suspend to memory

2015-06-16 Thread m.wege
https://bugs.kde.org/show_bug.cgi?id=349226

--- Comment #4 from m.w...@web.de ---
5.9.0-0ubuntu1

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: testing the single QQmlEngine plasmashell

2015-06-16 Thread Kai Uwe Broulik
> with x-test language seems to work now..

Works for me as well now. Thanks! :)


___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 349226] battery updated not updated after suspend to memory

2015-06-16 Thread Lukáš Tinkl
https://bugs.kde.org/show_bug.cgi?id=349226

--- Comment #3 from Lukáš Tinkl  ---
and "libkf5solid"?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 349226] battery updated not updated after suspend to memory

2015-06-16 Thread m.wege
https://bugs.kde.org/show_bug.cgi?id=349226

--- Comment #2 from m.w...@web.de ---
Plasma-Version is 5.3.1
Powerdevil: 4:5.3.1-0ubuntu1~ubuntu15.04~ppa1

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124110: SystemLoadViewer opens KSysGuard on click

2015-06-16 Thread Bhushan Shah

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124110/#review81510
---

Ship it!


Thanks for patch, do you have commit access or I can commit that for you.

- Bhushan Shah


On June 16, 2015, 4:35 p.m., Joshua Worth wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124110/
> ---
> 
> (Updated June 16, 2015, 4:35 p.m.)
> 
> 
> Review request for Plasma, David Edmundson and Martin Yrjölä.
> 
> 
> Repository: kdeplasma-addons
> 
> 
> Description
> ---
> 
> The SystemLoadViewer widget should open KSysGuard when clicked, but the 
> DataSource was set to the wrong source
> 
> 
> Diffs
> -
> 
>   applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 43acab4 
> 
> Diff: https://git.reviewboard.kde.org/r/124110/diff/
> 
> 
> Testing
> ---
> 
> Tested in panel and on desktop
> 
> 
> Thanks,
> 
> Joshua Worth
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 349226] battery updated not updated after suspend to memory

2015-06-16 Thread Lukáš Tinkl
https://bugs.kde.org/show_bug.cgi?id=349226

Lukáš Tinkl  changed:

   What|Removed |Added

 CC||lu...@kde.org
 Status|UNCONFIRMED |NEEDSINFO
 Resolution|--- |WAITINGFORINFO

--- Comment #1 from Lukáš Tinkl  ---
Already fixed in Solid:
https://projects.kde.org/projects/frameworks/solid/repository/revisions/125e37f155788f60b3b58febbb8789aeda63c51f
3 months ago. What's your version of the solid framework/package?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Review Request 124110: SystemLoadViewer opens KSysGuard on click

2015-06-16 Thread Joshua Worth

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124110/
---

Review request for Plasma, David Edmundson and Martin Yrjölä.


Repository: kdeplasma-addons


Description
---

The SystemLoadViewer widget should open KSysGuard when clicked, but the 
DataSource was set to the wrong source


Diffs
-

  applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 43acab4 

Diff: https://git.reviewboard.kde.org/r/124110/diff/


Testing
---

Tested in panel and on desktop


Thanks,

Joshua Worth

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 349226] New: battery updated not updated after suspend to memory

2015-06-16 Thread m.wege
https://bugs.kde.org/show_bug.cgi?id=349226

Bug ID: 349226
   Summary: battery updated not updated after suspend to memory
   Product: Powerdevil
   Version: 5.3.1
  Platform: Ubuntu Packages
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: plasma-devel@kde.org
  Reporter: m.w...@web.de

When close the lid (suspend to memory) the battery applet is not updated  in
the panel. When waking up the battery was at 9%, I plugged in the charger. The
battery applet now tell there is 9% and that the battery is charging. The 9%
never changes. Then I click on the applet, the longer version of it tells
correctly, that it is charging, now at 79%.
When the screen locker kicks in, there is also a battery icon which also says
the battery is at 9%.

Reproducible: Always

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 124108: [kcms/phonon] Don't overwrite audio profile entries with same priority

2015-06-16 Thread Harald Sitter

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124108/#review81509
---


looks good to me. I'll have to take a closer look at the code paths using it 
but at a glance it doesn't seem to concerning to switch to a multimap.

on schedule within the next 2 days.

- Harald Sitter


On June 16, 2015, 3:09 a.m., Bernie Innocenti wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124108/
> ---
> 
> (Updated June 16, 2015, 3:09 a.m.)
> 
> 
> Review request for Plasma and Harald Sitter.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> ---
> 
> My system has 3 HDMI ports, and HDMI2 is listed in pavucontrol,
> but not in kcm_phonon. The reason was that HDMI2 and HDMI3 had the
> same priority, and were overwriting each other in the cardInfo profiles
> map. The fix consists in switching to a multimap.
> 
> 
> Diffs
> -
> 
>   kcms/phonon/audiosetup.h dfb25bf8c111d72065a27bb91583026093f0598d 
> 
> Diff: https://git.reviewboard.kde.org/r/124108/diff/
> 
> 
> Testing
> ---
> 
> Manually tested with:
>   QT_PLUGIN_PATH="$PWD:$QT_PLUGIN_PATH" kcmshell5 kcm_phonon
> 
> 
> Thanks,
> 
> Bernie Innocenti
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel