FS#7105 - Music Led

Attached to Project: Rockbox
Opened by Dagni McPhee (donutman25) - Friday, 04 May 2007, 06:04 GMT
Last edited by Jonathan Gordon (jdgordon) - Monday, 06 August 2007, 02:29 GMT
Task Type Patches
Category Music playback
Status Closed
Assigned To No-one
Operating System Sansa e200
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No


First attempt to make the scrollwheel led beat to the music. Only tested it with the ipod_sansa theme. Please tell me if you experience bugs. Feel free to improve this because i have no programming knowledge.
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Monday, 06 August 2007, 02:29 GMT
Reason for closing:  Rejected
Comment by Dagni McPhee (donutman25) - Friday, 04 May 2007, 06:17 GMT
It does not work with themes that do not have the peak meter enabled. I'm gonna work more on this tomorrow. Its late and i have school in the morning.
Comment by Dagni McPhee (donutman25) - Saturday, 05 May 2007, 02:12 GMT
Small change to make the scrollwheel more sensitive to the changes in dbfs. I think :|
Comment by Michael Hahn (disorganizer) - Wednesday, 13 June 2007, 21:44 GMT
it would maybe be a good idea to check the scrollwheel light setting to enable or disable this patch via settings. if scrollwheel light is set to off then the patch should imho be disabled.
or maybe even only enable it when the scrollwheel is set to permanent on.

that way i can use the peakmeter in builds with included music-led patch :-)
Comment by Aarto Matti (Aarto) - Wednesday, 13 June 2007, 21:57 GMT
I'd like to have a separate settings for this feature, it shouldn't depend on scrollwheel settings.
Comment by Jack Suter (chrisjs169) - Thursday, 14 June 2007, 21:43 GMT
Since most WPSs look bad with the peakmeter in them, shouldn't the complaint be having a setting to turn it on without %pm in the WPS file, not the other way around? IMO it's already 'off' by default (unless you're using the default rockbox theme)
Comment by Michael Hahn (disorganizer) - Friday, 15 June 2007, 10:34 GMT
good point. maybe it could be a % directive in the WPS file to trigger the musicled feature.
Comment by Aarto Matti (Aarto) - Monday, 18 June 2007, 21:36 GMT
the better idea is to realize it as a plugin, thus it won't be necessary to worry about %pm tag or adjusting the scrollwheel light
to use the feature when you need it only
Comment by Michael Hahn (disorganizer) - Wednesday, 20 June 2007, 09:32 GMT
i think its own % tag for the wps would be the better idea because you can easily turn it off when switching the wps (for example no blinking in car-wps, but blinking in mobile-wps).