Skip to content

grasslands map + asset reorganization#12

Open
M4DM4N56 wants to merge 4 commits intomainfrom
grasslands-map
Open

grasslands map + asset reorganization#12
M4DM4N56 wants to merge 4 commits intomainfrom
grasslands-map

Conversation

@M4DM4N56
Copy link
Copy Markdown
Collaborator

  • added the grasslands map tileset

  • nearly halved the grasslands tileset

    • with such a drastic change to the tileset, i had to recreate the grasslands area by hand, aside from the random floor it is 100% accurate to the old one so there shouldn't be any issues there
    • removed "1-wide" and "1-thin" walls
    • these tiles were never in use. this cut will be a little limiting in terms of map creation, but tbh the "1-wide/thin" walls were never truly complete and fully completing the tile set would make it about 4x bigger than the dungeon one, which is too much work atm

[POSSIBLY REALLY BAD PLEASE REVIEW THIS]

  • renamed several files
    • i renamed all the tsx/tmx/png asset files to all follow the same naming convention
    • i tested it and the dungeon still works, i just hope nothing else broke from this
    • i dont really know how or where all the map loading stuff is so im hoping one of you guys can check this out and make sure i didnt do anything too bad.
    • moved the unused dungeon tsx/tmx files into an 'outdated' folder for cleanlines

@M4DM4N56 M4DM4N56 added documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed labels Jul 15, 2024
@M4DM4N56 M4DM4N56 self-assigned this Jul 15, 2024
Copy link
Copy Markdown
Collaborator

@swatkinson swatkinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks super good :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants