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

Simple object type property library #367

Closed
Bartvds opened this issue Feb 19, 2013 · 6 comments
Closed

Simple object type property library #367

Bartvds opened this issue Feb 19, 2013 · 6 comments
Labels
feature It's a feature, not a bug.

Comments

@Bartvds
Copy link

Bartvds commented Feb 19, 2013

I have a proposal for a library panel where we can manage object types.

The modification works with the current object types and the new panel allows us to manage the properties of each type.

The library value could/would be exported separately from the instances so we can look-up/ the values (maybe externalize file data etc). I don't think it should write the library values to each instance, that'll give ugly name collision issues.

As bonus: for tileobject library types we should edit the registration point of the graphic so we can make arrows etc.

I think this could be a relatively simple way for a library expansion: almost all the basic parts are there, we just need the panel and import/export.

@bjorn
Copy link
Member

bjorn commented Feb 19, 2013

Did you notice issues #24, #70 and #365? Sure, Tiled definitely needs something like this.

@Bartvds
Copy link
Author

Bartvds commented Feb 19, 2013

Yes, and I did but they seemed more involved and I didn't want to press on your time. A simple solution would already be good.

@bjorn
Copy link
Member

bjorn commented Feb 19, 2013

There's no need to worry about pressing on my time. There's not much to press on, really. :/

Basically, even if this solution sounds simple, it involves new UI and data models and that's never a quick job. I think issue #365 has a good chance of going through since that solution requires no new UI to be written. It will at first just rely on an external XML file in which you define your object types and their properties. The other reason is that somebody actually seems to have time to work on it.

@ghost
Copy link

ghost commented Feb 19, 2013

@Bartvds I will definitely have to implement this feature and then I can have a look at writing a good solution for the normal version of Tiled. Once I work through @bjorn 's code and get something up I could suggest a pull request. So far you can have a look at my own issue #365 and see what I did so far. Please just keep in mind that this is still experimental and both a bad style of writing code and not robust. Yet it is a beginning 😄

@ghost
Copy link

ghost commented Feb 20, 2013

@Bartvds I have been successful implementing what we are looking for. Please refer to the newest comments in #365. I will be working these days on providing this feature to the master branch.

@Bartvds
Copy link
Author

Bartvds commented Feb 20, 2013

Nice! The XML solution is fine, solves the main issue of managing recurring data. Any fancy options are ice but not the core problem.

Good to see you and Bjorn work together and try to add it to master! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature It's a feature, not a bug.
Projects
None yet
Development

No branches or pull requests

2 participants