> On July 26, 2013, 10:30 p.m., Christian Esken wrote: > > I have not followed this request, but it looks like I have not receieved > > any followups on my last comment. I would like to know what to do with this > > review request. Diego, Igor? Diego, are you still following up on this. > > Care to contact Igor?
Hi Christian, after hearing that someone else was working on it, I left the development of that plasmoid; I didnt' want to step over Igor's work. Anyway, by looking at the current kmix status, it's still missing the conversion to a full kded service: otherwise there will be two entries in the tray: the "old" kmix, and its plasmoid counterpart. We also need to extend its dbus interface and add a method to pop-up the full kmix gui: in that way, the plasmoid can show the user interface, allowing the end-user to configure kmix properly. As soon as those changes will be peformed (if you agree with it of course, and if you want to implement the new kmix as a qml plasmoid), I'd be happy to continue what I did so far :) - Diego ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6928/#review11062 ----------------------------------------------------------- On March 28, 2012, 6:49 p.m., Diego Casella wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/6928/ > ----------------------------------------------------------- > > (Updated March 28, 2012, 6:49 p.m.) > > > Review request for Plasma, Aaron Seigo, Christian Esken, and Marco Martin. > > > Description > ------- > > First attempt of making a declarative kmix applet for plasma. > What the apple does right now: > * modifies the volume level and the mute/unmute status of the master channel; > * reacts to changes of the volume level/status (i.e. made with multimedia > keys); > * disables the slider if the channel gets muted, and enables it back as soon > as the channel gets unmuted; > * collapses gracefully in a popup icon when placed inside the panel. > > > Diffs > ----- > > trunk/KDE/kdemultimedia/kmix/plasma/CMakeLists.txt 1287513 > > trunk/KDE/kdemultimedia/kmix/plasma/kmix-applet/contents/code/VerticalControl.qml > PRE-CREATION > > trunk/KDE/kdemultimedia/kmix/plasma/kmix-applet/contents/code/kmixapplet.qml > PRE-CREATION > trunk/KDE/kdemultimedia/kmix/plasma/kmix-applet/metadata.desktop > PRE-CREATION > > Diff: http://svn.reviewboard.kde.org/r/6928/diff/ > > > Testing > ------- > > Tested against r1287510. For basic audio management it works great imho. > > However, there is a lot of room for improvements, but this is gonna need some > extra work outside the kmix applet scope: > * first of all, the applet need kmix executable to run in order to perform > the dbus calls. You can of course disable KMix tray icon feature but, at > every login, KMix mainwindow will be shown and the user must closeby hand. > This is a kind of ugly behavior that should be avoided; > * it will be great to great to add an action to allow the user to select the > master channel (by reusing KMix "Select Master Channel" widget), but this > will require tweaking KMix dbus interface; > * as you noticed in the screenshots, the applet in the panel and in the > desktop have different size even if it __is__ actually the same: something is > going wrong when plasma shows the PopupApplet. This behavior was even worse > when I started implementing a "flip" action to change the layout from > horizontal to vertical and vice-versa, and for this reason I gave up and > simply stick with the vertical layout. > > Could this applet be shipped in the current status, or should we wait for all > the aforementioned improvements? > Comments/ideas/suggestions? > > Cheers :) > > > Screenshots > ----------- > > Applet look in panel and desktop > http://svn.reviewboard.kde.org/r/6928/s/627/ > Applet look in panel and desktop - audio muted > http://svn.reviewboard.kde.org/r/6928/s/628/ > > > Thanks, > > Diego Casella > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
