Brush Scale Modifier

[OrbWeaver]

This came through on the GTKRadiant devel mailing list:

 

Hi.

 

this seems to be d3 game related and caused by

 

transform->setType(TRANSFORM_PRIMITIVE);

 

in selection.cpp line 2415 + 2419.

 

namespace schrieb:

> Here is another bug:

>

> - the scale modifier only works with patches, brushes collaps into a zero volume

>

 

Just posting here for reference.

[greebo]

What shall we do with the modifier, fix or remove it? I'm always using the QER-style modifier that can both scale and translate, so I would vote for removing it. Or should we wait till the GTKRadiant people fix it?

[OrbWeaver]

I don't see the benefit in a scale modifer myself -- you wouldn't normally want to scale something in three dimensions, you would just drag to resize in whichever view you were using, and do the third dimension separately if required.

 

I think we should scrap the Scale modifer but fix the Translate modifier to do as SneaksieDave suggested - to allow brush translation by dragging no matter whether you clicked inside or outside the brush.

[greebo]

Agreed. Let's remove it then, it's just a matter of editing the XML file anyway. We can leave the code for now, perhaps it gets fixed sometime by the GTKRadiant people. If the fix turns out to be un-appliable at that time, we can strip the code as well.

[sparhawk]

Greebo, you wrote the inventory, right? Can you take a look there:

http://forums.thedarkmod.com/index.php?showtopic=4781

 

Thanks!

[OrbWeaver]

I thought it was Gildoran that wrote the inventory.

[greebo]

Yes, it's Gildoran, I believe. I haven't started yet looking at the SDK, as DarkRadiant is keeping me busy, but I definitely will sometime in the future.

 

BTW, is Gildoran away for a while? Does anybody know?

[sparhawk]

Sorry! I confused you two.