extragear/utils/yakuake

2010-05-24 Thread Eike Hein
SVN commit 1130207 by hein: Both Yakuake and KRunner used to suffer from the following bug: If there are two monitors, the top edges of which are alined, and on one of those monitors (but not the other) is a panel at the top edge, their windows would appear at a distance away from the top edge equ

Re: IRC meeting summary

2012-09-12 Thread Eike Hein
ation will have it). Aye, we sorta almost touched on that when I mentioned longer-term plans for things like jump lists and recent docs and suggested it has some overlap with SLC which was confirmed by Aaron and Marco. -- Best regards, Eike Hein ___ Pl

Re: [RFC] Trigger KRunner on top screenedge

2012-10-04 Thread Eike Hein
of users. -- Best regards, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: [RFC] Trigger KRunner on top screenedge

2012-10-04 Thread Eike Hein
d code path for non-KDE users. Let me know what you come up with for KDE sessions. -- Best regards, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 109993: Sort Folder View preview popup by default if main view is unsorted

2013-04-13 Thread Eike Hein
- plasma/applets/folderview/popupview.cpp c6e881b Diff: http://git.reviewboard.kde.org/r/109993/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 109995: Tint selected icons in Folder View by highlight color (as Dolphin 2.0)

2013-04-13 Thread Eike Hein
h the KDE/4.10 and master branches. Diffs - plasma/applets/folderview/iconview.cpp 14c136e Diff: http://git.reviewboard.kde.org/r/109995/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org

Re: Review Request 109995: Tint selected icons in Folder View by highlight color (as Dolphin 2.0)

2013-04-13 Thread Eike Hein
dated) Before http://git.reviewboard.kde.org/media/uploaded/files/2013/04/13/before.png After http://git.reviewboard.kde.org/media/uploaded/files/2013/04/13/after.png Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org

Re: Review Request 109995: Tint selected icons in Folder View by highlight color (as Dolphin 2.0)

2013-04-13 Thread Eike Hein
09995/diff/ Testing --- File Attachments Before http://git.reviewboard.kde.org/media/uploaded/files/2013/04/13/before.png After http://git.reviewboard.kde.org/media/uploaded/files/2013/04/13/after.png Thanks, Eike Hein ___ P

Re: Review Request 109995: Tint selected icons in Folder View by highlight color (as Dolphin 2.0)

2013-04-15 Thread Eike Hein
sting --- File Attachments Before http://git.reviewboard.kde.org/media/uploaded/files/2013/04/13/before.png After http://git.reviewboard.kde.org/media/uploaded/files/2013/04/13/after.png Thanks, Eike Hein ___ Plasma-devel mailing

Re: Review Request 109993: Sort Folder View preview popup by default if main view is unsorted

2013-04-15 Thread Eike Hein
. I would like to apply this patch to both the KDE/4.10 and master branches. Diffs - plasma/applets/folderview/popupview.cpp c6e881b Diff: http://git.reviewboard.kde.org/r/109993/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel

Re: Review Request 109995: Tint selected icons in Folder View by highlight color (as Dolphin 2.0)

2013-04-15 Thread Eike Hein
s know what you decide to do. - Eike --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109995/#review31063 --- On Apri

Re: Menu branch

2013-04-17 Thread Eike Hein
Hi, the branch generally looks OK to me, however: - Display -> Icons in the config dialog is getting to be pretty messy; with that abundance of options it might make sense to split things into group boxes for Arrangement (flow, grid, lock) and Sorting (role, direction, folders first) to a

Re: Menu branch

2013-04-18 Thread Eike Hein
On Wednesday 17 April 2013 22:20:26 Ignat Semenov wrote: > - setCurrentItem() - which one did you like better, monolithic > setCurrentItem(), find() and then set() depending on find(), or the last > one with set() being unnecessary? For me, it's usually "code what to do not > how", but this time I

Re: Menu branch

2013-04-18 Thread Eike Hein
On Thursday 18 April 2013 00:20:06 Ignat Semenov wrote: > A UI draft: > > http://imagebin.org/254401 > > Maybe move "Icon size" to the "Arrangement" section? Looks better than before, if you ask me. I agree that "Icon size" should be moved to "Arrangement", and I'm wondering if "Layout:" should

