Opened 12 years ago

Last modified 9 years ago

#856 closed bug

Shop shouldn't honour keymaps — at Version 5

Reported by: KZK Owned by: takkaria
Milestone: 3.3.0 Keywords: stores InputRework

Description (last modified by takkaria)

Macros no longer exist so that's good, but keymaps still work in shops, as long as they are mapped to the keys wtkPbI{}~ or D. This needs fixing.

Change History (5)

comment:1 Changed 11 years ago by magnate

  • Keywords stores added

comment:2 Changed 11 years ago by magnate

  • Milestone changed from Triage to Future

Milestone set per #121

comment:3 Changed 10 years ago by magnate

  • Keywords InputRework added
  • Milestone changed from Future to 3.3.0
  • Owner set to takkaria
  • Status changed from new to assigned
  • Summary changed from Shop text overrun to Shop honours macros

Assuming that the text formatting issue is now part of #121 and this ticket is about macros and keymaps in stores. Keymaps are currently ignored in stores, but macros are not. I agree that macros should be ignored also. takkaria says that macros should be unnecessary after InputRework is finished ...

comment:4 Changed 9 years ago by takkaria

  • Description modified (diff)
  • Summary changed from Shop honours macros to Shop shouldn't honour keymaps


comment:5 Changed 9 years ago by takkaria

  • Description modified (diff)

In the short term: I think we're best just mapping these keys directly to the relevant UI functions.

OK, so in the long term the UI should find out what keys correspond to various bare actions (e.g. wtkPbI{}~) and ensure that they are usable in the stores and are not duplicated as menu items. This will be a bit difficult and isn't achiveable for 3.3, so I'll do the short-term fix and then punt to 3.4.

Note: See TracTickets for help on using tickets.