Rockbox

  • Status Closed
  • Percent Complete
    0%
  • Task Type Patches
  • Category User Interface → Themes
  • Assigned To
    christi-s
  • Operating System
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by perholmang - 2005-07-01
Last edited by christi-s - 2005-07-05

FS#2556 - wps alignment patch

Patch to enable alignment settings in wps files.

Syntax:

%al - align left (defaults)
%ar - align right
%ac - align center

Put this before the item you wish to align. For example:

%al%ia - Left aligned Artist

If you want to use scrolling, use %s before the alignment:

%s%ac%ia - Center aligned scrolling Artist

Closed by  christi-s
2005-07-05 22:29
Reason for closing:  Accepted
Additional comments about closing:  

Logged In: YES
user_id=1077278

Commited to CVS. Thank you. When submitting future
patches, please ensure that they do not contain tab characters.

You don’t seem to have attached the patch.

Fixed now.

       switch (*fmt)

@@ -795,12 +798,30 @@

           case 0:

The following addition to wps-display.c appears to be
redundant. Is there a reason why you included this?

<pre> *buf++ = ‘%’;

               break;

-
+ case ‘(’:
+ ++fmt;
+ break;

           case 's':
               *flags |= WPS_REFRESH_SCROLL;

</pre>

No, I just forgot to clean that part up, it’s been removed
now, hopefully. I’ve never worked with diff/patch before…

Commited to CVS. Thank you. When submitting future
patches, please ensure that they do not contain tab characters.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing