Opened 9 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
comment:2 Changed 8 years ago by godiard
No, but I found a fix https://github.com/sugarlabs/sugar/pull/546
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.
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
@godiard, any connection between this one and #4866?