Opened 10 years ago

Closed 10 years ago

Last modified 6 years ago

#1300 closed defect (fixed)

Simple scheme for hidding ToolbarBox subpalettes

Reported by: alsroot Owned by: alsroot
Priority: Unspecified by Maintainer Milestone:
Component: Sugar Version: Git as of bugdate
Severity: Unspecified Keywords: r+
Cc: Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

|TestCase|

  • open Write
  • open palette(not subtoolbar) for Paragraph
  • change Bullet list combobox
  • move cursor away from subpalette
  • subpalette wasn't closed

The problem is, having followed lines in PaletteWindow:

    def __enter_notify_event_cb(self, widget, event):
        if event.detail != gtk.gdk.NOTIFY_INFERIOR and \
                event.mode == gtk.gdk.CROSSING_NORMAL:
            self.on_enter(event)

    def __leave_notify_event_cb(self, widget, event):
        if event.detail != gtk.gdk.NOTIFY_INFERIOR and \
                event.mode == gtk.gdk.CROSSING_NORMAL:
            self.on_leave(event)

there is not chance to detect leaving palette(and entering to combo) and leaving combo(and back entering to palette) in ToolbarBox._Palette, _Palette can catch only back entering.

Attachments (1)

sugar-1300.patch (2.2 KB) - added by alsroot 10 years ago.

Download all attachments as: .zip

Change History (6)

Changed 10 years ago by alsroot

comment:1 Changed 10 years ago by alsroot

Patched code could still fail due to common bug(#1301).

comment:2 Changed 10 years ago by tomeu

  • Keywords r+ added; r? removed
  • Owner changed from erikos to alsroot
  • Status changed from new to assigned

r+, some suggestions:

  • rename self._focus to self._has_focus
  • rename _handle_focus to _set_focus

comment:4 Changed 6 years ago by dnarvaez

  • Component changed from sugar-toolkit to Sugar

comment:5 Changed 6 years ago by dnarvaez

  • Milestone 0.86 deleted

Milestone 0.86 deleted

Note: See TracTickets for help on using tickets.