1. 13 May, 2013 1 commit
    • Allison Karlitskaya's avatar
      add GtkMenuTrackerItem · a4276a6c
      Allison Karlitskaya authored
      Add a new class, GtkMenuTrackerItem that represents a menu item, to be
      used with GtkMenuTracker.
      
      GtkMenuTracker's insert callback now works in terms of this new type
      (instead of passing reference to the model and an index to the item).
      
      GtkMenuShell now handles all of the binding tasks internally, mostly
      through the use of property bindings.  Having bindings for the label and
      visibility attributes, in partiular, will help with supporting upcoming
      extensions to GMenuModel.
      
      GtkModelMenu has been reduced to a helper class that has nothing to do
      with GMenuModel.  It represents something closer to an "ideal" API for
      GtkMenuItem if we didn't have compatibility concerns (eg: not emitting
      "activate" when setting toggle state, no separate subclasses per menu
      item type, supporting icons, etc.) Improvements to GtkMenuItem could
      eventually shrink the size of this class or remove the need for it
      entirely.
      
      Some GtkActionHelper functionality has been duplicated in
      GtkMenuTracker, which is suboptimal.  The duplication exists so that
      other codebases (such as Unity and gnome-shell) can reuse the
      GtkMenuTracker code, whereas GtkActionHelper is very much tied to
      GtkWidget.  Supporting binding arbitrary GtkWidgets to actions vs.
      supporting the full range of GMenuModel features for menu items turns
      out to be two overlapping but not entirely similar problems.  Some of
      the duplication (such as roles) can be removed from GtkActionHelper once
      Gtk's internal Mac OS menubar support is ported to GtkMenuTracker.
      
      The intent to reuse the code outside of Gtk is also the reason for the
      unusual treatment of the enum type introduced in this comment.
      
      This adds no new "public" API to the Gtk library, other than types that
      we cannot make private due to GType limitations.
      a4276a6c
  2. 01 Apr, 2013 1 commit
    • Allison Karlitskaya's avatar
      Introduce GtkMenuTracker · 5617b584
      Allison Karlitskaya authored
      GtkMenuTracker folds a nested structure of sections in a GMenuModel into
      a single linear menu, which it expresses to its user by means of 'insert
      item at position' and 'remove item at position' callbacks.
      
      The logic for where to insert separators and how to handle action
      namespaces is contained within the tracker, removing the need to have
      this logic duplicated in the 3 or 4 places that consume GMenuModel.
      
      In comparison with the previous code, the tracker no longer completely
      destroys and rebuilds menus every time a single change occurs.  As a
      result, the new gtkmenu testcase now runs in approximately 3 seconds
      instead of ~60 before.
      
      https://bugzilla.gnome.org/show_bug.cgi?id=696468
      5617b584
  3. 17 Sep, 2012 1 commit
    • Allison Karlitskaya's avatar
      GtkAccelLabel: add manual accel API · 778aa7ad
      Allison Karlitskaya authored
      Add an API to GtkAccelLabel for hardcoding the accel key to be displayed
      (ie: allowing us to bypass the GtkAccelGroup lookup).
      
      Use that from the GMenuModel-based GtkMenu construction code instead of
      passing around the accel group.
      
      This makes accel labels work in bloatpad again.
      
      This patch effectively removes any hope of automatic runtime accel
      changes in GMenuModel-based menus without additional application
      support but it leaves the door open for this to be supported again in
      the future (if we decide that it's important).
      
      https://bugzilla.gnome.org/show_bug.cgi?id=683738
      778aa7ad
  4. 20 Aug, 2012 2 commits
    • Lars Uebernickel's avatar
      gtkmodelmenu: add support for action namespaces · 71067472
      Lars Uebernickel authored
      If a section or submenu item has a "action-namespace" attribute, the
      action names of the created GtkModelMenuItems will be prefixed with that
      namespace.  Namespaces can be cascaded.
      71067472
    • Allison Karlitskaya's avatar
      gtkmodelmenu: move to new action regime · dd45862a
      Allison Karlitskaya authored
      Drop the explicit passing of GActionGroup into the GtkMenu(Bar)
      constructors and operate from the action context instead.
      
      With GtkMenuItem implementing GtkActionable, this turns out to be pretty
      easy (and most of the code can be removed from GtkModelMenuItem,
      including the GActionObserver implementation).
      dd45862a
  5. 27 Feb, 2012 1 commit
  6. 19 Dec, 2011 2 commits