andrei-drexler / ironwail

High-performance QuakeSpasm fork
GNU General Public License v2.0
530 stars 49 forks source link

Better descriptors for "always run" options #221

Open aoanla opened 1 year ago

aoanla commented 1 year ago

(I know this is inherited from QuakeSpasm, but there's no reason we can't make it better here)

Currently, the option names for the "always run" option in the menu are rather obscure - to the extent that in a recent discussion in the Quake mapping discord, it became clear that even rather experienced members of the community didn't understand what the differences were. This was the cause of some confusion regarding unexpected difficulties in replicating speedrunning techniques in challenge maps, which could probably be avoided if things were clearer.

I'm not sure if we can add tooltips to options, but another possibility would be renaming them to make it clearer that the "vanilla" option is not the same as "holding down run/+speed", and that the "quakespasm" option is the same as "holding down run/+speed (and makes the run button "walk" instead)".

daftmugi commented 1 year ago

Yep, that's a bit awkward.

The current settings are as follows:

Setting "Always Run", with options:
off          (cl_alwaysrun 0, cl_forwardspeed 200, cl_backspeed 200)
vanilla      (cl_alwaysrun 0, cl_forwardspeed 400, cl_backspeed 400)
quakespasm   (cl_alwaysrun 1, cl_forwardspeed 200, cl_backspeed 200)

Perhaps changing it to the following would be more clear?

Setting "Always Run", with options:
off          (cl_alwaysrun 0)
on           (cl_alwaysrun 1)

Along with changing the +speed binding description from "Run" to "Run / Walk".