Re: Menu branch

2013-04-20 Thread Eike Hein
On Friday 19 April 2013 11:32:48 Ignat Semenov wrote: > setLayout() and setAlignment() both trigger a complete relayout of the view. > Relayouting the view twice is suboptimal, however, in the code there is a > 10ms timer to delay the relayout so as not to block the caller. Should that > timer be e

Re: Menu branch

2013-04-20 Thread Eike Hein
Hi again, had a look at the diffs now. Looking good to me from the code side, but a couple of thoughts on the UI: - I think the split is working out as well as we had hoped, but the devil is in the details ... on the dialog page: - The repetition of "Sorting" between the group box title and

Re: Menu branch

2013-04-21 Thread Eike Hein
On Sunday 21 April 2013 12:30:20 Ignat Semenov wrote: > Done! Nice work. Two tiny bits left: - "Align Icons" could just be "Align" in the context menu (the parent menu name, "Icons", already provides this context). - I don't think we need a divider between the "Descending" and the "Folders F

Review Request 110158: Fix crash in Plasma::Applet::configDialogFinished() in the containment case

2013-04-24 Thread Eike Hein
. I would like to apply this patch to both the KDE/4.10 and master branches. This addresses bug 318528. http://bugs.kde.org/show_bug.cgi?id=318528 Diffs - plasma/applet.cpp 9ed62bc Diff: http://git.reviewboard.kde.org/r/110158/diff/ Testing --- Tha

Re: Review Request 110158: Fix crash in Plasma::Applet::configDialogFinished() in the containment case

2013-04-24 Thread Eike Hein
let.cpp 9ed62bc Diff: http://git.reviewboard.kde.org/r/110158/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 110158: Fix crash in Plasma::Applet::configDialogFinished() in the containment case

2013-04-24 Thread Eike Hein
y, visit: http://git.reviewboard.kde.org/r/110158/#review31507 --- On April 24, 2013, 6:21 p.m., Eike Hein wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http:/

Re: Review Request 110413: Show in the tasks applet the activities a task is available on

2013-05-13 Thread Eike Hein
at dilemma. In libplasma2 QML ToolTips are fortunately going to be much more flexible, ending the need for a fork. - Eike Hein On May 13, 2013, 11:10 a.m., José Millán Soto wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 110413: Show in the tasks applet the activities a task is available on

2013-05-13 Thread Eike Hein
> On May 13, 2013, 11:18 a.m., Eike Hein wrote: > > This isn't a call to reject this patch (which I think is a nice idea), but > > FWIW: The Task Manager widget is still intended to be replaced by a QML > > rewrite before feature freeze, and the QML bindings to Plasm

Re: Review Request 110413: Show in the tasks applet the activities a task is available on

2013-05-13 Thread Eike Hein
> On May 13, 2013, 11:18 a.m., Eike Hein wrote: > > This isn't a call to reject this patch (which I think is a nice idea), but > > FWIW: The Task Manager widget is still intended to be replaced by a QML > > rewrite before feature freeze, and the QML bindings to Plasm

Re: Review Request 110453: Order tasks by activity in tasks applet

2013-05-15 Thread Eike Hein
> On May 15, 2013, 5:36 p.m., José Millán Soto wrote: > > I'm aware that a new version of the tasks applet is being done in QML, but > > most of the changes are done in the library and not in the applet itself > > (the only change made to the applet is to add the ordering in the "Order > > by"

Re: Usability issue in the task bar applet

2013-05-21 Thread Eike Hein
FWIW, IMHO the real problem there is that the task manager uses the generic Disabled icon effect via KIconEffects, instead of using the "Panel" category. If it used the proper category, the user could disable this effect specifically only for the panel via Icons -> Advanced in System Settings. Unf

Re: Re: Usability issue in the task bar applet

2013-05-21 Thread Eike Hein
He's right though, this is entirely unrelated to using desktop effects or not, and that the icons of minimized icons get desaturized as of 4.10, with no good way to disable that behavior, is indeed quite unpopular over in #kde. The main reason being given is that users rely on telling windows apar

Re: Re: Re: Usability issue in the task bar applet

