> 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? > > Diego Casella wrote: > 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 :)
Commenting on: "if you want to implement the new kmix as a qml plasmoid" Well, I have no knowledge in QML and no desire to learn it. If you still want to do so - focussing on the tray popup/plasmoid - then it should be much more functional than your prototype - especially: supporting multiple controls, adding Media player control like in todays KMix. The KMix Mainwindow will surely always stay a normal application, everything else is a complete new application. - Christian ----------------------------------------------------------- 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
