Opened 9 years ago

Last modified 7 years ago

#5271 new defect

Sugar should track window property changes

Reported by: bert Owned by: marco
Priority: normal Milestone: 9.1.0-cancelled
Component: sugar Version:
Keywords: sugarization 9.1.0:? Cc: jg, bemasc, sayamindu, sascha_silbe
Blocked By: Blocking:
Deployments affected: Action Needed: never set
Verified: no


The Sugar-specific window properties currently must be set before opening the activity window, otherwise Sugar mistakes that window for a raw X app. This is difficult with many window toolkits that create and open a window in one go, with no chance to intervene. Activity authors are resorting to terrible hacks to work around this. Much better would be if Sugar tracked window property changes on open top-level "raw" X windows, so if an activity opens the window first and then sets the Sugar properties, it will still be recognized as an activity.

Change History (7)

comment:1 Changed 9 years ago by AlbertCahalan

Tracking property changes to implement the GUI simply can't be reliable. You have a race condition, and maybe even a (mild?) security problem.

The UI needs a mild redesign anyway. There are problems (bugs in trac too) with killing activities that don't start, timeouts (which will always be both too long and too short), and even just chewing up CPU time with the throbbing icon.

Starting the icon throb early is no good; there are alternate animations that could work. (example: icon flies up behind the donut, then comes through the hole if it shows up on the X server) For local connections on DISPLAY=:0.0, X can get lots of info about the client. It should do so, if it doesn't already. It should provide this info to sugar, if it doesn't already. That's enough for sugar to identify the client without needing racy cooperation from the client. For remote connections, things are fairly raw... but you can still ask for an icon (scale it as needed) and name.

comment:2 Changed 9 years ago by jg

  • Milestone changed from Never Assigned to Future Release

comment:3 Changed 9 years ago by bemasc

  • Cc bemasc added

comment:4 Changed 9 years ago by marco

  • Keywords 9.1.0:? added
  • Milestone changed from Future Release to 9.1.0

comment:5 Changed 9 years ago by cscott

  • Action Needed set to never set

We should be using the startup-notification-spec:

comment:6 Changed 9 years ago by sayamindu

  • Cc sayamindu added

comment:7 Changed 7 years ago by sascha_silbe

  • Cc sascha_silbe added
Note: See TracTickets for help on using tickets.