Opened 12 years ago
Last modified 9 years ago
#3204 assigned defect
Set the Sugar cursor in a non racy way
| Reported by: | erikos | Owned by: | erikos |
|---|---|---|---|
| Priority: | Low | Milestone: | 0.102.0 |
| Component: | Sugar | Version: | Git as of bugdate |
| Severity: | Major | Keywords: | |
| Cc: | humitos | Distribution/OS: | Unspecified |
| Bug Status: | New |
Description
We have a long history of setting the Sugar cursor with many races.
1) First, we tried to set the gnome cursor_theme to use the sugar one: http://git.sugarlabs.org/sugar/mainline/commit/23516911ebc21e9714f662a6c47cf385660a88b0
2) Because this broke the GNOME cursor when logging into GNOME we did reset the cursor once we logged into Sugar "xsetroot -cursor_name left_ptr" #1433 http://git.sugarlabs.org/sugar/mainline/commit/293a3fc5a11fb4309eba4414702570b778508f4f
3) We moved this code around since the "Alt+Space" shortcut was not working anymore, we do set the the cursor once the window-manager-changes for the shell window [#1476 http://git.sugarlabs.org/sugar/mainline/commit/b0520607503b0876020d5e764b1a98322a2bf949
4) we once more moved the code to supress a race condtion: #1371 http://git.sugarlabs.org/sugar/mainline/commit/26ba76b965984ddc8681a3c25ee075a6664c98ef
5)wrong cursor on welcome screen #10712
Let's try to fix this for good.
Change History (15)
comment:1 Changed 12 years ago by erikos
- Summary changed from Set the Sugar cursor to Set the Sugar cursor in a non racy way
comment:2 Changed 12 years ago by erikos
- Owner set to erikos
- Status changed from new to assigned
comment:3 Changed 11 years ago by erikos
- Milestone changed from 0.96 to 0.98
comment:4 Changed 11 years ago by humitos
- Cc humitos added
comment:5 Changed 11 years ago by erikos
- Milestone changed from 0.98 to 1.0
Moving out, not needed for 0.98.
comment:6 Changed 11 years ago by dnarvaez
I posted a patch for this on the mailing list.
comment:7 follow-up: ↓ 11 Changed 10 years ago by dnarvaez
My patchset doesn't apply after not being reviewed for months. Redoing it all would take some point, but perhaps it's easier to apply just this bit.
comment:8 Changed 10 years ago by dnarvaez
- Milestone changed from 1.0 to Unspecified
comment:9 Changed 10 years ago by dnarvaez
- Bug Status changed from New to Unconfirmed
comment:10 Changed 10 years ago by dnarvaez
- Bug Status changed from Unconfirmed to New
- Priority changed from Unspecified by Maintainer to Urgent
comment:11 in reply to: ↑ 7 Changed 10 years ago by erikos
Replying to dnarvaez:
My patchset doesn't apply after not being reviewed for months. Redoing it all would take some point, but perhaps it's easier to apply just this bit.
Can you point me to "this bit"? Thanks :)
comment:12 Changed 10 years ago by dnarvaez
comment:13 Changed 10 years ago by dnarvaez
- Milestone changed from Unspecified to 0.102
comment:14 Changed 9 years ago by godiard
- Priority changed from Urgent to Normal
comment:15 Changed 9 years ago by godiard
- Priority changed from Normal to Low
Moving out as we have more pressing needs.