Bug 200202

Summary:	broken keyboard mapping
Product:	[Fedora] Fedora	Reporter:	Ralf Corsepius <rc040203>
Component:	sturmbahnfahrer	Assignee:	Hans de Goede <hdegoede>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	5	CC:	bram, extras-qa
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2006-07-26 07:28:28 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Ralf Corsepius 2006-07-26 06:45:13 UTC

Description of problem:
As mention during the package review in
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=199310#c10


The "Examine keyboard controls" page says:
'Steer left  less than'
'Steer right greater than'
 
I found "left/right" on "," rsp. "." (German "qwertz" keyboard)

Version-Release number of selected component (if applicable):
sturmbahnfahrer-1.1-2.fc5

Comment 1 Hans de Goede 2006-07-26 07:28:28 UTC

Version 1.1-3, should show up on a mirror near you soon.
This version has the following changes which should fix this:
* Wed Jul 26 2006 Hans de Goede <j.w.r.degoede> 1.1-3
- Added a patch to call the left and right keys COMMA and POINT instead of
  LESSTHEN and GREATERTHEN, so that the text is correct for non qwerty
  keyboards too. And add a binding for normal brake to the Y key for more
  convenient controls with a German (QWERTZ) keyboard.
  Upstream has been notified that the current hardcoded keys are awkward
  with non qwerty keyboards so that upstream can create a proper fix.

Comment 2 Ralf Corsepius 2006-07-26 07:34:53 UTC

Hmm, this sounds like an ugly hack to me?

Doesn't the toolkit stormbahnfahrer applies (SDL?) provide proper keyboard
mapping routines?

Comment 3 Hans de Goede 2006-07-26 07:45:14 UTC

sturmbahnfahrer uses plib. And the code actually checks for ',' and '.' so the
new help text is correct. With regards to adding the Y binding that is because
the Y is on QWERTZ where the Z is on QWERTY. The problem is not as much keyboard
mapping which plib does fine AFAIK, the problem is that its best to use keys at
a certain location instead of keys with a certain symbol.

Comment 4 Bram Stolk 2006-07-26 16:52:03 UTC

The next version of sturmbahnfahrer will come with configurable key controls.
I never liked the in-game "change controls" options, so I will opt for a 
config file or something. (env var even?)

Also, I will make sure that in addition to configured keys, the 
cursor keys will work as well.

  Bram