Page 1 of 1

New MAPINFO property: ForceWiperEffect

Posted: Sun Jul 15, 2012 3:37 am
by hfc2x
Revising a previous sugestion I made here, I took my time and made a patch:
hubwipers.zip
(15.26 KiB) Downloaded 147 times
I added a new MAPINFO property: ForceWiperEffect
Setting this property for a map, forces the Wiper at exit even if the level belongs to a hub. The Wiper shown depends on the user settings (Melt by default for Doom and so on). I know this could be more polished, but it's a start. I used r3757 as a base, so it's possible that the patch could be outdated. Be advised that this is the first patch I've ever made, so if it does not work, please tell me and I will upload the modified source files instead (along with a text file pointing out the lines I added).

Re: New MAPINFO property: ForceWiperEffect

Posted: Sun Jul 15, 2012 1:37 pm
by hfc2x
Cleaned up the patch, so it doesn't include unnecesary file changes.

Re: New MAPINFO property: ForceWiperEffect

Posted: Sun Jul 15, 2012 3:27 pm
by Xaser
This is interesting, but it should really be extended with a parameter to pick the parameter specifying which wipe style to use (along with a "User" option that I suppose could serve as the default, making it possible to submit this now and patch it later and keep it backwards-compatible).

Re: New MAPINFO property: ForceWiperEffect

Posted: Tue Jul 17, 2012 5:20 pm
by Edward-san
sincerely I don't like the idea someone imposes anything different from my screen wipe style (none wins)...

Re: New MAPINFO property: ForceWiperEffect

Posted: Tue Jul 17, 2012 5:58 pm
by Xaser
"None" is exactly what I'd use it for -- inter-map transitions can have huge seams if a fade is inserted into the middle, and currently only hubs disable it. A "NoWipeEffect" could achieve that, though.

Re: New MAPINFO property: ForceWiperEffect

Posted: Tue Jul 17, 2012 6:45 pm
by NeuralStunner
Most users probably don't change it from the defaults. (Personally I use Burn for Heretic/Hexen, though it's not really seen in the latter much.)

Simplest case would probably be a flag for Cluster definitions, to enable transitions if it's a hub. (Or force them off for that cluster, perhaps.)

Or it could be could be a Map property:
  • Default (On for single maps, Off for hubs)
  • Off (No transitions at all)
  • On (Force user's chosen transition, even for hubs)
  • Melt, Fade, Burn (Force a specific transition)

Re: New MAPINFO property: ForceWiperEffect

Posted: Tue Jul 17, 2012 7:17 pm
by hfc2x
Edward-san wrote:sincerely I don't like the idea someone imposes anything different from my screen wipe style (none wins)...
I agree with Edward-san, and that's the reason I didn't give it a parameter to specify a wiper type. I think it would be annoying for someone to be forced to see a transition even if they have them set to "none" in their options. That's why there are no parameters. HeXen's MAPINFO doesn't need this property, but Strife surely does, as Vanilla Strife crossfades between levels (which the user can disable if he/she so desires). Anyways, I'm gonna see what I can do about parameters.

Re: New MAPINFO property: ForceWiperEffect

Posted: Tue Jul 17, 2012 11:00 pm
by hfc2x
Okay, adding parameters is a bit more complex than I first thought. So in the meantime, I added a "ForceNoWiper" property for maps, which disables the wiper transitions even if the map does not belong to a hub. "ForceNoWiper" takes a higher priority over "ForceWiperEffect", so if both are set for a map, "ForceWiperEffect" is ignored.
BTW, I also added "ForceWiperEffect" to Strife's MAPINFO, just in case :P