D13178: Compress calls to configuration slots upon new connections

jacopods updated this revision to Diff 36031. jacopods added a comment. I used the direct connection as suggested. Today I learned about QOverload... CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D13178?vs=35045&id=36031 REVISION DETAIL https://phabricator.kde.org/D13178 AFF

D13178: Compress calls to configuration slots upon new connections

jacopods added a comment. In D13178#270671 , @mart wrote: > are configureKeyboard configureMouse etc doing an immediate write to disk? (like KConfig::sync() Hey notmart, long time no see! The configureKeyboard() method is calling xmo

D13178: Compress calls to configuration slots upon new connections

jacopods created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. jacopods requested review of this revision. REVISION SUMMARY Some input devices register several copies of themselves upon connection (e.g. a mouse registers

Re: Review Request 128702: Use default weight rather than normal weight

marked as submitted. Review request for Plasma. Changes --- Submitted with commit 663c87bc6f3881091c04beb3ce447922945cf14e by David Edmundson on behalf of Jacopo De Simoi to branch master. Repository: plasma-workspace Description --- Honor the choice of a user that has chosen a

Review Request 128702: Use default weight rather than normal weight

, Jacopo De Simoi

Re: RFC: Change Krunner's default key shortcut for Next

> Currently the secondary shortcut is set to Alt + Shift + F2. I don't think > any of us will miss this. Actually this opens krunner with the content of the clipboard as the search term; quite handy sometimes. Is this change going to break this behavior? __J ___

Re: Review Request 112812: Use type description sort order in devicenotifier when non-removable devices are displayed

> On Sept. 28, 2013, 1:59 a.m., Jacopo De Simoi wrote: > > Let me have a look at the patch; I'll be able to do it tomorrow and commit > > it myself if it is ok. > > > > Thanks > > > > __J > > Device notifier mantainer Ok, even with the pat

Re: Review Request 112812: Use type description sort order in devicenotifier when non-removable devices are displayed

do it tomorrow and commit it myself if it is ok. Thanks __J Device notifier mantainer - Jacopo De Simoi On Sept. 19, 2013, 3:24 p.m., Benedikt Gollatz wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Mono icons in systray

> This is indeed a considerable overhead; perhaps then one could implement > shape-rendering:crispEdges rendering in QtSvg; actually if the svg routines > are shared with webkit, it could be that they are already implemented… ok, this might be pure nonsense; I checked out the specs of what crispEd

Re: Mono icons in systray

On Tuesday 11 December 2012 21:37:13 Marco Martin wrote: > On Tuesday 11 December 2012, Jacopo De Simoi wrote: > > recalling some recent post on icon fonts, I wonder if switching to an icon > > font for monochrome plasma icons would provide us with a solution; in my > > >

Re: Mono icons in systray

On Tuesday 11 December 2012 12:13:11 you wrote: > A Terça, 11 de Dezembro de 2012 10:06:36 você escreveu: > > Dear all, > > > > I noticed how the new air theme (which is just beautiful, btw) makes the > > > > mono icons in systray quite low-constrast; this is especially apparent > > with > > co

Mono icons in systray

Dear all, I noticed how the new air theme (which is just beautiful, btw) makes the mono icons in systray quite low-constrast; this is especially apparent with compositing turned off. I started thinking about a sensible solution which would not require rebuilding all mono icons every time

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

uality) is neither feasible nor desiderable > * fallback to a qicon if the svg is not found > * always paint in the nearest (smaller) kiconloader standard size (even > for svg, for consistency in layouts and because they aren't as scalable as > they seem, if their shape edges

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

> On Nov. 26, 2012, 9:35 a.m., Marco Martin wrote: > > This is really an issue that needs addressing, thanks for taking time for > > it. > > > > i'm very in favor of removing all those hardcoded sizes from the code. > > > > however i see two problems with that approach > > > > 1) icons are cl

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

it so hard to handle such things nicely. I hope to be able to test this during the weekend; the diff looks reasonable, though. Thanks __J --Device notifier mantainer - Jacopo De Simoi On Nov. 23, 2012, 10:11 p.m., Michael Zanetti wrote

Re: Review Request: Device notifier: show mounted device and path

On Friday 12 October 2012 15:56:18 Sebastian Kügler wrote: > On Friday, October 12, 2012 13:42:35 Lamarque Vieira Souza wrote: > > This tooltip looks really odd and out of place this way.:/ > > The transparency effect does not look good here. I would like to know how > > to disable it too, there is

Re: Review Request: Device notifier: show mounted device and path

> On Oct. 8, 2012, 9:52 a.m., Aaron J. Seigo wrote: > > any application which expects the user to access files on disk but does not > > provide a clear representation of mounted / removable devices is broken. > > there's no point in degrading our own primary UI for such fixable > > brokenness.

Re: Review Request: Device notifier: show mounted device and path

some other way (e.g. by making use of lsof in solid to identify blocking apps), so I am not super-sure that the user should be given such specific information as /dev/. - Jacopo De Simoi On Oct. 7, 2012, 2:55 p.m., Jonathan Marten wrote

Re: [RFC] Trigger KRunner on top screenedge

On Wednesday 03 October 2012 13:44:06 Martin Gräßlin wrote: > Hi all, > > just wanted some feedback on an idea I got today. What about binding the > showing of KRunner on hitting the top screen edge? > > Given that KRunner slides in from the top edge it would end up in a neat > functionality, thr

Re: Review Request: Use Product instead of description for device names

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106637/#review19654 --- Ship it! Ship It! - Jacopo De Simoi On Sept. 29, 2012, 8

Re: Review Request: better handling of Removable property for volumes in soliddevice engine

to the dataengine. I would rather consider inclusion in the solid API; I vaguely recall having read (or written) some similar code somewhere else, so it would definitely make sense to put it in the libs. Best Regards - Jacopo De Simoi On July 5, 2012, 6:44 a.m., Andriy Gapon wrote

Re: Plasma Applet Testing for 4.9

On Monday 21 May 2012 00:37:07 Viranch Mehta wrote: > On Thu, May 17, 2012 at 3:58 AM, David Edmundson > wrote: > > - a list of all the new QML-based applets (by the time of the first beta) > > > >(afaik, nowplaying, battery, locklogout, activitymanager.. but > > > > there are so many more

Re: Review Request: Avoid multiple map - engine connections

expected. Thanks, Jacopo De Simoi ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request: Avoid multiple map - engine connections

to have been like that from ages. Was there a reason for that? Diffs - plasma/generic/dataengines/soliddevice/devicesignalmapmanager.cpp 4549cd4 Diff: http://git.reviewboard.kde.org/r/105003/diff/ Testing --- basic testing: everything works as expected. Thanks, Jacopo De Simoi

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

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. - Jacopo De Simoi On Oct. 11, 2011, 1:30 p.m., Simon Persson

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

n't happen right now. I'll have a more in-depth look. Please bear with me a bit more; spare time is very scarce in these days. - Jacopo De Simoi On Oct. 11, 2011, 10:18 a.m., Simon Persson wrote: > > --- > This is an autom

Review Request: Make the activities runner available for single runner use

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101144/ --- Review request for Plasma and Aaron J. Seigo. Summary --- Enables the

Re: Review Request: Expand last plugged device in Device Notifier applet

What happens when a device with multiple partitions is plugged in? Afaics all of them are expanded, which I'm not sure it is really what I'd like to happen, because the second partition is likely to be hidden below the first one… > > --- >

Re: about commit 83ea19670eff0febff516380fb59a65a1d7667a9: use Qt::Popup in tasks applet

> On Tuesday, March 8, 2011, Jacopo De Simoi wrote: > > This strategy is quite debatable, but apparently has been there forever > > (<4.5) Can we workaround this fact? > > i don't know; to me it really sounds like something that needs to be fixed in > Qt.

about commit 83ea19670eff0febff516380fb59a65a1d7667a9: use Qt::Popup in tasks applet

Hi plasma ppl, The issue with pixmap initialization I referred to in my previous thread hits most notably popups in the tasks applet. Even if good care has been put in avoiding this to happen (with the fix described in the previous thread), the commit in the subject breaks this attempts, since

Take care of your WA_TranslucentBackground flag

A few days ago I started investigating about some visual glitches happening with the sliding popup effect, which are caused by a "bizarre" initial pixmap initialization strategy. Basically what happens is that under some conditions (most notably when the Qt widget window has the Qt::WA_NoSyste

Re: [patch] enabling autorun in devicenotifier applet

> On Wednesday, January 5, 2011, Jacopo De Simoi wrote: > > Was there a specific reason for patching the notifier? > > yep: Maarten, who wanted, the feature appear on irc and i offered a few > places > he could do this, and he picked the easiest and shortest pat

Re: [patch] enabling autorun in devicenotifier applet

> Hi, > > I've made this patch with some directions from aseigo. > > I wanted some kind of autorun settings for specific devices dependant on the > media of optical drives; so i made a patch to recognize X-KDE-Autorun in > .desktop file action entries. > > this patch was for 4.5.90 and it buil

Re: KDE/kdebase/workspace/plasma/generic/applets/devicenotifier

> On Wednesday 14 July 2010 20:10:42 Jacopo De Simoi wrote: > > SVN commit 1149977 by jacopods: > > > > First attempt to get rid of the capacity bar in the device notifier: use a > > Plasma::Meter with tooltips. Screenshot: > > http://imagebin.ca/view/ccJ3dD.html

Re: KDE/kdebase/workspace/plasma/generic/applets/devicenotifier

> On July 14, 2010, Jacopo De Simoi wrote: > > SVN commit 1149977 by jacopods: > > > > First attempt to get rid of the capacity bar in the device notifier: use a > > Plasma::Meter with tooltips. Screenshot: > > http://imagebin.ca/view/ccJ3dD.html > > the

KDE/kdebase/workspace/plasma/generic/applets/devicenotifier

SVN commit 1149977 by jacopods: First attempt to get rid of the capacity bar in the device notifier: use a Plasma::Meter with tooltips. Screenshot: http://imagebin.ca/view/ccJ3dD.html Feedback is more than welcome CCMAIL:plasma-devel@kde.org M +17 -17deviceitem.cpp M +3 -2 devic

Re: Review Request: Change device notifier tooltips to respect plasma tooltip style

> On May 22, 2010, Jacopo De Simoi wrote: > > I'm playing around with that now, and here's what I got so far... > > how do you like it? > > it puts a third size of icon in the dialog, and it isn't really aligned with > anything visually. Yep, I was try

Re: Review Request: Change device notifier tooltips to respect plasma tooltip style

> > Like one more font face > > and not aligned with anything either. > > the font should be the same as the top header, so there shouldn't be another > font face. as for alignment, iirc they are right aligned. You recall correctly, but it has been changed since :) Right aligning with (un)mount

Re: Review Request: Change device notifier tooltips to respect plasma tooltip style

> > you mean, to make the icon smaller? > > Yes, it takes *way* too much space. And, what's the point? The user > will learn very soon what this widget does, so then it's kinda > unnecessary. There are a lot of other plasmoids out there. Some of > them do have a header, some of them don't. I have

Re: Review Request: Change device notifier tooltips to respect plasma tooltip style

> Well, as far as device notifier goes, there's a couple more things about it > that I'd like to see fixed as well. Here I am, the device notifier mantainer, at your service :) > This one about tooltips sounds nice, but then it kinda duplicates what's in > the real device list, and there seems

Review Request: Routing (un)mounting error notifications via knotify; part II: dataengine

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3896/ --- Review request for Plasma, Aaron Seigo, Marco Martin, and Olivier Goffart. Summa

KDE/kdebase/workspace/plasma/generic/applets/devicenotifier

SVN commit 1113686 by jacopods: Show error messages inside the applet instead of using Plasma::MessageBox As soon as a patch in solid is accepted, the notifier will be able to react to errors triggered by other applications, hence errors will be shown also when triggered from other applications

Re: Running queries via dbus

I just got a quite crazy idea; it might work though. Since we are making the thread stall by waiting for the async reply we could as well check that while we get the pending reply the context is still valid and bail out discarding the reply otherwise. If this can be coded in a simple way I gues

Re: Running queries via dbus

> On Tuesday 23 March 2010 21:55:08 Alex Merry wrote: > > Instead, you should use QDBusConnection::asyncCall. This can be used in > > exactly the same way (implicit casting FTW), and casting to a QDBusReply > > will cause the calling thread to block as you would expect. But, > > importantly, it W

Re: Running queries via dbus

> CC'ing plasma-devel, since this is an important point regarding runners that > use D-Bus. Please keep any discussion on this issue at plasma-devel. > > On Tuesday 23 March 2010 17:26:54 Alex Merry wrote: > > On Monday 22 March 2010 22:05:26 Jacopo De Simoi wrote: >

Re: KCM icons, krunner and consistency

> On Sunday 07 March 2010 15:55:05 Jacopo De Simoi wrote: > > Browsing .desktop files of KCMs you can notice that there is no common > > convention; some of them use "xxx Configuration", some others "Configure > > xxx", others "xxx settings", o

Re: n+1st report about arrow keys in krunner

> On 8 March 2010 01:42, Aaron J. Seigo wrote: > > > this reminds me about jokes about the difference between theory and > > practice. > > > > if we remove the history then we'll get to deal with tons of bug reports > > about > > how you can't do "alt+f2, up arrow, enter" which is a very, very co

Re: 4.5 polishment: krunner (under_the_hood)

> Am Montag 08 März 2010 22:43:33 schrieb Jacopo De Simoi: > > > Am Montag 08 März 2010 01:28:23 schrieb Aaron J. Seigo: > > > > On March 7, 2010, Lukas Appelhans wrote: > > > > > I can give a patch if needed of course... it's just a matter of >

Re: 4.5 polishment: krunner (under_the_hood)

> Am Montag 08 März 2010 01:28:23 schrieb Aaron J. Seigo: > > On March 7, 2010, Lukas Appelhans wrote: > > > I can give a patch if needed of course... it's just a matter of porting > > > it from the app-runner to main KRunner... > > I'd love to see the patch! Further, I've a question, since I do

KCM icons, krunner and consistency

Dear devs, in view of our commitment to polish for 4.5, I'd like to raise a small issue regarding systemsettings and more specifically KCM icons and .desktop files At the moment icons for kcms can be shown in a few different settings, in systemsettings, in the "configure" menu of applets or ap

4.5 polishment: krunner (under_the_hood)

Hi there, this is the first part of my ideas for polishing krunner for 4.5. This thread will focus mostly about under_the_hood (i.e. not UI) changes, sooo here are my ideas: * Better sorting: at the moment krunner remembers which items the user runs frequently and kicks them up in the results l

Re: n+1st report about arrow keys in krunner

n any case I feel it's constructive to post my impressions here and hope for some smart feedback. Of course I can attach a patch if you're willing to try it out on your local copy. > Am Samstag 27 Februar 2010 23:09:57 schrieb Jacopo De Simoi: > > If (and only if) pressing Down

Re: n+1st report about arrow keys in krunner

> I have been annoyed many times by this, although I never reported a bug. I > also found it confusing to have autocompletion, but I still have to type it > through. you can use the End key to accept the completion; this is another active topic about krunner on bko, by the way. __

n+1st report about arrow keys in krunner

Dear plasmoids I just closed as WONTFIX the (n+1)st report about people trying to use downarrow key to select entries other than the first one in krunner default interface. In view of our committment to polish for 4.5 I guess it's time to do something about this issue, which seems to hit quite

Re: [Kde-hardware-devel] Status of encrypted devices in Plasma / Solid

> > looking forward to check the validity of a bug regarding the device > > notifier and encrypted devices (containers) I noticed several issues with > > them in the current implementation of the dataengines > > The imporant thing to know about them is that they show up in solid as a > > StorageVo

services runner matching categories

Dear plasma-devs After finally upgrading to 4.4 I noticed that the service runner started matching app categories; (the commit indeed happened ~6 months ago) which can be very handy for specific (i.e. single runner, plasma-netbook) queries, but might be imho a bit confusing in krunner common

Re: Review Request: update the data when the predicate files change

Ping Any news on this? This is meant to fix one of the last remaining bugs for the device notifier and it would be great to have it reviewed and merged soon Thanks On Monday 7 December 2009 18:38:59 Giulio Camuffo wrote: > > --- > This

Re: Solid - places runners conflict

On Wednesday 23 December 2009 06:50:50 Aaron J. Seigo wrote: > On December 22, 2009, Jacopo De Simoi wrote: > > I don't intend to disable the places runner itself, but just the matches > > referring to devices. > > that's probably ok; my only concern would be

Re: Solid - places runners conflict

On Tuesday 22 December 2009 22:41:01 Aaron J. Seigo wrote: > On December 18, 2009, Jacopo De Simoi wrote: > > Dear plasmadevs, > > I just noticed that we have some feature conflict among the runners; in > > fact the places runner shows solid devices as well and triggers

Solid - places runners conflict

Dear plasmadevs, I just noticed that we have some feature conflict among the runners; in fact the places runner shows solid devices as well and triggers dolphin on them upon selection. Since this feature is included in solid runner, which is specialized to treat solid devices in a fairly compl

Status of encrypted devices in Plasma / Solid

Dear plasma-devs, hardware-devs looking forward to check the validity of a bug regarding the device notifier and encrypted devices (containers) I noticed several issues with them in the current implementation of the dataengines The imporant thing to know about them is that they show up in soli

Re: Review Request: Add support for single Runner queries to krunner (part I: libplasma)

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2090/ --- (Updated 2009-11-24 12:59:38.449340) Review request for Plasma, Aaron Seigo and

Re: Solidrunner moved to kdereview

Should I move it to trunk? If so, kdebase or kde-plasma-addons? Thanks --J On Friday 13 November 2009 19:33:57 Jacopo De Simoi wrote: > I've just moved a the solidrunner to kdereview, for inclusion in 4.4 in > kde-plasma-addons or kdebase; I'm actually not quite sure on that.

Re: Review Request: Add removeMatch(es) methods to runnerContext

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2260/ --- (Updated 2009-11-22 15:32:19.727345) Review request for Plasma and Aaron Seigo.

Re: Plasma desktop crashed (mp3 player connected during startup)

On Saturday 21 November 2009 20:44:32 l...@tlen.pl wrote: > Device notifier automount version 0.3 crashes either when mp3 player is > connected during KDE startup > or when I tried to remove mounted mp3 player devicenotifier_automount is an unofficial add-on and as such it is not supported by k

Review Request: Add removeMatch(es) methods to runnerContext

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2260/ --- Review request for Plasma and Aaron Seigo. Summary --- This patch adds remo

Re: Review Request: Add support for single Runner queries to krunner (part I: libplasma)

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2090/ --- (Updated 2009-11-21 18:49:12.791648) Review request for Plasma, Aaron Seigo and

Review Request: Add action support to krunner default interface

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2259/ --- Review request for Plasma and Aaron Seigo. Summary --- This patch adds acti

Re: Review Request: Add support for single Runner queries to krunner (part I: libplasma)

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2090/ --- (Updated 2009-11-20 07:55:33.801723) Review request for Plasma, Aaron Seigo and

Re: Review Request: Update ItemBackground widget

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2194/#review3149 --- Ship it! I should have removed that HACK at least one month ago. ;) T

Re: Solidrunner moved to kdereview

now the solidrunner is in the more appropriate location kdereview/plasma/runners thanks for reviewing! On Friday 13 November 2009 19:33:57 Jacopo De Simoi wrote: > I've just moved a the solidrunner to kdereview, for inclusion in 4.4 in > kde-plasma-addons or kdebase; I'm ac

Re: Review Request: Add support for single Runner queries to krunner (part I: libplasma)

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2090/ --- (Updated 2009-11-17 17:52:14.417634) Review request for Plasma, Aaron Seigo and

Re: Review Request: Add support for single Runner queries to krunner (part I: libplasma)

add > > another default syntax. also prevents the need to explain how it works in > > the apidox :) > > Jacopo De Simoi wrote: > well, the correct name should indeed be > addSyntaxAndSetAsDefault, but I guess it's a little bit too much :) > > Aaron Seigo

Re: Review Request: Add support for single Runner queries to krunner (part II: krunner)

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2091/ --- (Updated 2009-11-16 23:50:00.387923) Review request for Plasma, Aaron Seigo and

Re: Review Request: Add support for single Runner queries to krunner (part II: krunner)

me of the > > runner, so it's really evident what's going on? > > Jacopo De Simoi wrote: > Clickmessage would be cool, however, since the lineedit immediately gets > focus, it is never shown so it is useless. > A label could be a solution indeed; > The

Re: Review Request: Add support for single Runner queries to krunner (part I: libplasma)

> > shouldn't be necessary at all. if RunnerContext::singleRunnerMode() then > > replaceEmptyQuery should be implied? > > Jacopo De Simoi wrote: > In fact it should. I was worried of changing the default behaviour of > this method. I was not sure to be able to c

Re: Solidrunner moved to kdereview

On Friday 13 November 2009 20:06:48 Burkhard Lück wrote: > Am Freitag, 13. novembre 2009 19:33:57 schrieb Jacopo De Simoi: > > > > Please let me know of any issues that you find > > There are some i18n calls, but the runner has no translation catalog, message > extra

Solidrunner moved to kdereview

I've just moved a the solidrunner to kdereview, for inclusion in 4.4 in kde-plasma-addons or kdebase; I'm actually not quite sure on that. The runner is conceived as an alternative to the device notifier, combining the same capabilities (although the actions support is still lacking for the de

Re: Review Request: Add support for single Runner queries to krunner (part I: libplasma)

> > shouldn't be necessary at all. if RunnerContext::singleRunnerMode() then > > replaceEmptyQuery should be implied? > > Jacopo De Simoi wrote: > In fact it should. I was worried of changing the default behaviour of > this method. I was not sure to be able to c

Re: Review Request: Add support for single Runner queries to krunner (part II: krunner)

me will be changed :) - Jacopo --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2091/#review3064 --- On 2009-11-07 20:29:26, Jacopo De Simoi wrote

Re: Review Request: Add support for single Runner queries to krunner (part I: libplasma)

- Jacopo --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2090/#review3063 --- On 2009-11-07 20:16:02, Jacopo De Simoi wrote: &g

Review Request: Adds windows runner single runner query support

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2092/ --- Review request for Plasma, Aaron Seigo and Martin Gräßlin. Summary --- This

Review Request: Add support for single Runner queries to krunner (part II: krunner)

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2091/ --- Review request for Plasma, Aaron Seigo and Ryan Bitanga. Summary --- This p

Review Request: Add support for single Runner queries to krunner (part I: libplasma)

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2090/ --- Review request for Plasma, Aaron Seigo and Ryan Bitanga. Summary --- This i

Re: Review Request: Embed related kcms in the device notifier configuration dialog

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1979/ --- (Updated 2009-10-31 10:40:50.365695) Review request for Plasma, Aaron Seigo, Tre

Re: Review Request: Add to the devicenotifier configuration buttons to open related kcms

On Tuesday 27 October 2009 17:16:58 Aaron Seigo wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/1979/#review2832 > --- > > >

Re: Review Request: Add to the devicenotifier configuration buttons to open related kcms

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1979/ --- (Updated 2009-10-26 18:33:26.585079) Review request for Plasma, Aaron Seigo, Tre

Review Request: Add to the configuration interface buttons calling the kcms for device actions and automounting

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1979/ --- Review request for Plasma, Aaron Seigo, Trever Fischer, and Kevin Ottens. Summar

Re: New devicenotifier moved to kdereview

On Tuesday 06 October 2009 22:17:02 Aaron J. Seigo wrote: > so much for simplicity :/ I'm confused; I can read in test-predicate-openinwindow.desktop: X-KDE-Solid-Predicate=[ [ StorageVolume.ignored == false AND [ StorageVolume.usage == 'FileSystem' OR StorageVolume.usage == 'Encrypted' ] ] OR

Re: New devicenotifier moved to kdereview

On Tuesday 06 October 2009 21:58:08 Aaron J. Seigo wrote: > > i think anyway a better approach would be to add an action > > file for them, and add a property in the DeviceItem to set if they are > > (un)mountable > > it would prevent any special-case code in the notifier widget, yes. If we ha

Re: Review Request: Unified ItemBackground in new Device Notifier

> On 2009-10-06 10:16:00, Jacopo De Simoi wrote: > > /trunk/kdereview/plasma/applets/devicenotifier-refactor/deviceitem.cpp, > > line 390 > > <http://reviewboard.kde.org/r/1790/diff/3/?file=12370#file12370line390> > > > > Now that the left(hehe)action

Re: Review Request: Unified ItemBackground in new Device Notifier

- > > (Updated 2009-10-05 23:27:10) > > > Review request for Plasma, Jacopo De Simoi and Giulio Camuffo. > > > Summary > --- > > proof of concept showing how the item background could be unified into > NotifierDialog;

Re: New devicenotifier moved to kdereview

On Sunday 04 October 2009 22:31:49 Aaron J. Seigo wrote: > > > * when an item is expanded, should the background remain painted? it > > > might make it more evident that the item is "open", and it would also > > > allow a way to solve the next point, too. (and if the background remains, > > > perh

Re: New devicenotifier moved to kdereview

> * when an item is expanded, should the background remain painted? it might > make it more evident that the item is "open", and it would also allow a way > to > solve the next point, too. (and if the background remains, perhaps the > capacity bar should too?) Now that I think some more about

Re: New devicenotifier moved to kdereview

> just move it back to playground. Done. > > ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: New devicenotifier moved to kdereview

On Sunday 04 October 2009 20:57:11 Aaron J. Seigo wrote: > On October 4, 2009, Jacopo De Simoi wrote: > > now, we believe that it is ready for merging into trunk. Of course we need > > your feedback first, so grab it, try it out and tell us what you think! > > with o

Re: New devicenotifier moved to kdereview

On Sunday 04 October 2009 12:36:49 you wrote: > > **Please notice that there is a OLD devicenotifier in kdereview; make sure > > you grab the right one** > > This is a joke right? Can't the plasma team make their mind up and remove the > old one if it's deprecated by this one? > I should at le

New devicenotifier moved to kdereview

Dear plasma-devs, hardware-devs, core-devs, giucam and I, with some very-much-appreciated help by notmart have been lately quite busy porting the devicenotifier to use QGraphicsWidgets; the superficial look-and-feel is pretty much the same as the old device notifier, plus the notable improvem

Re: Initial size and aspect ratio

> > Speaking of which, it would be nice if a popupApplet had a way to > > /remember/ its preferred AspectRatioMode; in fact last time I checked, > > it already does. > Ah, the joys of trunk! ___ Plasma-devel mailing list Plasma-devel@kde.org https://ma

Re: Initial size and aspect ratio

> > setAspectRatioMode(Plasma::IgnoreAspectRatio) > > Yes I just found it in the qalculate source :-) I was looking for a > PopupApplet example. Speaking of which, it would be nice if a popupApplet had a way to /remember/ its preferred AspectRatioMode; in fact last time I checked, once the pop

  1   2   >