----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105059/#review14353 -----------------------------------------------------------
Hi Anton, sorry for the long waiting about a review here. But I just checked your code and I found it very good. I'm just a bit worried about this "strange" interaction: why should a browser take care of this interaction? The change you requested is "monumental" compared to the "tiny" use case you provided. How many users are using the now playing plasmoid? How many players interact with it (everyone?)? How many browsers interact with it (noone?)? Is all this done just to provide a shortcut to play/stop music? What about other multimedia (non-html5) sources? How can users react to the missing functionality in their management? - Andrea Diamantini On May 26, 2012, 8:18 a.m., Anton Kreuzkamp wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105059/ > ----------------------------------------------------------- > > (Updated May 26, 2012, 8:18 a.m.) > > > Review request for rekonq. > > > Description > ------- > > This patch addes a mpris2-compliant dbus-interface to rekonq that allows > controling html5-video and -audio elements from e.g. the nowplaying-plasmoid. > This is espacially useful together with youtube's html5-player and the > plasmoid PlayControl (which allows to assign global shortcuts to > mediaplayer-actions (so you can pause all mediaplayers with the same > mediacontrol-keys)) > With this patch rekonq aggregates all opened media elements and provides one > interface to dbus (providing one interface per media element is not possible > with mpris) that contains the data of > and sends commands recieved via dbus to the last active mediaplayer (that was > last loaded or changed its playback-status most recently). > The communication to the media element happens via javascript. > > > Diffs > ----- > > src/CMakeLists.txt 1180ad0 > src/application.h 36114ae > src/application.cpp ef6c208 > src/dbus/mpris2/Mpris2DBusHandler.h PRE-CREATION > src/dbus/mpris2/Mpris2DBusHandler.cpp PRE-CREATION > src/dbus/mpris2/org.mpris.MediaPlayer2.Player.xml PRE-CREATION > src/dbus/mpris2/org.mpris.MediaPlayer2.root.xml PRE-CREATION > src/webpage.h abc9f64 > src/webpage.cpp ce1151d > > Diff: http://git.reviewboard.kde.org/r/105059/diff/ > > > Testing > ------- > > Comprehensive tests with different html5 mediaplayers (audio and video) and > different mpris-controllers. No problems anymore. > > > Thanks, > > Anton Kreuzkamp > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
