kevedit icon indicating copy to clipboard operation
kevedit copied to clipboard

Inconsistent handling of tiles without stats

Open computirman opened this issue 1 year ago • 4 comments

This is a request to reverse the following addition in the latest Kevedit:

https://github.com/cknave/kevedit/blob/2311ac98bcdd28ab33120af4018d5adf127c7cb9/src/kevedit/kevedit.c#L857

It is an expectation among Kevedit users that all tiles without stats are pulled into the buffer with ENTER, and no dialog boxes appear when this happens. The great majority of the time a user is pressing ENTER on a text tile, it is to put chars 220, 221, 222 and 223 (the most common text tiles used in ZZT games by far) into the buffer to place them as decoration on a board. Adding an additional dialog to the process will severely impact ZZTers workflow.

In general, this is a valuable expectation to maintain: if it has no stats, pressing ENTER pulls it into the buffer with no fuss.

computirman avatar Nov 17 '23 06:11 computirman

Oh that's a good point, I didn't think this would break existing workflows. I'll revert to the previous behavior for text tiles in the next one.

cknave avatar Nov 17 '23 18:11 cknave

I wouldn't mind the character window being tied to CTRL+T, myself.

asiekierka avatar Nov 17 '23 18:11 asiekierka

I wouldn't mind the character window being tied to CTRL+T, myself.

Same. I currently use Ctrl+T and have the color/char correlation memorized for several regions (220-223, letters/numbers) but if I want to do any work with the linewall characters I'm currently OOL. It would be great to be able to use the char selector in the Ctrl+T dialog when choosing what character to make text.

computirman avatar Nov 17 '23 19:11 computirman

The previous behavior was restored in 8ed72d56eae61a99ecace6fef604f1e31ea1abb7, and the tile info dialog shows the character selector for text tiles in 150f63dfbc67e3c366a6e42925d9d2a947e38e2b. Build 56 for as long as GitHub keeps it around.

cknave avatar Nov 19 '23 16:11 cknave