feat: rendering colors in creator#85
Conversation
24bartixx
left a comment
There was a problem hiding this comment.
Displaying all colors of one object type in the select is not ideal. Once we add more colors (imagine 15 colors), finding the exact object with a desired color will be hell. Therefore, this solution will have to be replaced with the color options menu popup for the object (click the expand icon on the object --> choose color)
If you have time, you can already do it like this
If you don't have time, you can finish up the current solution and that would be fine as well.
For the future, it would be advisable to include screenshots in the PR description as well as logic and explanation behind decisions regarding the code and solution.
bnszky
left a comment
There was a problem hiding this comment.
Add suggestions and fix bugs mentioned in the review. One more important feature:
Please adjust the color method to what is available in Phaser. We use in Phaser:
if (isColorized) {
const colorInt = Phaser.Display.Color.HexStringToColor(color).color;
this.sprite.setTint(colorInt);
}
In this code, color all pixels the same. We want to make this visibility the same:
Current difference:
Before or after implementing all suggestions and improvements, resolve conflicts and prepare this branch to merge
|
I resolved all the mentioned issues to fasten deployment. I added rotation for cables as well |



made the button and door colors generate in the editor, all the colors are created and stored in global variable