Re: [kde-promo] Plasma 5.1.1 announcement

[Albert Astals Cid]

El Dimecres, 26 de novembre de 2014, a les 07:01:09, Ömer Fadıl USTA va 
escriure:
> Link has a typo. "Plasma 5.1
> " link is showing to :
> http://kde.org/announcements/plasma5.1/index.php
> on the other hand it must be :
> http://kde.org/announcements/plasma-5.1/index.php

Oh man, what a rookie mistake :/

I fixed it this morning thanks to your email :)

Cheers,
  Albert

> 
> (The - is missing between plasma and 5.1 )
> 
> [image: Ömer Fadıl Usta on about.me]
> 
> Ömer Fadıl Usta
> about.me/omerusta
>   
> 
> 2014-11-25 23:49 GMT+02:00 Albert Astals Cid :
> > El Dimecres, 19 de novembre de 2014, a les 10:50:55, Ben Cooksley va
> > 
> > escriure:
> > > On Wed, Nov 19, 2014 at 8:14 AM, Albert Astals Cid 
> > 
> > wrote:
> > > > El Dimarts, 11 de novembre de 2014, a les 22:53:58, Albert Astals Cid
> > 
> > va
> > 
> > > > escriure:
> > > >> Hi guys, is there any reason
> > > >> https://www.kde.org/announcements/plasma-5.1.1.php
> > > >> 
> > > >> says "Plasma 5 was released in July" and links to
> > > >> https://www.kde.org/announcements/plasma5.0/index.php
> > > >> 
> > > >> instead of saying "Plasma 5.1 was released in October" and link to
> > > >> https://www.kde.org/announcements/plasma-5.1/index.php
> > > >> 
> > > >> ?
> > > > 
> > > > Was the announcement written by a ghost?
> > > > 
> > > > Or is the person that wrote the announcement for Plasma 5.1.1 not
> > > > subscribed to neither kde-promo nor plasma-devel?
> > > > 
> > > > Anyway, if noone oposes i'll fix it to refer Plasma 5.1 instead of
> > 
> > Plasma
> > 
> > > > 5.0.
> > > 
> > > I would recommend going ahead and making the change,
> > 
> > Done.
> > 
> > Albert
> > 
> > > I already had to
> > > fix the download urls for 5.1.1 which were broken.
> > > 
> > > > Cheers,
> > > > 
> > > >   Albert
> > > 
> > > Thanks,
> > > Ben
> > > 
> > > >> Cheers,
> > > >> 
> > > >>   Albert
> > > >> 
> > > >> ___
> > > >> This message is from the kde-promo mailing list.
> > > >> 
> > > >> Visit https://mail.kde.org/mailman/listinfo/kde-promo to
> > 
> > unsubscribe, set
> > 
> > > >> digest on or temporarily stop your subscription.
> > > > 
> > > > ___
> > > > This message is from the kde-promo mailing list.
> > > > 
> > > > Visit https://mail.kde.org/mailman/listinfo/kde-promo to unsubscribe,
> > 
> > set
> > 
> > > > digest on or temporarily stop your subscription.
> > > 
> > > ___
> > > This message is from the kde-promo mailing list.
> > > 
> > > Visit https://mail.kde.org/mailman/listinfo/kde-promo to unsubscribe,
> > 
> > set
> > 
> > > digest on or temporarily stop your subscription.
> > 
> > ___
> > Plasma-devel mailing list
> > Plasma-devel@kde.org
> > https://mail.kde.org/mailman/listinfo/plasma-devel

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

Re: Review Request 121259: [oxygen] fix kde4 build/configure/cmake stage

[Hugo Pereira Da Costa]

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

Ship it!


Ship It!

- Hugo Pereira Da Costa


On Nov. 26, 2014, 7:43 p.m., Hrvoje Senjan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121259/
> ---
> 
> (Updated Nov. 26, 2014, 7:43 p.m.)
> 
> 
> Review request for Plasma and Hugo Pereira Da Costa.
> 
> 
> Repository: oxygen
> 
> 
> Description
> ---
> 
> it's used unconditionally since r121083
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt b7cfc8e 
> 
> Diff: https://git.reviewboard.kde.org/r/121259/diff/
> 
> 
> Testing
> ---
> 
> KF5 and kde4 builds succeed
> 
> 
> Thanks,
> 
> Hrvoje Senjan
> 
>

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

Re: Review Request 121259: [oxygen] fix kde4 build/configure/cmake stage

[Hugo Pereira Da Costa]

> On Nov. 26, 2014, 8:19 p.m., Hugo Pereira Da Costa wrote:
> > Ship It!