2013-05-21 Thread Eike Hein
> > good way to disable that behavior, is indeed quite unpopular over in > #kde. > it is relevant. If it works fine in the composited setup, then the problem > is > not that it's put to grayscale, but that the contrast in the non-composited > setup is too bad. Which would mean that should be fixed.

Re: Usability issue in the task bar applet

2013-05-21 Thread Eike Hein
> > imho this is not a good solution. the idea of "icons in the panel" is > broken > in a number of ways: the icons may not only be in the panel at all, there > is > no "the" panel (or put another way: panels may have very different > purposes), > etc. the icon handling reflects how things were a d

Re: Usability issue in the task bar applet

2013-05-21 Thread Eike Hein
> i'm fine with bringing back some colour to the icons to aid such people. > painting it with some alpha might work nicely to preserve both indication > of > minimization and preserving colour cues. Hm, I'll experiment a bit and see if there's a compromise variant that looks decent. _

Re: Usability issue in the task bar applet

2013-05-21 Thread Eike Hein
> > btw.. when can we merge your branch? we are up against the soft feature > freeze > for 4.11 now. I had been hoping to make soft freeze, but due to other commitments I've been delayed a bit further - and today there's more Folder View stuff to do as well, it seems. So while I'm getting nervou

Re: Usability issue in the task bar applet

2013-05-21 Thread Eike Hein
> having tried your replacement out, it already works reasonably well. i'd > like > to see it merged into master sooner rather than later so we can get the > other > developers at least testing it and so we can compile a list of showstopper > issues, if any actually exist. > I understand and actua

Re: Usability issue in the task bar applet

2013-05-21 Thread Eike Hein
> Where is your branch anyway? Or is it still private on your computer only? > ;) > No, otherwise Aaron couldn't have reported to have tried it. It's at clones/kde-workspace/hein/kde-workspacefor quite some time now.

Re: Usability issue in the task bar applet

2013-05-22 Thread Eike Hein
I just browsed through the QML and i must say, the QML looks > surprisingly simple and certainly not long for something as > complicated as the taskbar. Also, using a tooltip proxy is really > neat. I was struggling with the tooltip code before but gave up due to > various reasons. > Thanks, it's

QML Task Manager merged

2013-06-05 Thread Eike Hein
Hi, I've just merged the QML port of the default Task Manager widget into kde-workspace master, along with a number of (non-API-break- ing) modifications and bug fixes to libtaskmanager as required by it. tl;dr: It's mostly a drop-in replacement for the old one with a whole bunch of bug fixes, h

Re: QML Task Manager merged

2013-06-06 Thread Eike Hein
Afk right now but quick replies: - Thanks for having a look :) - Sizing/clipping: Aye, I decided to go with 'correct' handling/respecting of the FrameSvg initially rather than optimize prematurely and clutter up the QML, but I agree a little extra mile for common small panel sizes would be swee

Re: QML Task Manager merged

2013-06-06 Thread Eike Hein
On Thursday 06 June 2013 18:12:30 Marco Martin wrote: > On Thursday 06 June 2013, Aaron J. Seigo wrote: > > btw, i thought quite a bit about perhaps doing the same also in the main > > applet UI itself .. but with multiple rows it will likely look ugly; imo > > grids do not look great when the sele

Re: QML Task Manager merged

2013-06-06 Thread Eike Hein
On Thursday 06 June 2013 22:44:28 Aaron J. Seigo wrote: > btw, just noticed another regression: windows don't minimize to the tasks > widget anymore. This should generally work, and actually for the most part better than in the old widget, but there's a few more cases I need to handle for republis

Re: Review Request 111087: widget-taskbar: vertical text layout

2013-06-18 Thread Eike Hein
lex It's largely an import from the old C++ applet, so it's not surprising the problem persists. - Eike Hein On June 18, 2013, 11:44 a.m., Greg T wrote: > > --- > This is an automatically generated e-mail. T

Re: [homerun] Repeated Entries

2013-07-19 Thread Eike Hein
On Wednesday 17 July 2013 17:36:43 Jacky Alcine wrote: > Hey Plasma developers, Jacky here. > > I'm curious about how Homerun goes about rendering entries. I've noticed > constantly that entries shown in "Applications" would appear in "Favorite > Applications". How does Homerun go about populating

