[Fwd: Review Request: New context menu applet to display guitar and bass-tablature]

[Rainer Sigle]

As mentioned in http://git.reviewboard.kde.org/r/100117 I've attached
the necessary png-files.

Location for these files would be "src/context/applets/tabs"

Regards
Rainer

-------- Weitergeleitete Nachricht --------
Von: Rainer Sigle <rainer.sigle at web.de>
An: Rainer Sigle <rainer.sigle at web.de>, Amarok <amarok-devel at kde.org>
Betreff: Review Request: New context menu applet to display guitar and
bass-tablature
Datum: Sun, 31 Oct 2010 14:28:26 -0000

This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100117/ 

Review request for Amarok.
By Rainer Sigle.

Description 

This patch adds a new applet to the Amarok context menu: "Tabs". It's designed for guitar and bass-players, as it downloads and displays tablature information for the current song. Directly playing along a song has therefore never been easier.

The tab data is currently fetched from two sites: www.ultimateguitar.com and www.fretplay.com. As these provide no direct API, the tab data is extracted using some simple HTML parsing. Multiple results, e.g. versions of the tab, are stored and can be choosen from a list. A settings dialog allows to define the type of tabs to be fetched. In this patch we've guitar and bass, but it could be extended to e.g. drums or piano in the future. Furthermore I've added a reload button which allows to easily refine the criteria for the tab-search. This can be used e.g. when no tab-results were found because the song name contains fragments like (live), remix, etc.

I'll send some new png-images to amarok devel mailinglist because I was not able to correctly include these binary files into the patch. Please be aware that I don't know if there are any copyright restrictions on these bitmaps! Maybe they can be replaced with some KDE/Plasma specific images. Unfortunately I didn't find anything appropriate.

Thanks for any feedback on this...

By the way: Also big Thanks! to all Amarok devs and supporters for their great work. I love this application.

Testing 

used and tested during the last months, works fine for me

Diffs 

      * src/context/applets/CMakeLists.txt (98bdd2b)
      * src/context/applets/tabs/CMakeLists.txt (PRE-CREATION)
      * src/context/applets/tabs/ReloadEditDialog.ui (PRE-CREATION)
      * src/context/applets/tabs/TabsApplet.h (PRE-CREATION)
      * src/context/applets/tabs/TabsApplet.cpp (PRE-CREATION)
      * src/context/applets/tabs/TabsItem.h (PRE-CREATION)
      * src/context/applets/tabs/TabsItem.cpp (PRE-CREATION)
      * src/context/applets/tabs/TabsSettings.ui (PRE-CREATION)
      * src/context/applets/tabs/TabsView.h (PRE-CREATION)
      * src/context/applets/tabs/TabsView.cpp (PRE-CREATION)
      * src/context/applets/tabs/amarok-context-applet-tabs.desktop
        (PRE-CREATION)
      * src/context/applets/tabs/amarok-tabs-bass.png (PRE-CREATION)
      * src/context/applets/tabs/amarok-tabs-drum.png (PRE-CREATION)
      * src/context/applets/tabs/amarok-tabs-guitar.png (PRE-CREATION)
      * src/context/applets/tabs/amarok-tabs-piano.png (PRE-CREATION)
      * src/context/engines/CMakeLists.txt (c1f029e)
      * src/context/engines/tabs/CMakeLists.txt (PRE-CREATION)
      * src/context/engines/tabs/TabsEngine.h (PRE-CREATION)
      * src/context/engines/tabs/TabsEngine.cpp (PRE-CREATION)
      * src/context/engines/tabs/TabsInfo.h (PRE-CREATION)
      * src/context/engines/tabs/amarok-data-engine-tabs.desktop
        (PRE-CREATION)


View Diff



Screenshots 

Tabs Applet

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20101031/dd6b0d9c/attachment-0001.htm 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: png-images-tabs-applet.zip
Type: application/zip
Size: 13404 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20101031/dd6b0d9c/attachment-0001.zip