oops, too late.
Thanks for the fix !


- Hugo


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


On Nov. 26, 2014, 7:43 p.m., Hrvoje Senjan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121259/
> ---
> 
> (Updated Nov. 26, 2014, 7:43 p.m.)
> 
> 
> Review request for Plasma and Hugo Pereira Da Costa.
> 
> 
> Repository: oxygen
> 
> 
> Description
> ---
> 
> it's used unconditionally since r121083
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt b7cfc8e 
> 
> Diff: https://git.reviewboard.kde.org/r/121259/diff/
> 
> 
> Testing
> ---
> 
> KF5 and kde4 builds succeed
> 
> 
> Thanks,
> 
> Hrvoje Senjan
> 
>

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

Re: Review Request 121259: [oxygen] fix kde4 build/configure/cmake stage

[Hrvoje Senjan]

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

(Updated Nov. 26, 2014, 7:43 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma and Hugo Pereira Da Costa.


Repository: oxygen


Description
---

it's used unconditionally since r121083


Diffs
-

  CMakeLists.txt b7cfc8e 

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


Testing
---

KF5 and kde4 builds succeed


Thanks,

Hrvoje Senjan

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

Re: Review Request 121259: [oxygen] fix kde4 build/configure/cmake stage

[David Edmundson]

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

Ship it!


Ship It!

- David Edmundson


On Nov. 26, 2014, 6:38 p.m., Hrvoje Senjan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121259/
> ---
> 
> (Updated Nov. 26, 2014, 6:38 p.m.)
> 
> 
> Review request for Plasma and Hugo Pereira Da Costa.
> 
> 
> Repository: oxygen
> 
> 
> Description
> ---
> 
> it's used unconditionally since r121083
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt b7cfc8e 
> 
> Diff: https://git.reviewboard.kde.org/r/121259/diff/
> 
> 
> Testing
> ---
> 
> KF5 and kde4 builds succeed
> 
> 
> Thanks,
> 
> Hrvoje Senjan
> 
>

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

Re: oxygen with USE_KDE4

[šumski]

On Wednesday 26 of November 2014 17:33:46 Jonathan Riddell wrote:
> oxygen is currently broken when compiled with USE_KDE4

https://git.reviewboard.kde.org/r/121259/
should fix it...


Cheers,
Hrvoje


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


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 120777: Plasma Active: Initial commit for Baloo Cloud Component

[Marco Martin]

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

Ship it!


looks better to me

- Marco Martin


On Nov. 26, 2014, 6:11 p.m., Antonis Tsiapaliokas wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120777/
> ---
> 
> (Updated Nov. 26, 2014, 6:11 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-mobile
> 
> 
> Description
> ---
> 
> At the moment, Baloo doesn't provide a timeline, which is something that we 
> need for the activefilebrowser.
> So this new component, is introducing support for the timeline.
> 
> Notes
> ===
> 
> * Baloocloud component contains the org.kde.baloo component inside it.The 
> reason behind that, is that the implementation for the timeline is kind of 
> terible because of its perfomance.
> * I have put the new component inside the plasma-mobile repository, for the 
> above reason. But if the Baloo team, wants it inside the baloo repo then i 
> can move it. I am fine with both approaches (keep it here or in the baloo 
> repository.
> * If someone has a better idea about the implementation, the pls shoot :)
>   
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt 9466447 
>   components/CMakeLists.txt 536b60e 
>   components/timelinemodel/CMakeLists.txt PRE-CREATION 
>   components/timelinemodel/qmldir PRE-CREATION 
>   components/timelinemodel/timelinemodel.h PRE-CREATION 
>   components/timelinemodel/timelinemodel.cpp PRE-CREATION 
>   components/timelinemodel/timelineplugin.h PRE-CREATION 
>   components/timelinemodel/timelineplugin.cpp PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/120777/diff/
> 
> 
> Testing
> ---
> 
> Everything looks ok. The performance is not bad, except from the fact that 
> the implementation is a bit of hackish...
> 
> 
> Thanks,
> 
> Antonis Tsiapaliokas
> 
>

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

Review Request 121259: [oxygen] fix kde4 build/configure/cmake stage

[Hrvoje Senjan]

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

Review request for Plasma and Hugo Pereira Da Costa.


Repository: oxygen


Description
---

it's used unconditionally since r121083


Diffs
-

  CMakeLists.txt b7cfc8e 

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


Testing
---

KF5 and kde4 builds succeed


Thanks,

Hrvoje Senjan

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

Re: Review Request 120777: Plasma Active: Initial commit for Baloo Cloud Component

[Antonis Tsiapaliokas]

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

(Updated Nov. 26, 2014, 6:11 p.m.)


Review request for Plasma.


Changes
---

I have fix all the issues.


Repository: plasma-mobile


Description
---

At the moment, Baloo doesn't provide a timeline, which is something that we 
need for the activefilebrowser.
So this new component, is introducing support for the timeline.

Notes
===

* Baloocloud component contains the org.kde.baloo component inside it.The 
reason behind that, is that the implementation for the timeline is kind of 
terible because of its perfomance.
* I have put the new component inside the plasma-mobile repository, for the 
above reason. But if the Baloo team, wants it inside the baloo repo then i can 
move it. I am fine with both approaches (keep it here or in the baloo 
repository.
* If someone has a better idea about the implementation, the pls shoot :)
  


Diffs (updated)
-

  CMakeLists.txt 9466447 
  components/CMakeLists.txt 536b60e 
  components/timelinemodel/CMakeLists.txt PRE-CREATION 
  components/timelinemodel/qmldir PRE-CREATION 
  components/timelinemodel/timelinemodel.h PRE-CREATION 
  components/timelinemodel/timelinemodel.cpp PRE-CREATION 
  components/timelinemodel/timelineplugin.h PRE-CREATION 
  components/timelinemodel/timelineplugin.cpp PRE-CREATION 

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


Testing
---

Everything looks ok. The performance is not bad, except from the fact that the 
implementation is a bit of hackish...


Thanks,

Antonis Tsiapaliokas

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

[kio-extras] [Bug 339792] Don't install libmolletnetwork.so symlink

[Antonio Rojas]

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

Antonio Rojas  changed:

   What|Removed |Added

   Version Fixed In|5.1.2   |5.2

-- 
You are receiving this mail because:
You are the assignee for the bug.


Make the world a better place. Donate to our year end fundraiser 
https://www.kde.org/fundraisers/yearend2014/
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

[kio-extras] [Bug 339792] Don't install libmolletnetwork.so symlink

[Andrea Scarpino]

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

Andrea Scarpino  changed:

   What|Removed |Added

  Latest Commit||9e44dfa16facbad57b1b13737ba
   ||0beda6ab083a8
 CC||scarp...@kde.org
   Version Fixed In||5.1.2

-- 
You are receiving this mail because:
You are the assignee for the bug.


Make the world a better place. Donate to our year end fundraiser 
https://www.kde.org/fundraisers/yearend2014/
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

[kio-extras] [Bug 339792] Don't install libmolletnetwork.so symlink

[Antonio Rojas]

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

Antonio Rojas  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

-- 
You are receiving this mail because:
You are the assignee for the bug.


Make the world a better place. Donate to our year end fundraiser 
https://www.kde.org/fundraisers/yearend2014/
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

oxygen with USE_KDE4

[Jonathan Riddell]

oxygen is currently broken when compiled with USE_KDE4

this patch seems to be the problem
 https://git.reviewboard.kde.org/r/121083/

I've submitted a request to have this build as part of KDE CI
"CI for breeze and oxygen Qt4 "
https://sysadmin.kde.org/tickets/index.php?page=tickets&act=view&id=QZY-2988

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

Re: Stress testing KWin's screen handling

[Luca Beltrame]

In data martedì 25 novembre 2014 13:17:28, Martin Gräßlin ha scritto:

> I hope that it doesn't fail for others and we can get the changes in to
> improve the situation.

I tested this today with a projector: unplugging it from my laptop did not 
cause rendering to go haywire like it used to do. 

-- 
Luca Beltrame - KDE Forums team
KDE Science supporter
GPG key ID: 6E1A4E79

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

[Breeze] [Bug 341291] Issues with vertical spacing between items in tree views

[Martin Klapetek]

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

--- Comment #3 from Martin Klapetek  ---
...was a *wrong* bug number, not first...eheh.

-- 
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

[Breeze] [Bug 341291] Issues with vertical spacing between items in tree views

[Martin Klapetek]

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

--- Comment #2 from Martin Klapetek  ---
The first duplicate was a first bug number, sorry :)

*** This bug has been marked as a duplicate of bug 340999 ***

-- 
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

[Breeze] [Bug 341291] Issues with vertical spacing between items in tree views

[Martin Klapetek]

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

Martin Klapetek  changed:

   What|Removed |Added

 CC||mklape...@kde.org
 Resolution|--- |DUPLICATE
 Status|UNCONFIRMED |RESOLVED

--- Comment #1 from Martin Klapetek  ---


*** This bug has been marked as a duplicate of bug 340623 ***

-- 
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

[Breeze] [Bug 340623] Breeze Widget style is very boxy and spacious. Takes too much space.

[Martin Klapetek]

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

Martin Klapetek  changed:

   What|Removed |Added

 CC||kf...@kde.org

--- Comment #4 from Martin Klapetek  ---
*** Bug 341291 has been marked as a duplicate of this bug. ***

-- 
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

[Breeze] [Bug 341291] New: Issues with vertical spacing between items in tree views

[Kevin Funk]

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

Bug ID: 341291
   Summary: Issues with vertical spacing between items in tree
views
   Product: Breeze
   Version: unspecified
  Platform: Other
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: plasma-devel@kde.org
  Reporter: kf...@kde.org

See attached screenshot.

IMO, the vertical spacing between items in the left-handed tree view is too
much. It's wasting vertical screen estate.

The vertical spacing in the right-handed tree view is fine. 

Findings:
- This is a Qt4-based application (KMail)
- This also seem to happen for Qt5-based applications
- This does not only happen in KMail -- The chat/nick lists in Quassel
(Qt4-version) show the same symptoms

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: Stress testing KWin's screen handling

[Daniel Vrátil]

On Wednesday 26 of November 2014 11:48:28 Martin Gräßlin wrote:
> On Wednesday 26 November 2014 11:39:18 Daniel Vrátil wrote:
> > > ok, that still sounds like a rendering error. A few questions:
> > > * does qdus.org.kde.KWin /KWin supportInformation report correct screen
> > > information?
> > 
> > Yes
> > 
> > > * does xrandr report correct screen information?
> > 
> > Yes
> > 
> > > * does restarting compositing fix it?
> > 
> > Yes
> 
> good, so we have the correct information just render in a wrong way. That
> significantly reduces the problem scope.
> 
> > > For three screens I'm completely out of testing possibilities. I don't
> > > have
> > > three screens and even if I had I would not be able to connect them.
> > 
> > I managed to reproduce this when I unplugged the 3rd screen, so I don't
> > think this is strictly related to having ridiculous amount of screens, but
> > should be reproducible with two screens too.
> > 
> > If there's any debug info I can provide, just let me know. I really want
> > this fixed, but KWin internals are beyond my understanding :-)
> 
> How was the layout of the screens and which one did you unplug (e.g. right
> most, middle or left most)?

