Opened 8 years ago

Last modified 5 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 8 years ago by erikos

  • Summary changed from Set the Sugar cursor to Set the Sugar cursor in a non racy way

comment:2 Changed 8 years ago by erikos

  • Owner set to erikos
  • Status changed from new to assigned

comment:3 Changed 7 years ago by erikos

  • Milestone changed from 0.96 to 0.98

Moving out as we have more pressing needs.

comment:4 Changed 7 years ago by humitos

  • Cc humitos added

comment:5 Changed 7 years ago by erikos

  • Milestone changed from 0.98 to 1.0

Moving out, not needed for 0.98.

comment:6 Changed 7 years ago by dnarvaez

I posted a patch for this on the mailing list.

comment:7 follow-up: Changed 6 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 6 years ago by dnarvaez

  • Milestone changed from 1.0 to Unspecified

comment:9 Changed 6 years ago by dnarvaez

  • Bug Status changed from New to Unconfirmed

comment:10 Changed 6 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 6 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:13 Changed 6 years ago by dnarvaez

  • Milestone changed from Unspecified to 0.102

comment:14 Changed 5 years ago by godiard

  • Priority changed from Urgent to Normal

comment:15 Changed 5 years ago by godiard

  • Priority changed from Normal to Low
Note: See TracTickets for help on using tickets.