Re: Review Request 111618: Fix launcher regression, only show icons instead of icons + text

2013-07-20 Thread Eike Hein
her items: Try some launchers together with a two-row layout and marvel at the fallout. I'm still working on a proper solution. - Eike Hein On July 20, 2013, 2:58 p.m., Mark Gaiser wrote: > > --- > This is an automatical

Re: Review Request 111621: Increase size of label pixmap to account for margins and shadows

2013-07-21 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111621/#review36232 --- Ship it! Looks good in practice, thanks. - Eike Hein On

Re: kde-workspace 4.10.90 ABI breakage in libtaskmanager

2013-07-21 Thread Eike Hein
On Sunday 21 July 2013 16:10:02 Marco Martin wrote: > yes, would be good that it retains compatibility, so if it can be quickly > adapted it should (if)... > however, we never promised any abi compatibility in kde-workspace libraries > (headers are not even installed) Interestingly the two ABI ch

Review Request 111748: Better wrap QMimeData by retaining support for QByteArray

2013-07-27 Thread Eike Hein
eimports/draganddrop/DeclarativeMimeData.cpp e7cab10 Diff: http://git.reviewboard.kde.org/r/111748/diff/ Testing --- I haven't even built the thing. It's terribad. Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.k

Re: Review Request 111748: Better wrap QMimeData by retaining support for QByteArray

2013-07-27 Thread Eike Hein
567 Diffs (updated) - src/declarativeimports/draganddrop/DeclarativeMimeData.h ec725af src/declarativeimports/draganddrop/DeclarativeMimeData.cpp e7cab10 Diff: http://git.reviewboard.kde.org/r/111748/diff/ Testing --- I haven't even built the thing. It's

Re: Libtaskmanager ported to Qt5/KF5

2013-08-09 Thread Eike Hein
On Friday 09 August 2013 14:09:29 you wrote: > Hi all, > > on special request by Eike I looked into porting libtaskmanager to Qt5/KF5. > And after a surprisingly easy port I just pushed to frameworks-scratch > branch a commit which enables building of libtaskmanager again. Thanks :) Nice to hear

Re: [RFC] Port PlasmaComponents to use QtQuick Controls where applicable

2013-08-19 Thread Eike Hein
On Monday 19 August 2013 01:05:50 David Edmundson wrote: > I think given the current API changes it makes sense to use this > opportunity to make this change. I believe we will get better more stable > code for it. I can do most the changes. I'm with you on this, and I'm really relieved it turned

Re: [RFC] Port PlasmaComponents to use QtQuick Controls where applicable

2013-08-19 Thread Eike Hein
On Monday 19 August 2013 11:55:14 Marco Martin wrote: > for being able to do a decent desktop application (or even just the settings > dialogs) some things are still needed, in part feature missing, in part > making it play well with oxygen (and oxygen play well with them) since it's > a qstyle inf

Re: Review Request 112241: Fix "Show Launcher when not running" option in taskbar widget

2013-08-24 Thread Eike Hein
patch, but I really still have to find a way to reproduce this bug before just applying this blindly - it might be treating a symptom instead of addressing the root cause. - Eike Hein On Aug. 24, 2013, 2 p.m., Bhushan Shah wrote: > > --

Re: Review Request 112241: Fix "Show Launcher when not running" option in taskbar widget

2013-08-24 Thread Eike Hein
> On Aug. 24, 2013, 3:17 p.m., Eike Hein wrote: > > Hm, on the face of it, this patch doesn't really make sense ... launcher > > items don't have an associated task, so the function should already return > > early and the extra condition should be r

Re: Review Request 112241: Fix "Show Launcher when not running" option in taskbar widget

2013-08-25 Thread Eike Hein
> On Aug. 24, 2013, 3:17 p.m., Eike Hein wrote: > > Hm, on the face of it, this patch doesn't really make sense ... launcher > > items don't have an associated task, so the function should already return > > early and the extra condition should be r

Re: Review Request 112241: Fix "Show Launcher when not running" option in taskbar widget

