Project

General

Profile

Actions

Bug #1496

closed

improve Help -> Controls section on Unit Movement:

Added by Dean Brown 6 days ago. Updated 3 days ago.

Status:
Closed
Priority:
Normal
Category:
client-common
Target version:
Start date:
06/10/2025
Due date:
% Done:

0%

Estimated time:

Description

Spun off from #1293.

Current version is

  1: move south-west\n\
  2: move south\n\
  3: move south-east\n\
  4: move west\n\
  6: move east\n\
  7: move north-west\n\
  8: move north\n\
  9: move north-east\n\

Patch for #1293  changes it to
  1,End:       move south-west\n\
  2,Down:      move south\n\
  3,Page Down: move south-east\n\
  4,Left:      move west\n\
  6,Right:     move east\n\
  7,Home:      move north-west\n\
  8,Up:        move north\n\
  9,Page Up:   move north-east\n\

But we want to note that some of these do not apply in some topologies, and better to say, e.g. "move down and left" instead of "move south-east".


Files

1496.patch (2.29 KB) 1496.patch Dean Brown, 06/10/2025 05:28 AM
1496First.patch (1.12 KB) 1496First.patch Dean Brown, 06/10/2025 05:45 AM
1496FirstCorrected.patch (1.13 KB) 1496FirstCorrected.patch Dean Brown, 06/10/2025 07:39 AM

Related issues 1 (1 open0 closed)

Blocks Bug #1293: Help -> Controls is wrong for qt clientNew04/14/2025

Actions
Actions #1

Updated by Marko Lindqvist 6 days ago

  • Related to Bug #1293: Help -> Controls is wrong for qt client added
Actions #2

Updated by Marko Lindqvist 6 days ago

  • Related to deleted (Bug #1293: Help -> Controls is wrong for qt client)
Actions #3

Updated by Marko Lindqvist 6 days ago

  • Blocks Bug #1293: Help -> Controls is wrong for qt client added
Actions #4

Updated by Marko Lindqvist 6 days ago

Better to get this one, a clearly defined one, done first, before we try to tackle what remains in #1293 (I suspect it, as well as #1488, will need further splitting)

Actions #5

Updated by Dean Brown 6 days ago

This patch, done after 1293.patch was done, makes it look like this -

Unit Movement:\n\
==============\n\
  (Some do not apply in some topologies)\n\n\
  1,End:       move down and left\n\
  2,Down:      move down\n\
  3,Page Down: move down and right\n\
  4,Left:      move left\n\
  6,Right:     move right\n\
  7,Home:      move up and left\n\
  8,Up:        move up\n\
  9,Page Up:   move up and right\n\

Actions #6

Updated by Dean Brown 6 days ago

1496First.patch done not after 1293.patch was done.

Actions #7

Updated by Marko Lindqvist 6 days ago

  • Target version set to 3.1.6
Actions #8

Updated by Marko Lindqvist 6 days ago

I would say "map topologies" instead of just "topologies", to make it clear "tileset topologies" are not the deciding factor.

Actions #10

Updated by Marko Lindqvist 5 days ago

  • Status changed from New to In Review
  • Assignee set to Marko Lindqvist
Actions #11

Updated by Marko Lindqvist 3 days ago

  • Status changed from In Review to Closed
Actions

Also available in: Atom PDF