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

Color as a property type #1275

Closed
superqix opened this Issue May 10, 2016 · 6 comments

Comments

Projects
None yet
3 participants
@superqix

superqix commented May 10, 2016

It's low priority, but would be nice to have a custom property that pulled up the color selector. Right now we type HEX colors into a string field.

@bjorn bjorn added the feature label May 10, 2016

@bjorn

This comment has been minimized.

Show comment
Hide comment
@bjorn

bjorn May 10, 2016

Owner

This should be really easy to implement, so I'll make sure to get to this soon.

Also, thanks for being such a generous patron, as well as for your work on ponytiled! Eventually I hope to get around to trying out / evaluating the many Tiled support libraries out there, and I'd definitely like to check this one out as well.

Owner

bjorn commented May 10, 2016

This should be really easy to implement, so I'll make sure to get to this soon.

Also, thanks for being such a generous patron, as well as for your work on ponytiled! Eventually I hope to get around to trying out / evaluating the many Tiled support libraries out there, and I'd definitely like to check this one out as well.

@superqix

This comment has been minimized.

Show comment
Hide comment
@superqix

superqix May 10, 2016

No worries... Tiled is a tool that we use all the time. Thank you for your efforts.

superqix commented May 10, 2016

No worries... Tiled is a tool that we use all the time. Thank you for your efforts.

@bjorn bjorn closed this in c420513 May 13, 2016

@bjorn

This comment has been minimized.

Show comment
Hide comment
@bjorn

bjorn May 13, 2016

Owner

Let me know if you find any issues. :-)

Owner

bjorn commented May 13, 2016

Let me know if you find any issues. :-)

@superqix

This comment has been minimized.

Show comment
Hide comment
@superqix

superqix May 13, 2016

Sweet. Thanks.

superqix commented May 13, 2016

Sweet. Thanks.

@ficion

This comment has been minimized.

Show comment
Hide comment
@ficion

ficion May 28, 2016

This is a really small issue, but I would like to add that instead of "color", it might be better called "rgba", since that's the way the color seems to be saved.

ficion commented May 28, 2016

This is a really small issue, but I would like to add that instead of "color", it might be better called "rgba", since that's the way the color seems to be saved.

@bjorn

This comment has been minimized.

Show comment
Hide comment
@bjorn

bjorn May 30, 2016

Owner

This is a really small issue, but I would like to add that instead of "color", it might be better called "rgba", since that's the way the color seems to be saved.

Hmm, I don't see why the storage format should determine the name of the property type. It could be stored in many ways, but it's still a color. The format is #AARRGGBB actually, which will be written in the documentation.

Owner

bjorn commented May 30, 2016

This is a really small issue, but I would like to add that instead of "color", it might be better called "rgba", since that's the way the color seems to be saved.

Hmm, I don't see why the storage format should determine the name of the property type. It could be stored in many ways, but it's still a color. The format is #AARRGGBB actually, which will be written in the documentation.

@bjorn bjorn referenced this issue Jul 27, 2016

Closed

Color Property #1324

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