2013-08-25 Thread Eike Hein
discarded. Review request for kde-workspace, Plasma and Eike Hein. Description --- Fix the crash in plasma-desktop caused by newer QML taskbar widget. Simple steps to reproduce this crash. 1) Pin any task/application to taskbar using "show launcher when not running" option

Re: Review Request 112375: Use a single frame width for all PM_DefaultFrameWidth

2013-08-30 Thread Eike Hein
> On Aug. 30, 2013, 12:43 p.m., Hugo Pereira Da Costa wrote: > > I respectfully disagree with the change. > > > > If I understand the descripion right, the only thing that matters for > > QtQuick is to change the default returned value (if all else fail) from 1 > > to FRAME_FRAMEWIDTH. Correct

Re: Review Request 112375: Use a single frame width for all PM_DefaultFrameWidth

2013-08-30 Thread Eike Hein
CT_LineEdit, > &frame, QSize(width, height)); > > > > Hugo Pereira Da Costa wrote: > @David, > reading this code, I understand > - that my "fears" above might be overstated. ;) > - that this is code you do not have control on, unless com

Re: Review Request 113124: MouseEventListener: Don't process event from children multiple times

2013-10-05 Thread Eike Hein
/mouseeventlistener.cpp <http://git.reviewboard.kde.org/r/113124/#comment30266> Not sure about the 'return true' - I think MouseEventListener is supposed to never accept events itself. Doesn't the dupe check itself suffice? - Eike Hein On Oct. 5, 2013, 9:01 p.m.,

Re: Review Request 113124: MouseEventListener: Don't process event from children multiple times.

2013-10-06 Thread Eike Hein
due to QGV's reuse of mouse events, but as it fixes a real world problem I think it's an improvement. - Eike Hein On Oct. 6, 2013, 7:36 p.m., Nicolás Alvarez wrote: > > --- > This is an automatically generated e-ma

Review Request 113443: Attempt to unbreak pre-configuring QML applets via desktop scripting

2013-10-25 Thread Eike Hein
be some value in a "good enough" too :). Diffs - libs/plasmagenericshell/scripting/applet.cpp 3c81f77 Diff: http://git.reviewboard.kde.org/r/113443/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 113447: Attempt to unbreak pre-configuring QML applets via desktop scripting, take 2

2013-10-26 Thread Eike Hein
: kdelibs Description --- Please see https://git.reviewboard.kde.org/r/113443/ for the earlier attempt and Aaron's review, which lead to this new take. Diffs - plasma/configloader.cpp 6e97cb9 Diff: http://git.reviewboard.kde.org/r/113447/diff/ Testing --- Thanks, Eike

Re: Review Request 113443: Attempt to unbreak pre-configuring QML applets via desktop scripting

2013-10-26 Thread Eike Hein
I am a bit pressed for time to ideally get this fixed for a release, so there'd be some value in a "good enough" too :). Diffs - libs/plasmagenericshell/scripting/applet.cpp 3c81f77 Diff: http://git.reviewboard.kde.org/r/113443/diff/ Testing --- Thanks, Eike

Re: Can we drop kcontrol/workspaceoptions?

