Rockbox

Good work!
Can we used some frequency scale for the radio? Even a progress bar that ranges from minimum to maximum frequency will do.

funny you should mention that :p
%pb (progressbar) is now working.

   radio_skin.patch (22.6 KiB)

sync and update. I apparently didnt update the last time I played with this.

   radio_skin.patch (24.8 KiB)

and add the last of the missing tags.. namely RDS and fm recording support.

someone with a archos recorderFM needs to test this patch. This is I tihnk ready for commit

   radio_skin.patch (31.6 KiB)

Excellent stuff! I think I’ve found a couple of bugs however:

a) In preset mode if you skip through presets using left/right then the preset name %TnX doesn’t update - but if you select the preset from the list then it works fine
b) In scan mode, the frequency doesn’t show properly. If the current frequency is the same as a preset it shows that, but if not it shows 106.6
c) When you switch between preset and scan mode then whilst the mode switches, %tm doesn’t update until you leave the screen and come back e.g. go to the menu and back

The fms/sbs I’m using is attached.

   CSB.fms (0.5 KiB)
   CSB.sbs (1 KiB)
   CSB.cfg (0.7 KiB)

OK, ignore b), that was an error and c) might be - I need to check more :)

a) is real however

   CSB.fms (0.8 KiB)
   CSB.sbs (1 KiB)

this should fix a and c

   radio_skin.patch (32.8 KiB)

jdgordon:
Would it be possible to display the current frequency using custom bitmap numbers? Perhaps tokenizing each digit of the current frequency and using a bitmap strip with the digits 0-9 and a decimal point. That would allow for a prominent frequency display (similar to most OF radio screens) without using multi-font to get larger digits. It would look good on large displays where the FM screen is mostly empty.

it is possible, but its not going to happen. There is a patch on the tracker to do this but its been rejected.

Rejected? That hasn’t stopped you before ;) Think of it as album art for the FM screen…some will use it, others won’t.

I thought that with the skin engine this would be easy (although I don’t really know how the skin engine works). Wouldn’t it just be 6 extra tokens (xxx.xx). I assume the bitmap loading and display is already in the engine. The bitmap strip would need digits 0-9, a decimal, and a black space (for the leading digit for frequency < 100). If it uses the skin buffer, there’s no real additional cost.

Maybe I’m just misunderstanding how the skin engine & tokens work.

oh its a dead simple patch.  FS#9710 . I don’t really remember why it was rejected, if you’re game we can try bringing it back. especially now that the buffer usage is 0 if you dont use the tokens.

fixed hwcodec compile

   radio_skin.patch (33.4 KiB)

fix the warnings and the WPS not hiding the statusbar correctly. This is just about ready me thinks.
I want to nail down the tokens we want to support first though. is the current set good? or should they be fixed?

also, we need to deprecate LANG_FM_STATION and come up with a new label for it. Changing the source string is BAD! especially when its fed directly to printf

   radio_skin.patch (34.9 KiB)

sync to svn

   radio_skin.patch (35.5 KiB)

minor change, dont update the screen unless we need to (the whole fm screen is pretty damn static), but do a full update when it does change.

I want to commit this already, so testers neded

   radio_skin.patch (35.6 KiB)

Looks great, with two minor provisos:

The backdrop only gets drawn in viewports - the same issue as was fixed for the WPS in r24571

If the fms contains %C it breaks on target (but not on sim). I know %C isn’t supported yet for fms, but JdGordon requested I pop it here :)

how did %C break it on target? it looks like it is safe (although, I’m ok with the workaround of dont use that tag)
attached should fix the backdrop isssue. I’m commiting this tomorow night!

   radio_skin.patch (35.9 KiB)

resync

   radio_skin.diff (36 KiB)

remove the %Tf/n/iX buisness and just use %tf/n/i for the “current” preset. “current” being the current one, or the one in the playlist viewer.
I don’t think we need to add special handling for the next one because the playlist viewer is a better way to do this?

no other changes in this version.

bah

   radio_skin.diff (36.6 KiB)

fixed backdrop, scrolling, added the file handling (i.e menu items)

   radio_skin.diff (42.7 KiB)

A couple of little things: The menu backdrop shows through on scrolling lines, and when you exit the fms to the menu the menu backdrop isn’t properly drawn again.

dump1 attached shows a scrolling line in the WPS, and dump2 shows the menu immediately after leaving the fms

There is also a small glitch at the top of the fms where the menu backdrop shows through the section of the UI viewport there. This also happens on the WPS, but SVN is fine.

I can attach the theme if needed.

   dump1.png (21.5 KiB)
   dump2.png (20.9 KiB)

Attached is a patch to make the playlist viewer support alignment tags with scrolling (by JdGordon). It is on top of the latest fms patch.

   alignment+scroll.diff (2.4 KiB)

sync and fix the backdrop issue.
can you upload a theme with the scrolling issue?

   radio_skin.diff (43.3 KiB)

oh, hmm.. looks like a bad sync a while back… this one should fix the backdrop issue

   radio_skin.diff (43.2 KiB)

All looking good - I can’t see any remaining issues with the testing I’ve done :)

Thanks :)

still a bit of code duplication that can be cleaned up, but this is the first version that I tihnk is ready for commit.

   radio_skin.diff (48.2 KiB)

sync…

   radio_skin.diff (48.3 KiB)

sync

   radio_skin.diff (48.6 KiB)

another sync…
There is a problem where there isnt enough room on the buffer for the fms on non colour targets, so if you test on mono/grey then modify line 58 of apps/gui/skin_engine/skin_buffer.c. change that 2 to some larger number (10 or 12 perhaps) and it should work.

   radio_skin.diff (49.1 KiB)

sync

   radio_skin.patch (47.7 KiB)

AlexP-proof the two preset tags (fixes the issue where it hangs if you dont have any presets loaded)

   radio_skin.diff (47.8 KiB)

I have found a couple of issues while testing the latest patch.

1. In the main menu, while FM Radio is highlighted, if you bring up the context menu and load a preset file you are taken into the FM screen after the file is loaded. If there is a progress bar in the FMS (including the in-built default), the progress bar does not work. The progress bar will work properly if you exit the radio screen and reenter.

2. In scan mode, I use this %?Tn<%s%ac%Tn|%Tf> to display a presets name (or frequency if it is unnamed) when a presets frequency is tuned. When not on a preset frequency, the preset name of the highest preset is displayed. I am attaching a fix for this, although it may not be the best way to handle this issue.

   preset_name_fix.patch (1.4 KiB)

straight sync from my previous patch.. starting numbers again.
Michael I dont quite understand your fix?

   radio_skin.1.patch (48.5 KiB)

Here is a better fix:

change skin_tokens.c line 390 from “if (radio_preset_count() == 0)” to “if (radio_preset_count() == 0 || preset == -1)”

Thanks, that change is in this patch, also fixed the issue in your comment 2 up about the bar not working when loading a presets file.

   radio_skin.2.diff (48.5 KiB)