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

HealLoction Event #68

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

HealLoction Event #68

wants to merge 7 commits into from

Conversation

garakmon
Copy link
Contributor

@garakmon garakmon commented Aug 10, 2018

Adds EventType::HealLocation and icon (#36 ).

The user can move existing heal locations on existing maps but cannot add or replace them.

  • requires:
    pokeruby commit dcf8c33a3b48ede46cf7e96491c0dbc2eb6c8073
    pokeemerald commit 18d518663061cdb8850d3e41d3d5acae8de7e049

garakmon added 5 commits July 29, 2018 17:22
update fork with source
Changes:
- disallow deleting and insertion of heal locations for existing maps
- save heal location constants to new headers
- added index to HealLocation struct
- allow constants for multiple heal locations in same map
@garakmon
Copy link
Contributor Author

garakmon commented Aug 13, 2018

demo:
Alt text

event.cpp Outdated
@@ -150,6 +163,21 @@ QString Event::buildWarpEventMacro(QMap<QString, QString> *mapNamesToMapConstant
return text;
}

HealLocation Event::buildHealLocationEventMacro()
Copy link
Contributor

Choose a reason for hiding this comment

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

This function's name is not correct. Should be just buildHealLocation?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yep...

@garakmon
Copy link
Contributor Author

garakmon commented Sep 7, 2018

changed it

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