2013-10-26 Thread Eike Hein
On Thursday 17 October 2013 19:51:24 Martin Graesslin wrote: > It offers the following features: > * switch between workspace type "Desktop" and "Netbook" > * configure dashboard to "show Desktop widgets" or "Show an independent > widget set" > * checkbox for "show informational tips" (whatever tha

Re: Review Request 113447: Attempt to unbreak pre-configuring QML applets via desktop scripting, take 2

2013-10-27 Thread Eike Hein
org/r/113447/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 113447: Attempt to unbreak pre-configuring QML applets via desktop scripting, take 2

2013-10-27 Thread Eike Hein
sting --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 113447: Attempt to unbreak pre-configuring QML applets via desktop scripting, take 2

2013-10-29 Thread Eike Hein
http://git.reviewboard.kde.org/r/113447/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 113659: system-monitor plasmoid: respect settings from js code

2013-11-05 Thread Eike Hein
> On Nov. 5, 2013, 7:39 p.m., Eike Hein wrote: > > This is a promising first start. Indeed, the System Monitor applet didn't > > implement configChanged(), causing scripting its configuration not to work, > > as we talked about on IRC. > > > > However, th

Re: Review Request 113659: system-monitor plasmoid: respect settings from js code

2013-11-05 Thread Eike Hein
, but in that event you need to call configChanged() on the child applets to have them reread their config. - Eike Hein On Nov. 5, 2013, 8:58 p.m., Sergei Lopatin wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 113659: system-monitor plasmoid: respect settings from js code

2013-11-05 Thread Eike Hein
s with the current approach at least. - Eike Hein On Nov. 5, 2013, 10:34 p.m., Sergei Lopatin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.review

Panel desktop scripting

2013-11-16 Thread Eike Hein
Hi, while creating a template for a 'sidebar' panel type, I've noticed that using desktop scripting to set the geometry of a vertical panel doesn't work quite right. Take this following example: var panel = new Panel; panel.location = 'right'; panel.length = 900; panel.height = 200; Given that

Re: Panel desktop scripting

2013-11-16 Thread Eike Hein
On Saturday 16 November 2013 20:10:13 Eike Hein wrote: > 1. Spinning the event loop in scripting's location >setter so the constraints update is guaranteed to >happen before control is returned to the script. >Dirty ... Just for clarity: Appending 'QCoreAppl

Re: Panel desktop scripting

2013-11-18 Thread Eike Hein
On Monday 18 November 2013 16:49:57 Aaron J. Seigo wrote: > call flushPendingConstraintsEvents > > this kind of use case is exactly why it is a public slot :) Thanks, I missed that bit of API :) I've added the flush to the implementation of setLocation() in the scripting wrapper, and committed t

Re: Panel desktop scripting

2013-11-19 Thread Eike Hein
On Tuesday 19 November 2013 08:38:54 Aaron J. Seigo wrote: > awesome :) > > the same fix is probably also needed in the Plasma2 desktop scripting. > > along with the additions for panel min/max size setting i made the other day > as well. I merged KDE/4.11 to master - if the desktop/shell code

Re: Looking around and thinking ahead

2013-12-12 Thread Eike Hein
On Thursday 12 December 2013 02:26:42 Sebastian Kügler wrote: > In plasma-shell, we have some construction sites as well. Kickoff still > feels quite unfinished (some would say broken ;)) I wrote a new menu implementation for KDE 4 recently: http://i.imgur.com/eiq6D5w.png http://i.imgur.com/h9Y5j

Re: Looking around and thinking ahead

2013-12-12 Thread Eike Hein
On Thursday 12 December 2013 15:29:00 Aaron J. Seigo wrote: > so i don’t think it makes much sense to toss N of them in a ring and discuss > relative merits. Ah yup, and that's the sort of reply I expected from this mailing list, which is why I wasn't going to do so until I was prodded. Plasma is

Re: Looking around and thinking ahead

2013-12-12 Thread Eike Hein
On Thursday 12 December 2013 17:46:13 Aaron J. Seigo wrote: > complicating this is that one can have multiple tasks widgets or docks, you > might be running the netbook containment (or not), etc. there’s a lot of > dynamic pieces there. Aye, "Add to Panel" for example tries to add to the panel con

Re: Plasma team social health

2014-01-04 Thread Eike Hein
On Friday 03 January 2014 16:29:11 Valorie Zimmerman wrote: > Disagreements always happen; what matters is how those disagreements > are dealt with. It seems to me that trust has broken down, and so the > 'thinking together' that has made the Plasma development team so > central to the KDE communit

Re: Systemtray breakout notes

2014-01-17 Thread Eike Hein
On Friday 17 January 2014 10:23:48 Martin Klapetek wrote: > On Thu, Jan 16, 2014 at 6:01 PM, Sebastian Kügler wrote: > > Some applications nest in the systemtray, and appear in both taskbar and > > systemtray. We'd like to merge their entries. Example: Kmail's taskbar > > item > > could show the n

Re: Notes from "KRunner Breakout"

2014-01-17 Thread Eike Hein
On Thursday 16 January 2014 21:15:02 Vishesh Handa wrote: > * Runner Library needs a certain amount of work > - Add item type capabilities to the results > - Add preview capabilities > - Refactor so that async runners do not need to spawn an event loop and > block Do you expect the API will stay i

