"Move brushes to ..." should select all brushes in the resulting merged func

[ericwa]

Current behaviour is:

1. create two brushes, A is worldspawn, and make B a func_wall
2. cmd+click A, then B
3. right click and choose "move brushes to func_wall"
4. A will be selected, B is deselected

I'd prefer if the entire func_wall (A and B) were selected at this point (how TB1 behaved).

One reason to select all brushes in the merged func is, since the order of selections matters for determining what is merged into what, if you accidentally merge in the wrong direction (trying to merge a few brushes from entity X into entity Y, but instead you merge entity Y into entity X), you won't see it unless you double click to select all brushes in the merged func.

[kduske]

The reason for the current behavior is that it allows you to see what has actually changed.

[kduske]

Why do you have to select all brushes to see what has been merged into what?

[ericwa]

Why do you have to select all brushes to see what has been merged into what?

This applies to the use case when you have two existing func_details (say, buildings) and you want to transfer a few brushes from one to the other. Without selecting all brushes in the merged result, the only other visual hint telling you which direction the merge happened is the bounding box, which can be easy to miss. (in this case if you merge in the "wrong" direction, you'll end up with both buildings in a single func_detail)

[ericwa]

Fixed in a2248d8