Re: Review Request for KDecoration

2014-12-11 Thread Albert Astals Cid
El Dijous, 11 de desembre de 2014, a les 13:15:55, Martin Gräßlin va escriure: > On Thursday 11 December 2014 00:05:52 Albert Astals Cid wrote: > > El Dimecres, 10 de desembre de 2014, a les 15:44:02, Thomas Lübking va > > > > escriure: > > > On Mittwoch, 10. Dezember 2014 13:11:07 CEST, Christoph

Re: Re: Review Request for KDecoration

2014-12-11 Thread Martin Gräßlin
On Thursday 11 December 2014 00:05:52 Albert Astals Cid wrote: > El Dimecres, 10 de desembre de 2014, a les 15:44:02, Thomas Lübking va > > escriure: > > On Mittwoch, 10. Dezember 2014 13:11:07 CEST, Christoph Feck wrote: > > > Every interactive on-screen element must have a text, either as a > >

Re: Review Request for KDecoration

2014-12-10 Thread Albert Astals Cid
El Dimecres, 10 de desembre de 2014, a les 15:44:02, Thomas Lübking va escriure: > On Mittwoch, 10. Dezember 2014 13:11:07 CEST, Christoph Feck wrote: > > Every interactive on-screen element must have a text, either as a > > label, or as a tooltip. This is a simple accessibility requirement. > >

Re: Review Request for KDecoration