Re: Notes from "Async and representation"

2014-01-17 Thread Eike Hein
General comment: Please note that these size hints aren't only there so the shell can decide which representation to pick, but at times also do things like: * Set useful default sizes (e.g. for popups from panels) *before* appearance. * Affect interaction behavior (e.g. lock down resizing in on

Re: Notes from "KRunner Breakout"

2014-01-17 Thread Eike Hein
On Friday 17 January 2014 18:47:09 Vishesh Handa wrote: > The only big change that I can see coming is the conversion of RunnerManager > into a model. For Homerun, it needs to be able to either: * Get individual runner's results as a model with low overhead. or * Be able to filter a single glob

Re: Notes from "Async and representation"

2014-01-17 Thread Eike Hein
On Saturday 18 January 2014 01:11:54 Mark Gaiser wrote: > I guess my arguments for going with the "Plasmoid" name rather then > "Applet" are just ignored then? Mark, I doubt Marco meant any disrespect; discussing the name just isn't really a priority while we're trying to work out the semantics. K

Re: Release schedule for Plasma Next

2014-01-29 Thread Eike Hein
On Wednesday 29 January 2014 13:52:43 Sebastian Kügler wrote: > What do you think? I'd like to propose: * Renaming Beta 1 to "Alpha", mostly to hedge our bets on expectation management. I realize we haven't been using Alpha for the KDE 4 SC schedules lately, but Plasma Next isn't in "always s

Re: Release schedule for Plasma Next

2014-01-29 Thread Eike Hein
On Wednesday 29 January 2014 15:53:21 Sebastian Kügler wrote: > Let's aim for one week, we can delay the Alpha if need be, but > realistically, we should only integrate things that are really done. Ways > to make the post- feature-freeze week not cause havoc: One week (rather than the three days i

Re: Release schedule for Plasma Next

2014-01-29 Thread Eike Hein
On Wednesday 29 January 2014 16:36:22 Martin Graesslin wrote: > Do we really need this one week in between? Just tag and release on the day > of feature freeze. It's the responsibility of all of us to not screw up. > Why add to the release schedule the possibility to screw up? We are giving > out t

Re: Release schedule for Plasma Next

2014-01-31 Thread Eike Hein
On Thursday 30 January 2014 18:23:41 Martin Graesslin wrote: > > Is Alpha unstable code? A release that contains known critical bugs? why > > do > > we release them? Are distros going to package Alphas? > > Is Beta unstable code? A release that contains known critical bugs? why do > > we release th

Re: Release schedule for Plasma Next

2014-01-31 Thread Eike Hein
On Friday 31 January 2014 15:57:19 Martin Gräßlin wrote: > Isn't it better to communicate the meaning through our release announcements > than through the implied meanings of words like "alpha" and "beta"? I just > read the wikipedia article [1] for it and from that I assume that everybody > in our

Re: Review Request 115536: Add window thumbnail to taskmanager

2014-02-07 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115536/#review49229 --- Ship it! Ship It! - Eike Hein On Feb. 7, 2014, 12:44 p.m

Re: Review Request 115536: Add window thumbnail to taskmanager

2014-02-07 Thread Eike Hein
take a look at it as part of that. Thanks! - Eike Hein On Feb. 7, 2014, 12:44 p.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.r

Re: Review Request 116024: Port IconItem to native QSGTexture

2014-02-24 Thread Eike Hein
I was hoping we'd get around to doing, and from what I can see the chosen approach seems sound. Yay! - Eike Hein On Feb. 24, 2014, 4:09 p.m., David Edmundson wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 117627: Let the default taskbar setting for "Show only from current screen" be false

2014-04-22 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117627/#review56181 --- Ship it! Ship It! - Eike Hein On April 19, 2014, 12:40

RFC: Additional hint elements for frame theme SVGs

2014-04-24 Thread Eike Hein
Hi, I think we have a conceptual problem in the theming system that needs addressing, specifically related to frame elements that can have borders partially clipped away, such as dialogs and panels. Let's recap the current state of things: Frame SVGs are so-called nine- patches, i.e. there's an

