Opened 7 years ago

Closed 6 years ago

#5060 closed defect (fixed)

Fn-space does not generate XF86XK_Start

Reported by: bert Owned by: sayamindu
Priority: high Milestone: 8.2.0 (was Update.2)
Component: x window system Version:
Keywords: blocks-:8.2.0 Cc: mstone, cscott, kimquirk, gregorio, bernie
Blocked By: Blocking: #6781
Deployments affected: Action Needed: test in release
Verified: no

Description

... hence "view source" is broken (with bad effects in general #452, etoys #2132, browse #4853).

Attachments (1)

5060.patch (312 bytes) - added by sayamindu 6 years ago.
Workaround

Download all attachments as: .zip

Change History (21)

comment:1 Changed 7 years ago by bert

tested broken using xev in update1-639

comment:2 Changed 7 years ago by jg

  • Milestone changed from Never Assigned to Update.1

comment:3 Changed 7 years ago by jg

  • Keywords Update.1? removed
  • Owner changed from jg to bernie

comment:4 Changed 7 years ago by bernie

  • Owner changed from bernie to ApprovalForUpdate

To fix it, please tag libX11-1.1.3-3.git20071123.olpc2 from koji. It's been in joyride for a while and it contains the missing keysym.

comment:5 Changed 7 years ago by jg

  • Owner changed from ApprovalForUpdate to dgilmore

Approved.

comment:6 Changed 7 years ago by dgilmore

  • Owner changed from dgilmore to bert

this is already in update.1 663

comment:7 Changed 7 years ago by bert

  • Resolution set to fixed
  • Status changed from new to closed

yep, works.

comment:8 Changed 7 years ago by bert

  • Resolution fixed deleted
  • Status changed from closed to reopened

On a B4 at build 691 with an arabic (Libyan?) keyboard fn-space does not work. At the same build it works on a MP machine with English keyboard.

comment:9 Changed 7 years ago by bert

  • Owner changed from bert to bernie
  • Status changed from reopened to new

comment:10 Changed 7 years ago by bert

Here's an xev log:

KeyPress event, serial 27, synthetic NO, window 0x1000001,
    root 0x3d, subw 0x0, time 2421333, (600,450), root:(600,450),
    state 0x0, keycode 216 (keysym 0x0, NoSymbol), same_screen YES,
    XLookupString gives 0 bytes: 
    XmbLookupString gives 0 bytes: 
    XFilterEvent returns: False

KeyPress event, serial 27, synthetic NO, window 0x1000001,
    root 0x3d, subw 0x0, time 2421558, (600,450), root:(600,450),
    state 0x0, keycode 236 (keysym 0x0, NoSymbol), same_screen YES,
    XLookupString gives 0 bytes: 
    XmbLookupString gives 0 bytes: 
    XFilterEvent returns: False

KeyRelease event, serial 27, synthetic NO, window 0x1000001,
    root 0x3d, subw 0x0, time 2421705, (600,450), root:(600,450),
    state 0x0, keycode 236 (keysym 0x0, NoSymbol), same_screen YES,
    XLookupString gives 0 bytes: 
    XFilterEvent returns: False

KeyRelease event, serial 27, synthetic NO, window 0x1000001,
    root 0x3d, subw 0x0, time 2421733, (600,450), root:(600,450),
    state 0x0, keycode 216 (keysym 0x0, NoSymbol), same_screen YES,
    XLookupString gives 0 bytes: 
    XFilterEvent returns: False

comment:11 Changed 7 years ago by bert

  • Blocking 6781 added

comment:12 Changed 6 years ago by bert

  • Action Needed set to never set
  • Cc mstone cscott kimquirk gregorio added
  • Keywords blocks?:8.2.0 added

This still happens in 8.2-759 on an arabic-keyboard B4, meaning view-source is broken with at least some keyboard layouts.

comment:13 Changed 6 years ago by gregorio

  • Keywords blocks-:8.2.0 added; blocks?:8.2.0 removed

comment:14 Changed 6 years ago by sayamindu

  • Action Needed changed from never set to diagnose
  • Cc bernie added
  • Owner changed from bernie to sayamindu
  • Status changed from new to assigned

Looks like anything that KP_XF86Start is not generated when more than two layouts are enabled. While the reason for this is not clear to me, for the time being, we can have a workaround where group(olpc) contains the appropriate mapping.

Bernie, does that make sense to you ?

Changed 6 years ago by sayamindu

Workaround

comment:15 Changed 6 years ago by cscott

This is in xkeyboard-config-1.3-4 and has been in joyride for while. Does it work? It is appropriate for the stable build?

comment:16 Changed 6 years ago by cscott

  • Action Needed changed from diagnose to approve for release

comment:17 Changed 6 years ago by cscott

  • Action Needed changed from approve for release to add to release

Approving for stable build 762 on my own authority (every one else is asleep! and the build must go out!).

comment:18 Changed 6 years ago by cscott

  • Action Needed changed from add to release to test in release

Packages added to stable repository in commit b586e68e for build 762. Please confirm the package versions are correct and test in stable build 762 or later.

comment:19 Changed 6 years ago by sayamindu

This is fixed.

comment:20 Changed 6 years ago by sayamindu

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.