Skip to content

some code refactor, terminal & layout changes #27

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jquast
Copy link

@jquast jquast commented Jan 8, 2021

  • properly setup backspace vs. delete (requires upgrade of blessed library)
  • use "bright_black" instead of "dim", dim has less compatibility (didn't work on whatever terminal I was using at the time)
  • create functional alias for print as "echo", which does not print a newline
  • center the crosswords puzzle horizontally on the screen
  • move "you win" message, it was occluding / printing over the puzzle

I wanted to make it easier to use a smaller terminal dimension, I think if we put the puzzle on the left or right side of the screen with some padding, there should be plenty of room for the information, hints, etc on the other side of the screen (using textwrap library if necessary), reducing the vertical requirement even further, the more the vertical height can be reduced, the larger the fontsize can be! I was also hoping to make it so you could resize the window during runtime. Anyway, this was what I was doing while looking for a job, found a job, now i'm busy, that's how it goes!

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