There were three screens next to each other, I unplugged the right-most one. 

I now realize that I can reliably reproduce this by placing a maximized window 
(like KMail) on that screen. Once unplugged, the KMail window (the actual 
window with content) is moved one screen left, to the one that is now the 
right-most one, while KMail window decorations are moved all the way to the 
left-most screen.

> Are you using the OpenGL or XRender compositor
> (just asking as the xrender could still be wrong as I didn't adjust it)?

I'm using OpenGL 2.0 with GLX interface.

> 
> Cheers
> Martin

-- 
Daniel Vrátil | dvra...@redhat.com | dvratil on #kde-devel, #kontact, #akonadi
Software Engineer - KDE Desktop Team, Red Hat Inc.

GPG Key: 0xC59D614F6F4AE348
Fingerprint: 4EC1 86E3 C54E 0B39 5FDD B5FB C59D 614F 6F4A E348

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 121253: Fix password focus in lockscreen

[Sebastian Kügler]

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

(Updated Nov. 26, 2014, 12:45 p.m.)


Status
--

This change has been marked as submitted.


Review request for Plasma.


Repository: plasma-workspace


Description
---

Fix password focus in lockscreen

When navigating the listview with the keyboard, the focus of the
password field can get lost as it moves to the button. This patch
reclaims the focus whenever the password field becomes visible.


Diffs
-

  lookandfeel/contents/lockscreen/LockScreen.qml 
8a495ea208325ba3b2ef09efaef49515cc99830d 

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


Testing
---

Navigated lockscreen with keyboard and mouse. Focus is handled correctly.


Thanks,

Sebastian Kügler

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

Re: Re: Stress testing KWin's screen handling

[Martin Gräßlin]

On Wednesday 26 November 2014 11:39:18 Daniel Vrátil wrote:
> > ok, that still sounds like a rendering error. A few questions:
> > * does qdus.org.kde.KWin /KWin supportInformation report correct screen
> > information?
> 
> Yes
> 
> > * does xrandr report correct screen information?
> 
> Yes
> 
> > * does restarting compositing fix it?
> 
> Yes

good, so we have the correct information just render in a wrong way. That 
significantly reduces the problem scope.

> 
> > For three screens I'm completely out of testing possibilities. I don't
> > have
> > three screens and even if I had I would not be able to connect them.
> 
> I managed to reproduce this when I unplugged the 3rd screen, so I don't
> think this is strictly related to having ridiculous amount of screens, but
> should be reproducible with two screens too.
> 
> If there's any debug info I can provide, just let me know. I really want
> this fixed, but KWin internals are beyond my understanding :-)

How was the layout of the screens and which one did you unplug (e.g. right 
most, middle or left most)? Are you using the OpenGL or XRender compositor 
(just asking as the xrender could still be wrong as I didn't adjust it)?

Cheers
Martin

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Stress testing KWin's screen handling

[Daniel Vrátil]

On Wednesday 26 of November 2014 08:19:26 Martin Gräßlin wrote:
> On Tuesday 25 November 2014 16:28:16 Daniel Vrátil wrote:
> > On Tuesday 25 of November 2014 13:17:28 Martin Gräßlin wrote:
> > > Hi all,
> > > 
> > > I spent some time on screen management in KWin today and got it to the
> > > point where it doesn't fail any more no matter what I try. So please
> > > everyone using multiple screens and especially dynamically plug in and
> > > out, please give a try to the patch set in [1]. Please ensure to have
> > > latest master as it contains a crash fix for a crash triggered by the
> > > patch set.
> > > 
> > > Short summary of the changes in the patch set:
> > > * uses XRandR instead of QDesktopWidget
> > > * uses KWin internal information about overall screen geometry instead
> > > of
> > > relying on the information in the X11 screen structure.
> > > 
> > > The second part is the code I added today. My testing showed that
> > > unplugging a screen gives us proper XRandR events so KWin's internal is
> > > up to date, but the X11 screen information is wrong. So when we
> > > partially
> > > used the one and partially the other the rendering was just horribly
> > > broken. Now it's all based on the KWin internal information and I
> > > couldn't get the rendering broken any more.
> > > 
> > > When changing screens please be patient. It takes time to settle the
> > > changes. Especially plasmashell takes quite some time on my system to
> > > render correctly again.
> > 
> > Coincidentally, I just merged my KScreen redesign, which should make this
> > faster.
> 
> sounds like I need to trigger kdesrc-build ;-)
> 
> > > I hope that it doesn't fail for others and we can get the changes in to
> > > improve the situation.
> > 
> > So far it's much better than before, but still it sometimes happens, that
> > after screen reshuffle, window decorations get detached from the windows
> > and moved elsewhere. It just happened to me, after plugging in the 3rd
> > screen: http://pub.dvratil.cz/kwin-bug.ogv, but I'm not able to reliably
> > reproduce this.
> 
> ok, that still sounds like a rendering error. A few questions:
> * does qdus.org.kde.KWin /KWin supportInformation report correct screen
> information?

Yes

> * does xrandr report correct screen information?

Yes

> * does restarting compositing fix it?

Yes

> 
> For three screens I'm completely out of testing possibilities. I don't have
> three screens and even if I had I would not be able to connect them.

I managed to reproduce this when I unplugged the 3rd screen, so I don't think 
this is strictly related to having ridiculous amount of screens, but should be 
reproducible with two screens too.

If there's any debug info I can provide, just let me know. I really want this 
fixed, but KWin internals are beyond my understanding :-)

> A kingdom, a kingdom for unit testing xrandr.

Amen to that


Dan

> 
> Cheers
> Martin

-- 
Daniel Vrátil | dvra...@redhat.com | dvratil on #kde-devel, #kontact, #akonadi
Software Engineer - KDE Desktop Team, Red Hat Inc.

GPG Key: 0xC59D614F6F4AE348
Fingerprint: 4EC1 86E3 C54E 0B39 5FDD B5FB C59D 614F 6F4A E348

signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 121240: Port to new KScreen API

[Luca Beltrame]

Il 26 novembre 2014 10:44:24 CET, Marco Martin  ha scritto:

>here (after such commit tough) i can't reproduce the crash with neither
>one or 
>two screens

Works also  for  me now.
-- 
Luca Beltrame - KDE Forums team
GPG key ID: 6E1A4E79
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 121240: Port to new KScreen API

[Marco Martin]

On Wednesday 26 November 2014, Aleix Pol wrote:
> > ___
> > Plasma-devel mailing list
> > Plasma-devel@kde.org
> > https://mail.kde.org/mailman/listinfo/plasma-devel
> 
> Can you check if you still get this crash now?
> 
> http://commits.kde.org/plasma-workspace/55bb013376c8688b74b5401587289b662fc
> 5315b

here (after such commit tough) i can't reproduce the crash with neither one or 
two screens

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