Skip to content

Conversation

@BenHetherington
Copy link

This adds an option to show a 3x4 grid of 3:1 banners in the game selection menu, allowing game banners to be displayed without needing them to be stretched.

This can be configured using a menu_grid_layout .ini setting; set this to square_icons for the existing layout, and to banners for this new layout. The default is square_icons (the status quo).

This replaces the magic number of ‘8’ icons per row with a variable, allowing it to be modifiable.
This allows it to either be set to `square_icons` for the existing 8x4 grid of square icons, or `banners` for the new 3x4 grid of banners (which have an aspect ratio of 3:1).

If not specified, it’ll default to the square icons (the current behaviour).
This adds support for showing a 3x4 grid of banners (at 3:1 aspect ratio), when this grid type is set in the .ini file.

To show them at this aspect ratio, it just scales the cubes used for the icons by 3 on the x axis.
This replaces `ASSETS_PER_LINE` with the `columns_per_line` variable, and adds new variables to replace `ASSETS_PER_PAGE` and `ASSETS_INITIAL_COUNT`.
@BenHetherington
Copy link
Author

While tinkering with this, I had another idea for how this could be laid out – by scaling the banners by 0.75x, it's possible to fit a 4x4 grid on-screen (leaving more vertical space between rows), which might be preferable for those with a lot of games.

I'll add this as an extra commit, which can be used by setting menu_grid_layout to small_banners – let me know if you'd rather split this commit out into a separate PR!

While tinkering with grids of banners, I had another idea for how to lay it out. By shrinking the grid elements by 0.75x, it’s possible to fit an additional banner per row (leaving more vertical space between rows). This might be preferable for those with larger libraries.

This option can be used by setting the `menu_grid_type` .ini setting to `small_banners`.
# Conflicts:
#	cubeboot/source/main.c
#	patches/source/grid.c
# Conflicts:
#	cubeboot/source/main.c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant