TrenchBroom / TrenchBroom

Cross-Platform Level Editor
kristianduske.com/trenchbroom
GNU General Public License v3.0
1.88k stars 224 forks source link

Suggestion: Increment sequential targetnames / targets when dragging #4554

Open Eli5723 opened 3 weeks ago

Eli5723 commented 3 weeks ago

I want to implement a feature to assist in the creation of linked entities such as paths. My proposal is that:

If this seems reasonable, I can throw together a draft MR.

kduske commented 3 weeks ago

Have you seen this issue: https://github.com/TrenchBroom/TrenchBroom/issues/1167?

kduske commented 3 weeks ago

Besides, I'm not sure that this behavior is what we want to have as a default. I would expect there to be many situations where this behavior is not wanted.

Eli5723 commented 3 weeks ago

I saw this existing issue, it's a similar ask: #3803 I think requiring the condition of having an existing entity already pointing to the target could help mitigate this. Additionally, a toggle could be implemented in the UI for the behavior. I do also like the idea of the #1167 copy increment UI, but I would consider that outside the scope of this request. (Though I am somewhat interested in trying to implement it) Cases

kduske commented 3 weeks ago

I think there too many situations where an automatic system will get in the way. I would rather see a solution that gives users more control. And on top of that, it should be extensible so that it can also solve the other requests for making entity links easier to create and maintain.

JafiMapping commented 1 week ago

I Think just but a underscore end of Targetname could do it. Ex. Targetname "Path_" you duplicate it and "bing" the second path( or a door or what ever) with targetname of Path_1, Path_2, Path_3..... But whitout it : Ex.: "Path" it duplicate the same name as "Path '. It could resolve the problem if we want to aply or not the incrementation.