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

275 improve roadmap (DO NOT MERGE YET) #276

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JanekKermit
Copy link

@JanekKermit JanekKermit commented May 7, 2024

#275 Changelog:

  1. Most important changes to /pokete_classes/roadmap.py
  • enlarged displayed roadmap (61x17 instead of 40x11)
  • added new StationObject class (extension of se.Box)
  • class Station is now an extension of StationObject, not se.Square
  • added map decorations (they are like stations, but do not require full-scale map and it's impossible to get to them with WSAD). They are stored in a variable 'decorations' in /pokete_data/mapstations.py
  • changed method set_color() to hide_if_visited(), the new one hides features like villages, Shops etc if the place has not been visited.
  • added animations for water and cursor (now the indicator of the place where you are blinks making it more visible) [methods StationObject.animate_water() and Station.animate_blink()]
  1. Added new more detailed roadmap layout, stations colors, decorations at /pokete_data/mapstations.py
  2. Changed /pokete_classes/color.py to use 256 color palette (old system caused weird colors in some terminals, like one built-in VS Code)

Preview:

obraz

@lxgr-linux
Copy link
Owner

Changed /pokete_classes/color.py to use 256 color palette (old system caused weird colors in some terminals, like one built-in VS Code)

I wouldn't do that, I chose the smaller color pallet, since it will also be displayed on fullscreen tty

@lxgr-linux
Copy link
Owner

PXL_20240508_145533190.MP.jpg

Character wise, this is how the roadmap looks like on TTY, but the colors seam to work.

@JanekKermit
Copy link
Author

Alright, i think i know what the problem is. I'll install TTY and work on that

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.

None yet

2 participants