Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a one pixel gap between each line in textboxes #1124

Merged
merged 2 commits into from
Feb 1, 2024
Merged

Conversation

TerryCavanagh
Copy link
Owner

Changes:

This change adds a one pixel gap between each line in textboxes, though only in the main game for now. It addresses this issue: #1051

A few screenshots of how it looks:

screen1

screen2

screen3

So! There was a lot of discussion about whether or not this change was a good idea on the discord, which is very fair - it does change the look of the game pretty noticeably, and it breaks the C64 8x8 grid. I found myself going back and forth on it a little bit - so I figured the best thing to do was just to try it, and make it a bit less hypothetical. I'll distribute a compiled version on the discord for easy testing.

Now that I've done it, I really like it! I actually find it quite jarring to go back to the zero gap textboxes in custom levels:

image

What do you think?

Legal Stuff:

By submitting this pull request, I confirm that...

  • My changes may be used in a future commercial release of VVVVVV
  • I will be credited in a CONTRIBUTORS file and the "GitHub Friends"
    section of the credits for all of said releases, but will NOT be compensated
    for these changes

@InfoTeddy InfoTeddy added the localization Implies 'required for 2.4' unless otherwise noted label Jan 17, 2024
@TerryCavanagh TerryCavanagh merged commit 8b73a38 into master Feb 1, 2024
7 checks passed
@TerryCavanagh TerryCavanagh deleted the linegap branch February 1, 2024 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
localization Implies 'required for 2.4' unless otherwise noted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants