billyrayvalentine / asm68k-megadrive-slidedeck Goto Github PK

If seems that if the 32nd char is a newline then there is a wrap of only 31 chars when the display is 32.
The newline char should be ignored instead if the newline is present at the 33rd, not the 32nd - I think!

Ideally this should be fixed as part of #6 instead to allow for a 40 char wide display

This requires increasing the scroll plane size and updating the function that populates the plane table. This will require inserting blank tiles on every row as scroll plane width is > than visible width.

Background
The entire code base is un-linted and the style is a bit all over the place

Solution
Define or borrow a style guide.
Find a way to implement said style either manually or with a linter

The implementation for the de-bouncing of button presses is not perfect. Along with checking that a button has been pressed for a certain duration of frames it should also check that the button was released

Background
NTSC resolutions are much better supported across various hardware regions. For example, my NTSC X'eye will not run the current PAL resolution. PAL consoles will happily play NTSC (albeit at the wrong speed)
Also worth noting the extra 16 pixels (240-224) are usually not entirely displayed on most modern display when scaled.

Solution

• Disable PAL V30 and use NTSC V28

• Enable H40 to make up for the lost space
  This requires increasing the scroll plane size and updating the function that populates the plane table. This will require inserting blank tiles on every row as scroll plane width is > than visible width.

Need to create a demo slide with all the available ASCII characters

Background
The text parsing at present does no formatting other than spaces. This makes it painful to format text for a slide.

Solution
Implement more formatting controls such:

• LF for a newline

Background
The text parsing at present does no formatting other than spaces. This makes it painful to format text for a slide.

Solution
Implement text justify / word wrap.

There should be an option to disable it

Background
Image sizes per slide are fixed.

Solution
Allow images to be optional so that a full screen of text can be shown and allow for images to be of a specified size, leaving more or less room for text.