2014-12-10 Thread Thomas Lübking
On Mittwoch, 10. Dezember 2014 13:11:07 CEST, Christoph Feck wrote: Every interactive on-screen element must have a text, either as a label, or as a tooltip. This is a simple accessibility requirement. Does this imply a formal representation (eg. to be accessed as object property from some sc

Re: Review Request for KDecoration

2014-12-10 Thread Christoph Feck
On Wednesday 10 December 2014 08:07:25 Martin Gräßlin wrote: > My personal opinion is that it doesn't need the tooltips. If one > cannot recognize the buttons than the design language of the > decoration is really bad. Every interactive on-screen element must have a text, either as a label, or as

Re: Re: Review Request for KDecoration

2014-12-09 Thread Martin Gräßlin
On Wednesday 10 December 2014 00:31:03 Thomas Lübking wrote: > On Mittwoch, 10. Dezember 2014 00:21:38 CEST, Albert Astals Cid wrote: > > Where did the strings such as > > > > "Not on all desktops" > > They were/are not part of KDecoration, but KCommonDecoration - an optional > "simplifying" wrap

Re: Re: Review Request for KDecoration

2014-12-09 Thread Martin Gräßlin
On Wednesday 10 December 2014 00:21:38 Albert Astals Cid wrote: > El Divendres, 28 de novembre de 2014, a les 15:30:23, Martin Gräßlin va > > escriure: > > On Friday 28 November 2014 15:25:03 Thomas Lübking wrote: > > > On Freitag, 28. November 2014 14:55:43 CEST, Martin Gräßlin wrote: > > > > Bot

Re: Review Request for KDecoration

2014-12-09 Thread Thomas Lübking
On Mittwoch, 10. Dezember 2014 00:21:38 CEST, Albert Astals Cid wrote: Where did the strings such as "Not on all desktops" They were/are not part of KDecoration, but KCommonDecoration - an optional "simplifying" wrapper class. I assume it would make sense to deliver pre-defined (and i18n'd)

Re: Review Request for KDecoration

2014-12-09 Thread Albert Astals Cid
El Divendres, 28 de novembre de 2014, a les 15:30:23, Martin Gräßlin va escriure: > On Friday 28 November 2014 15:25:03 Thomas Lübking wrote: > > On Freitag, 28. November 2014 14:55:43 CEST, Martin Gräßlin wrote: > > > Both signals are now removed. Works like a charm with the > > > handling direct

Re: Re: Review Request for KDecoration

2014-11-28 Thread Martin Gräßlin
On Friday 28 November 2014 15:25:03 Thomas Lübking wrote: > On Freitag, 28. November 2014 14:55:43 CEST, Martin Gräßlin wrote: > > Both signals are now removed. Works like a charm with the > > handling directly in KWin. > > I foresee trouble/complicfication on this (at least limitation) > > Assum

Re: Review Request for KDecoration

2014-11-28 Thread Thomas Lübking
On Freitag, 28. November 2014 14:55:43 CEST, Martin Gräßlin wrote: Both signals are now removed. Works like a charm with the handling directly in KWin. I foresee trouble/complicfication on this (at least limitation) Assume a decoration wants to alter the current tab on wheel, depending on the

Re: Re: Review Request for KDecoration

2014-11-28 Thread Martin Gräßlin
On Friday 28 November 2014 12:52:44 Christoph Feck wrote: > > > > Q_PROPERTY(QFont font > > > > > > When I want to compute pixel sizes for the requested font, how > > > do I get the correct QPaintDevice needed for the QFontMetrics? > > > Otherwise, please provide a QFontMetrics (see QStyleOption).

Re: Re: Re: Review Request for KDecoration

2014-11-28 Thread Martin Gräßlin
Martin GräßlinOn Friday 28 November 2014 12:00:43 wrote: > > >void titleBarDoubleClicked(); > > > > Maybe it might sense to have the event here, to check keyboard > > modifiers? > > > > >void titleBarWheelEvent(const QPoint &angleDelta); > > > > Why not simply pass the QWheelEvent? I mi

Re: Review Request for KDecoration

2014-11-28 Thread Sebastian Kügler
On Friday, November 28, 2014 14:16:38 Martin Gräßlin wrote: > On Friday 28 November 2014 13:53:07 Sebastian Kügler wrote: > > On Friday, November 28, 2014 12:00:43 Martin Gräßlin wrote: > > > > > > The documentation is copied from Plasma. I don't know anything about > > > high > > > DPI, so maybe

Re: Re: Review Request for KDecoration

2014-11-28 Thread Martin Gräßlin
On Friday 28 November 2014 13:53:07 Sebastian Kügler wrote: > On Friday, November 28, 2014 12:00:43 Martin Gräßlin wrote: > > > Given that you call it "fundamental", I suggest to allow qreal here. > > > A "millimeter" is really small, and if you only allow integer values, > > > the precision might

Re: Review Request for KDecoration

2014-11-28 Thread Sebastian Kügler
On Friday, November 28, 2014 12:00:43 Martin Gräßlin wrote: > > Given that you call it "fundamental", I suggest to allow qreal here. > > A "millimeter" is really small, and if you only allow integer values, > > the precision might be too coarse. > > The documentation is copied from Plasma. I don't

Re: Review Request for KDecoration

2014-11-28 Thread Christoph Feck
On Friday 28 November 2014 12:00:43 Martin Gräßlin wrote: > On Sunday 16 November 2014 23:30:35 Christoph Feck wrote: > > On Friday 31 October 2014 08:22:53 Martin Gräßlin wrote: > > > today I want to start the review process for the new > > > KDecoration > > > > Hi Martin, > > > > thanks for the

Re: Re: Review Request for KDecoration

2014-11-28 Thread Martin Gräßlin
On Sunday 16 November 2014 23:30:35 Christoph Feck wrote: > On Friday 31 October 2014 08:22:53 Martin Gräßlin wrote: > > today I want to start the review process for the new KDecoration > > Hi Martin, > > thanks for the work, here are some random comments from my side. I > hope I am not too late.

Re: Review Request for KDecoration

2014-11-16 Thread Christoph Feck
On Friday 31 October 2014 08:22:53 Martin Gräßlin wrote: > today I want to start the review process for the new KDecoration Hi Martin, thanks for the work, here are some random comments from my side. I hope I am not too late. * decoration.h > Rendering whether the Decoration is fully opaque.

Re: Re: Review Request for KDecoration

2014-11-04 Thread Martin Gräßlin
On Tuesday 04 November 2014 13:40:30 Milian Wolff wrote: > > I'd rather stress "notice that the return value can be nullptr and change > > during the lifetime of the DecoratedClient". > > > > The user can still do > > Decoration *m_deco = m_client->decoration(); // fails at some point > > > > Ove

Re: Review Request for KDecoration

2014-11-04 Thread Milian Wolff
On Tuesday 04 November 2014 11:53:55 Thomas Lübking wrote: > On Dienstag, 4. November 2014 10:20:18 CEST, Martin Gräßlin wrote: > >> DecorationSettings > >> onAllDesktopsAvailableChanged > >> -> remove the "on" prefix > > > > No, maybe the name is bad, but if the on is removed the meaning > > chan

Re: Review Request for KDecoration

2014-11-04 Thread Thomas Lübking
On Dienstag, 4. November 2014 10:20:18 CEST, Martin Gräßlin wrote: DecorationSettings onAllDesktopsAvailableChanged -> remove the "on" prefix No, maybe the name is bad, but if the on is removed the meaning changes. It's whether one can put DecoratedClients "on all desktops" (at lest two virtu

Re: Re: Review Request for KDecoration

2014-11-04 Thread Martin Gräßlin
On Monday 03 November 2014 23:33:15 Milian Wolff wrote: > On Friday 31 October 2014 08:22:53 Martin Gräßlin wrote: > > Hi KDE core developers, > > > > today I want to start the review process for the new KDecoration > > library[1]. This library is intended to replace the window decoration > > libr

Re: Re: Review Request for KDecoration

2014-11-03 Thread Martin Gräßlin
On Monday 03 November 2014 22:29:22 Albert Astals Cid wrote: > El Divendres, 31 d'octubre de 2014, a les 08:22:53, Martin Gräßlin va > > escriure: > > Hi KDE core developers, > > > > today I want to start the review process for the new KDecoration > > library[1]. This library is intended to repla

Re: Review Request for KDecoration

2014-11-03 Thread Milian Wolff
On Friday 31 October 2014 08:22:53 Martin Gräßlin wrote: > Hi KDE core developers, > > today I want to start the review process for the new KDecoration library[1]. > This library is intended to replace the window decoration library used by > KWin and addresses some of the shortcomings we hit with

Re: Review Request for KDecoration

2014-11-03 Thread Albert Astals Cid
El Divendres, 31 d'octubre de 2014, a les 08:22:53, Martin Gräßlin va escriure: > Hi KDE core developers, > > today I want to start the review process for the new KDecoration library[1]. > This library is intended to replace the window decoration library used by > KWin and addresses some of the s