[sheets-] record key-col toggle for replay as key-col-on/-off

[midichef]

I'm following up on @saulpw's comment about trouble caused by keycol toggling in cmdlog replay.

it's tricky to change behavior based on existing column state, as this can make cmdlog replay a lot less predictable. We already have this with e.g. key-col and it's caused some problems. Maybe the ultimate answer is to have key-col put a more precise command like key-col-on or key-col-off (please help come up with better longnames if we do this) on the cmdlog.

Does the problem with key-col for cmdlog-replay still exist? If so, this PR should fix it, by recording key-col-on or key-col-off instead of key-col.

I'm preparing to submit the feature proposed in that Discussion: changing a column's sort direction while preserving other column's sort direction. So if this kind of fix is still needed for key-col, then I'll apply the same idea to sort-asc and the commands like it.

[saulpw]

It hadn't quite occurred to me to do it this way, but you know what, I'll take it. Nice job.

[midichef]

Hmm, this PR still needs work. I hadn't thought about the case of macros.

With this change, when key-col is used in a macro, the macro will contain two commands, the original key-col and key-col-on (or -off). So that needs a fix.

But there's also a UI difference between macros and replay logs that makes this approach difficult. Users may expect ! in a macro to perform toggling. Whereas in a cmdlog replay file that's only replayed once, what is desired is setting the keycol state, not toggling.