Opened 8 years ago

Closed 8 years ago

#4795 closed defect (fixed)

shortcut keys f1 f2 f3 ineffective after startup

Reported by: quozl Owned by:
Priority: Normal Milestone: Unspecified
Component: Sugar Version:
Severity: Minor Keywords:
Cc: godiard Distribution/OS: OLPC
Bug Status: New

Description

the shortcut keys for changing view are ineffective after startup of sugar.

  • scenario 1, immediately after second or subsequent boot,
    • workaround: move the cursor off the centre icon palette,
    • reproducer: restart Sugar, and wait for the display to settle,
  • scenario 2, immediately after first boot,
    • workaround: move the cursor off the centre icon palette, and click on the background of the home view,
    • reproducer: remove the /home/olpc/.sugar directory and contents, restart Sugar, choose nick, colour, gender, age, and wait for the display to settle.

Change History (6)

comment:1 Changed 8 years ago by quozl

  • Cc godiard added

@godiard, any connection between this one and #4866?

comment:2 Changed 8 years ago by godiard

comment:3 Changed 8 years ago by SAMdroid

Can not reproduce on sugar-runner/osbuild. How do you reproduce?

comment:4 Changed 8 years ago by quozl

I'm not surprised. sugar-runner/osbuild is a very different environment, and is not the environment used by the majority of users. The two scenarios have verified reproducers in the ticket description; but they do not use sugar-runner.

Perhaps the fastest method for you to reproduce might to spin up a local VM of Ubuntu and apply our packages. See http://wiki.sugarlabs.org/go/Ubuntu for instructions.

comment:5 Changed 8 years ago by quozl

On XO-1 with 13.2.4 build 16, Sugar 0.105.2 and kernel 4.1, reproduced scenario #2 by removing .sugar directory; problem happened as described in the ticket description.

Applied patch by removing lines from /usr/lib/python2.7/site-packages/jarabe/model/shell.py and repeated reproducer; problem has gone away, it was no longer necessary to click on the background of the home view.

Scenario #1 is also fixed.

Last edited 8 years ago by quozl (previous) (diff)

comment:6 Changed 8 years ago by Gonzalo Odiard

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

Don't check window type before change view - Fixes #4795

The code removed was introduced on commit 9406c42e to avoid change
views when the My Settings window was opened, but that didn't worked,
and we added the control of modal windows on the shell on 14956762
to solve that case.
But this control have the lateral effect of block change the views
when a popup is opened. Remove this control solves the problem.

Changeset: 8f119810d3fb29851e599c5c0ddab1b5b9fe0de9

Note: See TracTickets for help on using tickets.