Keys can shift the screen #537

Merged
NQNStudios merged 6 commits from screen-shift-keys into master 2025-01-22 14:28:28 +00:00
NQNStudios commented 2025-01-19 01:11:26 +00:00 (Migrated from github.com)

This makes it so arrow keys/numpad can control screen shift when it's available.

When this preference is enabled, #482 will not be an issue (Fix #482). But the player will also miss out on cases where adjacent targeting would be a good thing. I haven't done anything smart to determine whether one mode or the other is better on a case-by-case basis.

This makes it so arrow keys/numpad can control screen shift when it's available. When this preference is enabled, #482 will not be an issue (Fix #482). But the player will also miss out on cases where adjacent targeting would be a good thing. I haven't done anything smart to determine whether one mode or the other is better on a case-by-case basis.
CelticMinstrel (Migrated from github.com) reviewed 2025-01-20 14:17:49 +00:00
@@ -41,7 +41,13 @@
<led name='slow' relative='pos pos-in' anchor='med' top='0' left='15'>Slow</led>
<led name='snail' relative='pos pos-in' anchor='slow' top='0' left='15'>Quite Slow</led>
</group>
CelticMinstrel (Migrated from github.com) commented 2025-01-20 14:17:47 +00:00

I think this should make it more clear that it's swapping one capability with another. Something like "Directional keys shift… instead of…".

I think this should make it more clear that it's swapping one capability with another. Something like "Directional keys shift… instead of…".
NQNStudios (Migrated from github.com) reviewed 2025-01-20 15:14:12 +00:00
@@ -41,7 +41,13 @@
<led name='slow' relative='pos pos-in' anchor='med' top='0' left='15'>Slow</led>
<led name='snail' relative='pos pos-in' anchor='slow' top='0' left='15'>Quite Slow</led>
</group>
NQNStudios (Migrated from github.com) commented 2025-01-20 15:14:12 +00:00
Screenshot 2025-01-20 at 9 14 10 AM

How about this?

<img width="471" alt="Screenshot 2025-01-20 at 9 14 10 AM" src="https://github.com/user-attachments/assets/5867592f-9ccc-44b1-be05-6d8f3521ac04" /> How about this?
CelticMinstrel (Migrated from github.com) reviewed 2025-01-20 18:34:34 +00:00
@@ -41,7 +41,13 @@
<led name='slow' relative='pos pos-in' anchor='med' top='0' left='15'>Slow</led>
<led name='snail' relative='pos pos-in' anchor='slow' top='0' left='15'>Quite Slow</led>
</group>
CelticMinstrel (Migrated from github.com) commented 2025-01-20 18:34:33 +00:00

Better, but I think a different word than "squares" should be used. Maybe "tiles" or "spaces"?

And I didn't check what you set the default to be, but right now I think it should default to the legacy behaviour.

Better, but I think a different word than "squares" should be used. Maybe "tiles" or "spaces"? And I didn't check what you set the default to be, but right now I think it should default to the legacy behaviour.
NQNStudios (Migrated from github.com) reviewed 2025-01-20 18:51:21 +00:00
@@ -41,7 +41,13 @@
<led name='slow' relative='pos pos-in' anchor='med' top='0' left='15'>Slow</led>
<led name='snail' relative='pos pos-in' anchor='slow' top='0' left='15'>Quite Slow</led>
</group>
NQNStudios (Migrated from github.com) commented 2025-01-20 18:51:21 +00:00

done and done

done and done
NQNStudios (Migrated from github.com) reviewed 2025-01-20 18:56:49 +00:00
@@ -41,7 +41,13 @@
<led name='slow' relative='pos pos-in' anchor='med' top='0' left='15'>Slow</led>
<led name='snail' relative='pos pos-in' anchor='slow' top='0' left='15'>Quite Slow</led>
</group>
NQNStudios (Migrated from github.com) commented 2025-01-20 18:56:49 +00:00

@CelticMinstrel What if holding shift or option/alt performed the opposite function so both are always available?

@CelticMinstrel What if holding shift or option/alt performed the opposite function so both are always available?
NQNStudios (Migrated from github.com) reviewed 2025-01-20 18:57:04 +00:00
@@ -41,7 +41,13 @@
<led name='slow' relative='pos pos-in' anchor='med' top='0' left='15'>Slow</led>
<led name='snail' relative='pos pos-in' anchor='slow' top='0' left='15'>Quite Slow</led>
</group>
NQNStudios (Migrated from github.com) commented 2025-01-20 18:57:04 +00:00

The dialog could have a parenthetical text explaining this

The dialog could have a parenthetical text explaining this
CelticMinstrel (Migrated from github.com) reviewed 2025-01-22 00:22:00 +00:00
@@ -41,7 +41,13 @@
<led name='slow' relative='pos pos-in' anchor='med' top='0' left='15'>Slow</led>
<led name='snail' relative='pos pos-in' anchor='slow' top='0' left='15'>Quite Slow</led>
</group>
CelticMinstrel (Migrated from github.com) commented 2025-01-22 00:21:59 +00:00

Sure, why not. The only question would be which modifier to pick… what precedent do we have so far? There's also the option of Control/Command (that is, Command on Mac, Control on non-Mac).

Given how "shift" typically means "extend", it seems like that doesn't really fit this kind of use. I guess "alt" fits better. Not sure whether "control" fits.

Sure, why not. The only question would be which modifier to pick… what precedent do we have so far? There's also the option of Control/Command (that is, Command on Mac, Control on non-Mac). Given how "shift" typically means "extend", it seems like that doesn't really fit this kind of use. I guess "alt" fits better. Not sure whether "control" fits.
NQNStudios commented 2025-01-22 00:34:33 +00:00 (Migrated from github.com)

Hmm. I currently have it implemented it with Shift, which might make less sense but is functional.

Just now I tried changing it to Alt/Option, and ran into a problem. Targeting for Look mode is already checking for the Option key, which makes Look Target mode stay activated after targeting a spot.

Hmm. I currently have it implemented it with Shift, which might make less sense but is functional. Just now I tried changing it to Alt/Option, and ran into a problem. Targeting for Look mode is already checking for the Option key, which makes Look Target mode stay activated after targeting a spot.
NQNStudios (Migrated from github.com) reviewed 2025-01-22 00:36:20 +00:00
@@ -41,7 +41,13 @@
<led name='slow' relative='pos pos-in' anchor='med' top='0' left='15'>Slow</led>
<led name='snail' relative='pos pos-in' anchor='slow' top='0' left='15'>Quite Slow</led>
</group>
NQNStudios (Migrated from github.com) commented 2025-01-22 00:36:20 +00:00

replied to this out-of-thread (as a new comment)

replied to this out-of-thread (as a new comment)
Sign in to join this conversation.
No description provided.