Re: RFC: Additional hint elements for frame theme SVGs

2014-04-24 Thread Eike Hein
On Friday 25 April 2014 01:40:11 Eike Hein wrote: > - Much more importantly, the applet can't actually know what margin > is correct, because it doesn't know anything about the design of > This is a critical point: The width of the border row > or column as the di

Re: Which repos?

2014-04-25 Thread Eike Hein
On Friday 25 April 2014 08:52:53 Sebastian Kügler wrote: > Hey all, > > Since I was absent for a bit, when the split happened, which repositories > did we end up with for what was kde-runtime and kde-workspace? > workspace ├── kde-cli-tools ├── khelpcenter ├── khotkeys ├── kinfocenter ├── kio-ex

Re: Which repos?

2014-04-25 Thread Eike Hein
These dependency listings might also help: kde/workspace/plasma-workspace: frameworks/kdoctools kde/workspace/plasma-workspace: frameworks/krunner kde/workspace/plasma-workspace: frameworks/kjsembed kde/workspace/plasma-workspace: frameworks/knotifyconfig kde/workspace/plasma-workspace: framework

Review Request 117891: Fix x/y property values in DeclarativeDragDropEvent

2014-04-29 Thread Eike Hein
QPoint(0,0) in the scene. Diffs - src/qmlcontrols/draganddrop/DeclarativeDragDropEvent.cpp 374a09b Diff: https://git.reviewboard.kde.org/r/117891/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde

Re: Review Request 117891: Fix x/y property values in DeclarativeDragDropEvent

2014-04-29 Thread Eike Hein
------ Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 117891: Fix x/y property values in DeclarativeDragDropEvent

2014-04-30 Thread Eike Hein
x27;s reference frame - and actually causes them to be wrong when a DeclarativeDropArea isn't at QPoint(0,0) in the scene. Diffs - src/qmlcontrols/draganddrop/DeclarativeDragDropEvent.cpp 374a09b Diff: https://git.reviewboard.kde.org/r/117891/diff/ Testing --- Thanks,

Review Request 117897: Don't hide Dialog with hideOnWindowDeactivate==true on focus out when dialog is ancestor of new focus window

2014-04-30 Thread Eike Hein
lasmaquick/dialog.cpp 58de55b Diff: https://git.reviewboard.kde.org/r/117897/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 117897: Don't hide Dialog with hideOnWindowDeactivate==true on focus out when dialog is ancestor of new focus window

2014-04-30 Thread Eike Hein
d) - src/plasmaquick/dialog.cpp 58de55b Diff: https://git.reviewboard.kde.org/r/117897/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 117897: Don't hide Dialog with hideOnWindowDeactivate==true on focus out when dialog is ancestor of new focus window

2014-04-30 Thread Eike Hein
ldn't actually be hidden, simplifying applet code. Diffs (updated) - src/plasmaquick/dialog.cpp 58de55b Diff: https://git.reviewboard.kde.org/r/117897/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-dev

Re: Review Request 117897: Don't hide Dialog with hideOnWindowDeactivate==true on focus out when dialog is ancestor of new focus window

2014-04-30 Thread Eike Hein
, the dialog shouldn't actually be hidden, simplifying applet code. Diffs - src/plasmaquick/dialog.cpp 58de55b Diff: https://git.reviewboard.kde.org/r/117897/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Pl

Re: Plasma 5?

2014-05-05 Thread Eike Hein
On Monday 05 May 2014 15:55:27 Sebastian Kügler wrote: > Hi all, > > I am not happy with the 2014.6 name and naming scheme. There I said it. I'm not either. I think that going with the 5 scheme: (a) Provides a nice continuity for users, packagers and developers, who don't need to get used t

Review Request 118024: Insert new Icon applets before the applet at the panel's center, if any

2014-05-06 Thread Eike Hein
before the last spacer or at the end of the layout. Diffs - containments/panel/contents/ui/main.qml 3dd531f Diff: https://git.reviewboard.kde.org/r/118024/diff/ Testing --- Thanks, Eike Hein ___ Plasma-devel mailing list Plasma-dev

  1   2   3   4   5   6   7   8   9   10   >