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

Jump to bottom

TrenchBroom V2.1.0-RC1 has crashed. #2368

Closed
jonathanlinat opened this issue Oct 20, 2018 · 6 comments
Closed

TrenchBroom V2.1.0-RC1 has crashed. #2368

jonathanlinat opened this issue Oct 20, 2018 · 6 comments
Assignees
@kduske
Labels
Prio:1 Highest priority: Crash or crippling bugs, features that enable new ways of working Type:Bug Errors and problems
Milestone
2.1.0

Comments

@jonathanlinat
Copy link
Contributor

System Information

TrenchBroom V2.1.0-RC1 on Windows 10

image

crash-files.zip
@jonathanlinat
Copy link
Contributor Author

jonathanlinat commented Oct 20, 2018 •
edited

This crash occurred when I've been attempting to rename at once all of the grouped brushes.

  1. I grouped and named each of the towers individually.
  2. I selected all of the towers.
  3. I tried to renamed all of the selected towers at once using the right-click rename option.
  4. TrenchBroom crashed.

@kduske kduske added this to the 2.1.0 milestone Oct 20, 2018
@kduske kduske added Prio:1 Highest priority: Crash or crippling bugs, features that enable new ways of working Type:Bug Errors and problems labels Oct 20, 2018
@kduske
Copy link
Collaborator

kduske commented Oct 20, 2018

I don‘t think this should work at all. It was never intended to rename multiple groups at once.

@jonathanlinat
Copy link
Contributor Author

jonathanlinat commented Oct 20, 2018 •
edited

Yep. The Rename option is supposedly not available for multiple groups at once if an entity is also selected. By the way, it was available in a certain condition that I cannot reproduce at this moment. I'll investigate.

@kduske
Copy link
Collaborator

kduske commented Oct 20, 2018

I have an idea what was going on. Also there‘s no reason really do disallow it for multiple groups other than ease of implementation, so maybe I‘ll just change that.

@kduske kduske assigned ericwa and kduske and unassigned ericwa Oct 26, 2018
@kduske
Copy link
Collaborator

kduske commented Oct 26, 2018

I was wrong, it's perfectly ok to rename multiple groups at once. No idea what went wrong in your case.

@kduske kduske mentioned this issue Oct 26, 2018
Merged
@kduske
Copy link
Collaborator

kduske commented Oct 26, 2018

Aha, I think I have it.

@ghost ghost removed the Status:Resolved label Oct 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kduske kduske

Labels
Prio:1 Highest priority: Crash or crippling bugs, features that enable new ways of working Type:Bug Errors and problems
Projects
None yet
Milestone
2.1.0
Development

No branches or pull requests
3 participants
@kduske @ericwa @jonathanlinat