Re: Thoughts about statusbar

2011-10-25 Thread David Edmundson
2011/10/25 Aurélien Gâteau : > Le 25/10/2011 16:40, Martin Gräßlin a écrit : >> >> On Tuesday 25 October 2011 14:14:03 Aurélien Gâteau wrote: And yes this is clearly Workspace and not application related. We as the workspace define how an application has to look like and how it has t

Re: Review Request: Changed the wallpaper virus config. dialog so it looks like the 'image' configuration.

2011-10-25 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102930/#review7647 --- This review has been submitted with commit 3b7387f694aeea940c20

Re: Thoughts about statusbar

2011-10-25 Thread Aurélien Gâteau
Le 25/10/2011 16:40, Martin Gräßlin a écrit : On Tuesday 25 October 2011 14:14:03 Aurélien Gâteau wrote: And yes this is clearly Workspace and not application related. We as the workspace define how an application has to look like and how it has to behave. The statusbar is inside the applicatio

Re: Thoughts about statusbar

2011-10-25 Thread Aurélien Gâteau
Le 25/10/2011 16:32, Martin Gräßlin a écrit : On Tuesday 25 October 2011 20:41:23 David Edmundson wrote: KDE Usability is the mailing list I had in my mind when I wrote it. I didn't mean do distract from the focus of a thread I think is quite important. Interesting that two devs mentioned the u

Re: Thoughts about statusbar

2011-10-25 Thread Aurélien Gâteau
Le 25/10/2011 15:41, David Edmundson a écrit : Anyway, given I've decided I should pay more attention to plasma and get away from just hacking constantly on one project, to make up for it here is a breakdown of statusbars the most commonly used apps in KDE. This was made with a fresh account with

Re: Re: Thoughts about statusbar

2011-10-25 Thread Martin Gräßlin
On Tuesday 25 October 2011 14:14:03 Aurélien Gâteau wrote: > > And yes this is clearly Workspace and not application related. We as the > > workspace define how an application has to look like and how it has to > > behave. > The statusbar is inside the application window, so I disagree this is a >

Re: Re: Thoughts about statusbar

2011-10-25 Thread Martin Gräßlin
On Tuesday 25 October 2011 20:41:23 David Edmundson wrote: > KDE Usability is the mailing list I had in my mind when I wrote it. I > didn't mean do distract from the focus of a thread I think is quite > important. Interesting that two devs mentioned the usability mailing list. I hope this is an in

Re: Thoughts about statusbar

2011-10-25 Thread todd rme
On Tue, Oct 25, 2011 at 9:41 PM, David Edmundson wrote: > 2011/10/25 Aurélien Gâteau : >> Le 25/10/2011 11:32, Martin Gräßlin a écrit : >>> >>> On Tuesday 25 October 2011 15:37:20 David Edmundson wrote: On Tue, Oct 25, 2011 at 3:35 PM, Alex Fiestas  wrote: > > On Tuesday, October

Re: QML device notifier

2011-10-25 Thread Marco Martin
On Tuesday 25 October 2011, Sebastian Kügler wrote: > > Nope, not if we don't want to depend on a development snapshot. PA3 is the > first version that can (without patching packages) depend on 4.8 :/ > > PA2 will be out before 4.8 ... hm, not sure what of those two things will cause more troubl

Re: QML device notifier

2011-10-25 Thread Sebastian Kügler
On Tuesday, October 25, 2011 18:54:34 Marco Martin wrote: > On Tuesday 25 October 2011, Aaron J. Seigo wrote: > > On Tuesday, October 25, 2011 16:14:39 Marco Martin wrote: > > > - scrollbar: for this it will depend from qtcomponents, they are in kde- > > > runtime master for now, will still really

Re: Thoughts about statusbar

2011-10-25 Thread David Edmundson
> KOrganizer: Has a statusbar. Seemingly not used. > Get rid of it. If it's used for errors KMessageBox would be better I meant KMessageWidget. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Thoughts about statusbar

2011-10-25 Thread David Edmundson
2011/10/25 Aurélien Gâteau : > Le 25/10/2011 11:32, Martin Gräßlin a écrit : >> >> On Tuesday 25 October 2011 15:37:20 David Edmundson wrote: >>> >>> On Tue, Oct 25, 2011 at 3:35 PM, Alex Fiestas  wrote: On Tuesday, October 25, 2011 11:57:33 AM Aaron J. Seigo wrote: > > yes; as we

Re: Thoughts about statusbar

