Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Plugins
  • 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 teru - 2008-07-29
Last edited by nls - 2008-10-25

FS#9224 - Calendar button map for Gigabeat F and other targets

This patch will make calendar plugin available for the targets with RTC.
1. I added button maps just like other plugins and fixed some codes.
2. use rb→memset to initialize struct variables.
3. draw lines if the LCD is large enough.
4. use List Widget to show memos and menu API for edit menu.

I tested this on my Gigabeat F21 and on the sim.
Sorry for my bad English.

Closed by  nls
2008-10-25 16:35
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

I committed the last patch with two small tweaks, thanks!

I will give this a test over the weekend.

Thanks.

teru commented on 2008-08-16 11:19

fix a bug that the display of month and year could overlap day.

teru commented on 2008-09-14 09:15

updated to revision 18510 and reverted some changes.

nls commented on 2008-10-24 09:03

This is definitely the right thing to do with this plugin.
The patch needs to be synced to svn though.

edit: using DRMODE_COMPLEMENT and fillrect works well on monochrome displays but
it is not so nice on color displays since it fills the rectangle with the complement
color which is often not the same as the background color.

teru commented on 2008-10-24 16:30

I made a patch without touching functions drawing calendar.
this patch does following things;
1.add button maps.
2.use List Widget to show memos.
3.use menu API for edit menu.
button maps needs to be tested.

Nils: then, what value should be set?
i don't understand well how the setting affects.

nls commented on 2008-10-25 10:21

An example can be seen in the "set time and date" screen which uses
DRMODE_SOLID|DRMODE_INVERSEVID for the selected item this will draw the
text with background and foreground colors reversed but will not extend beyond
the text drawn so to get the visual effect i think you should fill a rectangle
with the background color first and then draw the text with DRMODE_SOLID|DRMODE_INVERSEVID set.
I forgot to mention that the effect is only visible when not using black and white as
fg/bg since they are each other's inverse it is pretty visible if you use different colors though.

The patch works fine for me but i want to suggest a few changes to the menu.
1) The "New" item is confusing I understand that it's meant as a sort of subtitle but
this doesn't work so well with rockbox' lists imho. I would suggest to either remove the "New" item
and call the others "New Daily" etc, or make "New" open a submenu that contains the different types of memos.
2) The "Return" menu item should be removed so that this works consistently with other lists in rockbox, which
you exit by pressing left (or equivalent button).

Button maps look good to me, if something should be changed people will complain after it's committed ;)

edit: forgot to mention, the "set time and date" screen is in screens.c and around line 570 the drawing is done.

teru commented on 2008-10-25 15:39

here is a updated patch.
Thank you for your suggestions.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing