Thanks both for the feedback.    I'm completely with you on non-bloated
code. the tweaks necessary to support the BOOM are small:

1) Use $client->isa("Slim::Player::Boom") to identify Boom, and do all
the following inside a "if ( $client->isa("Slim::Player::Boom") ) "
wrapper
2) Adjust gxwidth to move the standard timeline display to the right
3) Add a ":" to the custom 5x7 character set, so the time can display
properly
4) Tweak retrieveWeather function:
- Capture both wind & precip info (instead of one or the other) 
- Minor string formatting (mostly stripping extra spaces) for
weather strings
- Draw text to different locations, store forecast/wind/precip to
variables 
5) Add date/time display to tictac (not retrieveWeather) -- good call
on this,  I was wrong to put this in the retrieveWeather section


Again, thanks for the suggestions.   I'll make the modifications to the
latest code, and then offer it up for scrutiny.


-- 
bsc001
------------------------------------------------------------------------
bsc001's Profile: http://forums.slimdevices.com/member.php?userid=22844
View this thread: http://forums.slimdevices.com/showthread.php?t=21989

_______________________________________________
plugins mailing list
[email protected]
http://lists.slimdevices.com/lists/listinfo/plugins

Reply via email to