Opened 7 years ago

Last modified 5 years ago

#2978 new defect

Remove hierachy-changed signal handlers when not needed

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

Description

Fixing OLPC #10930 we found the ToolButton has a signal handler for hierachy-changed because need set the accelerator when is attached to a activity. But the new toolbar parent and unparent the page every time is displayed/hidden.

We can remove the signal handler once the accelerator is set, but need check the case when the accelerator is changed (like in Terminal)

In the activities logs appear many times"WARNING root: No gtk.AccelGroup in the top level window." after fixing it we can remove the log, because is not useful. Or think a more efficient way of resolve this.

Change History (3)

comment:1 Changed 5 years ago by dnarvaez

  • Component changed from sugar-toolkit to Sugar

comment:2 Changed 5 years ago by dnarvaez

  • Type changed from enhancement to defect

comment:3 Changed 5 years ago by dnarvaez

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