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 screen size macros #85

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add screen size macros #85

wants to merge 1 commit into from

Conversation

zenodante
Copy link

Hi, I added SCREEN_WIDTH and SCREEN_HEIGHT defines to the picosystem.hpp. Then the code in picosystem.cpp could be simplified a little.

@Gadgetoid
Copy link
Member

sigh the Python linting fails are my bad, I didn't pin the flake8 version so it's picking up things that previously didn't register as an error.

With that said, uh, do you mind... fixing them... and popping on another commit so I can merge this? 😆

test.py's update should be:

def update(tick):
    global view

    if pressed(RIGHT):
        view += 1

    if pressed(LEFT):
        view -= 1

    view %= view_count

@ahnlak ahnlak mentioned this pull request Sep 12, 2022
@Gadgetoid
Copy link
Member

The above fixes have been added in #87 - you should just be able to rebase this PR and it'll pass CI!

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.

2 participants