Skip to content
This repository was archived by the owner on Oct 28, 2019. It is now read-only.

Support for generating skeleton for custom map objects - #14

Open
allejo wants to merge 20 commits into
masterfrom
feature/map-object-support
Open

Support for generating skeleton for custom map objects#14
allejo wants to merge 20 commits into
masterfrom
feature/map-object-support

Conversation

@allejo

@allejo allejo commented Apr 29, 2018

Copy link
Copy Markdown
Owner

Feedback on map object editor

I think if the provided ones had an explanation why they're there instead of just blocking you from editing them, it would help
I didn't know what I was looking at
also, a note that you click to edit the names or types
and an option to delete properties or arguments
basically, any field that's there but can't be edited needs to explain its presence, and any field that can be edited needs to let the user know that and also let the user know how to edit it

/cc @asinck

Still Left To Do

  • Add constructor definition to map object classes
  • Handle the for loop for parsing map object data and saving it into classes
  • Add vector for handling map objects in the main plug-in class
  • Add support for making optional map property arguments
  • Add support for deleting properties
  • Add support for deleting property arguments
  • Improve event emission for triggering a rebuild of the plug-in template

@allejo
allejo force-pushed the feature/map-object-support branch from 79e20d1 to 8d63288 Compare May 31, 2018 05:30
@allejo
allejo force-pushed the feature/map-object-support branch from 30ca1c6 to 2443925 Compare May 31, 2018 07:12
@allejo allejo added the WIP A PR that is currently in progress label Jul 24, 2018
@allejo allejo changed the title [WIP] Support for generating skeleton for custom map objects Support for generating skeleton for custom map objects Jul 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Enhancement WIP A PR that is currently in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant