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

-Add LayerBrush which edits all Tilemaps sharing the same Grid #257

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marcus-robinson
Copy link

Please add layer brush to master branch and include in the next release.

@ChuanXin-Unity
Copy link
Collaborator

Hi, we would like to work on this feature a little more before adding it for release. If you could share some suggestions on how you would like to use this feature or improve it, do let us know! Thanks!

@marcus-robinson
Copy link
Author

@ChuanXin-Unity I'm using the LayerBrush to move around entire areas of my grid - multiple layers of tiles that form a scene. This is a crucial use case for anyone building out large levels with tilemaps. It's working just fine in its current form for my use case, which is why I raised the PR, since right now, I have to copy/paste the brush file into the 1.5.0-preview release which is inconvenient.

@ChuanXin-Unity
Copy link
Collaborator

Thanks for the feedback! We will work on getting it into master and into the main package.

In the meantime, you could add this file and its corresponding metafile directly into your project to use it while using the package directly. Mixing and matching the scripts is possible as long as there are no duplicates between the project and the package.

@brogan89
Copy link

brogan89 commented Sep 3, 2021

I second this. It's a very useful feature. I merged it into master fine in my fork, then you can keep up to date with other features as well.

One feature that might be handy is having it toggle on with a shortcut key i.e from the default brush holding down ctrl or something to use the Layer Bush. Mainly because sometimes I forget I'm using that brush and I end up painting tiles into wrong layers etc. But that could just be me.

@ChuanXin-Unity
Copy link
Collaborator

Thanks for the comment! If you have any other suggestions or feedback on the Layer Brush, do let us know about it!

@GitFlip
Copy link

GitFlip commented Sep 26, 2022

Thanks for the comment! If you have any other suggestions or feedback on the Layer Brush, do let us know about it!

This PR has been opened for over two years. Is there a status on what's preventing it from being merged into master?

@ChuanXin-Unity
Copy link
Collaborator

@GitFlip For a new feature to go in, it has to be validated by the various stakeholders within Unity. For example, a product manager would like to validate that this is a viable product for users to use; a UX designer will need to check if the user experience is right, etc. So far, this has not been approved by these stakeholders as viable, but having feedback from you and other users can help bring prominence to the stakeholders for approval.

@GitFlip
Copy link

GitFlip commented Oct 13, 2022

@GitFlip For a new feature to go in, it has to be validated by the various stakeholders within Unity. For example, a product manager would like to validate that this is a viable product for users to use; a UX designer will need to check if the user experience is right, etc. So far, this has not been approved by these stakeholders as viable, but having feedback from you and other users can help bring prominence to the stakeholders for approval.

I appreciate this response a lot. Any chance you can provide details on why it's not viable? What updates are required? Can you leverage on the community more then to make this viable? I think it's just not clear to us what we can do to help.

@ChuanXin-Unity
Copy link
Collaborator

ChuanXin-Unity commented Oct 23, 2022 via email

@alipha
Copy link

alipha commented Jun 8, 2023

I also find this layer brush very useful. For instance, I'm working on a top-down 2d rpg and am creating buildings for a town. A building is composed of multiple layers (at least 3). Now, if I decide I think the building would look better at a different location, I have to move each layer one at a time. With this brush, I can move all the layers which compose this house at the same time.

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.

5 participants