Re: [Rhythmbox-devel] [Weekly report] Week 11 : Rhythmbox Context Pane

[giopas <linux giopas eu>]

Hi John,

It's about one week that I'm testing your plugin and as far now it works definitely pretty well.

On Sat, Aug 15, 2009 at 7:35 AM, John Iacona <plate0salad gmail com> wrote:

I have posted an update which fixes quite a few of the bugs people
have reported to me including:
* Toolbar button: I added a button which toggles the visibility of the panel
* 0 length albums: Some albums from last.fm have 0 tracks. I now
filter those out

Toolbar button is very useful, thank you!

There is still two changes I'd suggest:

1. do not display the context panel when you listen radio
2. when there is a missing information about a certain artist/song (like no history album) the plugin still shows previous artist retrieved information (e.g. Santa Esmeralda - Don't Let Me Be Misunderstood). Maybe it should just display an error message instead.
 

* Size issues with the side panel: this is still somewhat problematic
but I have a patch which prevents the side panel from getting too
small. I have not, however, figured out how to give it a maximum size.
This bug requires a patch to the rhythmbox core code which can be
found at http://bugzilla.gnome.org/show_bug.cgi?id=591874

Unfortunally, I can no more test on that computer, but I've seen that Jonathan has already committed the patch upstream.
 

The updated plugin files can be downloaded at

http://www.cise.ufl.edu/~jiacona/soc/context.tar

Regards,
John Iacona

Thank you again for your work. I hope it will be included in source code soon!

giopas

ps: I'd like very much to have also a panel retrieving a list of available guitar and bass tabs/chords...