2011-10-25 Thread Aurélien Gâteau
Le 25/10/2011 11:32, Martin Gräßlin a écrit : On Tuesday 25 October 2011 15:37:20 David Edmundson wrote: On Tue, Oct 25, 2011 at 3:35 PM, Alex Fiestas wrote: On Tuesday, October 25, 2011 11:57:33 AM Aaron J. Seigo wrote: yes; as well as sometimes just saying "that information is not worth it"

Re: QML device notifier

2011-10-25 Thread Aaron J. Seigo
On Tuesday, October 25, 2011 18:54:34 Marco Martin wrote: > yep, it is > the key question is, will be possible to use 4.8 for active two? for that i think it would be very good if we can, because then distros can more easily ship PA packages for their distros since our patches to 4.7 right now al

Re: QML device notifier

2011-10-25 Thread Marco Martin
On Tuesday 25 October 2011, Aaron J. Seigo wrote: > On Tuesday, October 25, 2011 16:14:39 Marco Martin wrote: > > - scrollbar: for this it will depend from qtcomponents, they are in kde- > > runtime master for now, will still really rather move them in their own > > repository (and for me, they are

Re: QML device notifier

2011-10-25 Thread Aaron J. Seigo
On Tuesday, October 25, 2011 16:14:39 Marco Martin wrote: > - scrollbar: for this it will depend from qtcomponents, they are in kde- > runtime master for now, will still really rather move them in their own > repository (and for me, they are 100% needed for active two) if they are in kde-runtime m

Re: Some help for recreating the Plasma Tool Box in QML

2011-10-25 Thread Aaron J. Seigo
On Tuesday, October 25, 2011 15:22:16 Mark wrote: > > kde-workspace/plasma/desktop/shell/toolbox/paneltoobox.* > > > > which really should be in desktop/toolboxes imo, and not in with the > > shell. > > i'll fix that right now and then they will be in: > > > > kde-workspace/plasma/desktop/toolbox

Re: Thoughts about statusbar

2011-10-25 Thread Aaron J. Seigo
On Tuesday, October 25, 2011 15:37:20 David Edmundson wrote: > Is plasma-devel really the right mailing list for something that > affects every app, and nothing in plasma? given that these discussions are not happening elsewhere, and that is the kind of thing that evidently the people here do car

Re: Review Request: Workaround Taskbar ghost entries bug

2011-10-25 Thread Aaron J. Seigo
On Tuesday, October 25, 2011 16:36:24 Alex Fiestas wrote: > Speaking of this bug, aseigo did you finish the debugging before you went on > vacation ? if so, what was the conclusion ? > > (I'm refering to the potential bug in Qt not emitting destroyed signal). before i left for vacation, i confirm

Re: Re: Re: Thoughts about statusbar

2011-10-25 Thread Martin Gräßlin
On Tuesday 25 October 2011 15:37:20 David Edmundson wrote: > On Tue, Oct 25, 2011 at 3:35 PM, Alex Fiestas wrote: > > On Tuesday, October 25, 2011 11:57:33 AM Aaron J. Seigo wrote: > >> yes; as well as sometimes just saying "that information is not worth it". > >> the kmail example is a great one.

Re: Re: Thoughts about statusbar

2011-10-25 Thread David Edmundson
On Tue, Oct 25, 2011 at 3:35 PM, Alex Fiestas wrote: > On Tuesday, October 25, 2011 11:57:33 AM Aaron J. Seigo wrote: >> yes; as well as sometimes just saying "that information is not worth it". >> the kmail example is a great one. i do not care what column and line i am >> on and spell checking s

Re: Re: Review Request: Workaround Taskbar ghost entries bug

2011-10-25 Thread Alex Fiestas
Speaking of this bug, aseigo did you finish the debugging before you went on vacation ? if so, what was the conclusion ? (I'm refering to the potential bug in Qt not emitting destroyed signal). ___ Plasma-devel mailing list Plasma-devel@kde.org https://

Re: Re: Thoughts about statusbar

2011-10-25 Thread Alex Fiestas
On Tuesday, October 25, 2011 11:57:33 AM Aaron J. Seigo wrote: > yes; as well as sometimes just saying "that information is not worth it". > the kmail example is a great one. i do not care what column and line i am > on and spell checking should be shown in the toolbar if at all. More conservative

QML device notifier

2011-10-25 Thread Marco Martin
Hi all, i fixed some of the things that were still missing in the device notifier, now should be basically indistinguisheable, pixel-by-pixel similar to the c++ one, modulo some details things that are still missing: - scrollbar: for this it will depend from qtcomponents, they are in kde- runtim

Re: Some help for recreating the Plasma Tool Box in QML

2011-10-25 Thread Mark
On Tue, Oct 25, 2011 at 3:07 PM, Aaron J. Seigo wrote: > On Tuesday, October 25, 2011 14:10:54 Mark wrote: > > - The tool box icon in the panel can be clicked -- where is that code > > hidden? -- > > kde-workspace/plasma/desktop/shell/toolbox/paneltoobox.* > > which really should be in desktop/to

Re: Review Request: Make the task manager items move in realtime when user manually arranges items

2011-10-25 Thread Aaron J. Seigo
> On Oct. 25, 2011, 12:48 p.m., Aaron J. Seigo wrote: > > plasma/desktop/applets/tasks/taskgroupitem.cpp, lines 1037-1038 > > > > > > i wonder what happens here when you drag an entry from one tasks widget > > to

Re: Review Request: Make the task manager items move in realtime when user manually arranges items

2011-10-25 Thread Shantanu Tushar Jha
> On Oct. 25, 2011, 12:48 p.m., Aaron J. Seigo wrote: > > plasma/desktop/applets/tasks/taskgroupitem.cpp, line 1039 > > > > > > watch the indentation level :) ouch, will fix that before merging. > On Oct. 25, 20

Re: Some help for recreating the Plasma Tool Box in QML

2011-10-25 Thread Aaron J. Seigo
On Tuesday, October 25, 2011 14:10:54 Mark wrote: > - The tool box icon in the panel can be clicked -- where is that code > hidden? -- kde-workspace/plasma/desktop/shell/toolbox/paneltoobox.* which really should be in desktop/toolboxes imo, and not in with the shell. i'll fix that right now and

Re: Review Request: Changed the wallpaper virus config. dialog so it looks like the 'image' configuration.

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102930/#review7605 --- Ship it! would be nice to eliminate the duplication here.. but

Re: Review Request: Fix ordinal suffix in plasmaclock configuration page for TTS

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102878/#review7604 --- a different approach to this problem can be found in this review

Re: Review Request: Update UI for settings of "Speak Time" in digital clock.

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102944/#review7603 --- Ship it! Ship It! - Aaron J. Seigo On Oct. 23, 2011, 1:11 a.

Re: Review Request: Make the task manager items move in realtime when user manually arranges items

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102833/#review7601 --- Ship it! nice improvement :) plasma/desktop/applets/tasks/tas

Re: Review Request: Add option to not show popup when devices are added

2011-10-25 Thread Aaron J. Seigo
> On Oct. 18, 2011, 2:47 p.m., Jacopo De Simoi wrote: > > I am not sure if I actually would allow to block the device notifier from > > popping up altogheter; > > If a user doesn't want the notifier to popup for a specific device she can > > simply hide it by right-clicking on the device item.

Re: Review Request: Add option to not show popup when devices are added

2011-10-25 Thread Aaron J. Seigo
> On Oct. 18, 2011, 2:47 p.m., Jacopo De Simoi wrote: > > I am not sure if I actually would allow to block the device notifier from > > popping up altogheter; > > If a user doesn't want the notifier to popup for a specific device she can > > simply hide it by right-clicking on the device item.

Re: Review Request: Device notifier popup did not hide after the intended timeout of 7.5 s.

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102824/#review7598 --- removing the "emit activate()" line is fine, but the rest should

Re: Some help for recreating the Plasma Tool Box in QML

2011-10-25 Thread Mark
On Tue, Oct 25, 2011 at 12:10 PM, Aaron J. Seigo wrote: > On Friday, October 21, 2011 13:38:49 Viranch Mehta wrote: > > Well, I'm not sure if it is possible to just change a part of the panel > (the > > tool box). You'd need to re-do the panel in qml, afaik. > > the toolbox _icon_ itself is a ver

Re: Review Request: picture frame widget - open potd picture

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102817/#review7597 --- applets/frame/slideshow.cpp

Re: Review Request: picture frame widget - reduce polling intervall

2011-10-25 Thread Aaron J. Seigo
> On Oct. 25, 2011, 11:23 a.m., Aaron J. Seigo wrote: > > the fix is in the wrong place; it is actually in the dataengine where this > > polling interval is set. and it is indeed set to 2 seconds there! without > > changing the potd dataengine, the network traffic and some of the cpu usage > >

Re: Review Request: picture frame widget - reduce polling intervall

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102816/#review7595 --- the fix is in the wrong place; it is actually in the dataengine

Re: Review Request: Workaround Taskbar ghost entries bug

2011-10-25 Thread Aaron J. Seigo
On Friday, October 14, 2011 05:19:09 John Stanley wrote: > Hi, > I submitted a patch which hopefully fixes Bug > https://bugs.kde.org/show_bug.cgi?id=275469), > which may be related to this issue (see Comment #50). The patch is > short, so here it is: > > --- kde-workspace-4.7.2.old/libs/taskmanag

Re: Review Request: Plasmate: Metadata is now able to detect the correct api

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102609/#review7594 --- Ship it! mainwindow.cpp

Re: Review Request: Plasmate:images can now be added to projects

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102290/#review7593 --- editors/editpage.cpp

Re: Some help for recreating the Plasma Tool Box in QML

2011-10-25 Thread Aaron J. Seigo
On Friday, October 21, 2011 13:38:49 Viranch Mehta wrote: > Well, I'm not sure if it is possible to just change a part of the panel (the > tool box). You'd need to re-do the panel in qml, afaik. the toolbox _icon_ itself is a very tiny piece and doesn't need to be touched. it could be re-done in

Re: Left set of buttons in the title bar

2011-10-25 Thread Aaron J. Seigo
On Tuesday, October 18, 2011 13:15:07 Ivan Čukić wrote: > This provides more actions than max/min and close buttons do. It is > available via right-click, but this way it is more discoverable. it also works in touch-only environments, which has already saved me at least once. :) -- Aaron J. Sei

Re: How to do a Plasma runner which only triggers once the query string is complete?

2011-10-25 Thread Aaron J. Seigo
On Monday, October 17, 2011 20:32:29 Friedrich W. H. Kossebau wrote: > while Plasma runners try to resolve on every change of the entered string as > it seems, the Marble runners only try to resolve once Enter is pressed, > i.e. they only try to resolve for the full string (e.g. to reduce traffic >

Re: Thoughts about statusbar

2011-10-25 Thread Aaron J. Seigo
On Tuesday, October 11, 2011 15:10:42 Martin Gräßlin wrote: > On Tuesday 11 October 2011 12:54:23 todd rme wrote: > > On Tue, Oct 11, 2011 at 12:16 PM, Alex Fiestas wrote: > > > Taking a look at some apps, I've noticed that most of them have an > > > almost > > > unused statusbar (or totally not u

Re: Review Request: ensure activity resources exist

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102905/#review7590 --- this looks reasonable to me; would be good to have a "shipit" co

Re: Replace kimpanel in kdeplasma-addons with a rewrite version

2011-10-25 Thread Sebastian Kügler
On Tuesday, October 25, 2011 10:18:12 Weng Xuetian wrote: > On Tue, Oct 25, 2011 at 4:03 PM, Sebastian Kügler wrote: > > On Tuesday, October 25, 2011 08:54:20 Weng Xuetian wrote: > >> Since 4.8 Soft Freeze is coming, and I'm interested introduce this in > >> 4.8. (No matter a separate one or insid

Re: Replace kimpanel in kdeplasma-addons with a rewrite version

2011-10-25 Thread Weng Xuetian
On Tue, Oct 25, 2011 at 4:03 PM, Sebastian Kügler wrote: > On Tuesday, October 25, 2011 08:54:20 Weng Xuetian wrote: >> Since 4.8 Soft Freeze is coming, and I'm interested introduce this in >> 4.8. (No matter a separate one or inside with kdeplasma-addons) >> Can I add this to feature plan? > > Ye

Review Request: Update UI for settings of "Speak Time" in digital clock.

2011-10-25 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102944/ --- Review request for Plasma. Description --- A rewrite of https://git.r

Re: Review Request: Add option to not show popup when devices are added

2011-10-25 Thread Simon Persson
On Tuesday 18 October 2011 14.47.00 Jacopo De Simoi wrote: > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102825/#review7464 > ---

Re: Replace kimpanel in kdeplasma-addons with a rewrite version

2011-10-25 Thread Sebastian Kügler
On Tuesday, October 25, 2011 08:54:20 Weng Xuetian wrote: > Since 4.8 Soft Freeze is coming, and I'm interested introduce this in > 4.8. (No matter a separate one or inside with kdeplasma-addons) > Can I add this to feature plan? Yes, please do so. :) -- sebas http://www.kde.org | http://vizZzio