Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Recording
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by jdgordon - 2008-05-13
Last edited by jdgordon - 2008-05-28

FS#8989 - start of rec trigger settings -> viewport

the plan for this patch was to change the rec trigger settings screen to not use inline settings… ideally to show the menu and optoins in a VP at the top of the screen and have a 2nd vp for the peak meters at the bottom… I unfortunatly finished the top bit before realsing the peakmeters wouldnt be updated :(

so im putting this here in the hope someone comes up woth a solution

Closed by  jdgordon
2008-05-28 11:02
Reason for closing:  Accepted
jdgordon commented on 2008-05-13 15:15

so, this isnt as good as I was planning, but it still saves about 1.3k on the rec build…

2 line settings, the top line has an icon so it should be workable.

NOTE: this version doesnt talk, need to put that back in…

   rectrigger.diff (20.4 KiB)
jdgordon commented on 2008-05-15 07:24

new version.. should actually work now and buttons are hopefully better, attached is the full zip for h120

   rockbox.h120.zip (1.86 MiB)
   rectrigger.1.diff (27.4 KiB)
jdgordon commented on 2008-05-21 07:54

new version.. talks goodly.

   rectrigger.2.diff (28.2 KiB)
jdgordon commented on 2008-05-21 14:07

hoipefully final version… commiting in a day or two unless anyone comments… If the only negative against this patch is the 2 line settings then IMO thats not a big enough deal to hold this out. major issues fixed here is no need to keep setting values in sync in multiple places, should give a big green delta, no more abusing a setting, etc

   rectrigger.3.diff (42.6 KiB)
pixelma commented on 2008-05-21 22:24

The last patch does not compile for my OndioFM (with or without the “changes” for the backlight mod). Building fails with

/1_builds/ondiofm/apps/menus/recording_menu.o: In function `_rectrigger’:
recording_menu.c:(.text+0×538): undefined reference to `_option_select_next_val’

However it builds for M5 and c200. Tried to play around with it and it seems to do something on M5 (and the trigger indicator showed correctly different behaviour when changing the thresholds for example but I didn’t have enough experience with triggered recording and some handy recording equipment to test more.

jdgordon commented on 2008-05-21 22:38

ondio is missing quickscreen which currently that function isnt compiled for, that’ll be fixed

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing