GList *menu_frame_visible;
+static ObMenuEntryFrame* menu_entry_frame_new(ObMenuEntry *entry,
+ ObMenuFrame *frame);
+static void menu_entry_frame_free(ObMenuEntryFrame *self);
static void menu_frame_render(ObMenuFrame *self);
static void menu_frame_update(ObMenuFrame *self);
void menu_frame_free(ObMenuFrame *self)
{
if (self) {
+ while (self->entries) {
+ menu_entry_frame_free(self->entries->data);
+ self->entries = g_list_delete_link(self->entries, self->entries);
+ }
+
stacking_remove(MENU_AS_WINDOW(self));
XDestroyWindow(ob_display, self->items);
}
}
-ObMenuEntryFrame* menu_entry_frame_new(ObMenuEntry *entry, ObMenuFrame *frame)
+static ObMenuEntryFrame* menu_entry_frame_new(ObMenuEntry *entry,
+ ObMenuFrame *frame)
{
ObMenuEntryFrame *self;
XSetWindowAttributes attr;
return self;
}
-void menu_entry_frame_free(ObMenuEntryFrame *self)
+static void menu_entry_frame_free(ObMenuEntryFrame *self)
{
if (self) {
XDestroyWindow(ob_display, self->icon);
void menu_frame_move(ObMenuFrame *self, gint x, gint y)
{
- /* XXX screen constraints */
RECT_SET_POINT(self->area, x, y);
XMoveWindow(ob_display, self->window, self->area.x, self->area.y);
}
for (f = self; f; f = f->parent)
menu_frame_move(f, f->area.x + dx, f->area.y + dy);
+ for (f = self->child; f; f = f->child)
+ menu_frame_move(f, f->area.x + dx, f->area.y + dy);
XWarpPointer(ob_display, None, None, 0, 0, 0, 0, dx, dy);
}
}
{
RrAppearance *item_a, *text_a;
gint th; /* temp */
+ ObMenu *sub;
item_a = ((self->entry->type == OB_MENU_ENTRY_TYPE_NORMAL &&
!self->entry->data.normal.enabled) ?
text_a->texture[0].data.text.string = self->entry->data.normal.label;
break;
case OB_MENU_ENTRY_TYPE_SUBMENU:
- text_a->texture[0].data.text.string =
- self->entry->data.submenu.submenu->title;
+ sub = self->entry->data.submenu.submenu;
+ text_a->texture[0].data.text.string = sub ? sub->title : "";
break;
case OB_MENU_ENTRY_TYPE_SEPARATOR:
break;
switch (self->entry->type) {
case OB_MENU_ENTRY_TYPE_NORMAL:
- case OB_MENU_ENTRY_TYPE_SUBMENU:
XMoveResizeWindow(ob_display, self->text,
self->frame->text_x, 0,
self->frame->text_w, self->frame->item_h);
RrPaint(text_a, self->text, self->frame->text_w, self->frame->item_h);
XMapWindow(ob_display, self->text);
break;
+ case OB_MENU_ENTRY_TYPE_SUBMENU:
+ XMoveResizeWindow(ob_display, self->text,
+ self->frame->text_x, 0,
+ self->frame->text_w - self->frame->item_h,
+ self->frame->item_h);
+ text_a->surface.parent = item_a;
+ text_a->surface.parentx = self->frame->text_x;
+ text_a->surface.parenty = 0;
+ RrPaint(text_a, self->text, self->frame->text_w - self->frame->item_h,
+ self->frame->item_h);
+ XMapWindow(ob_display, self->text);
+ break;
case OB_MENU_ENTRY_TYPE_SEPARATOR:
XUnmapWindow(ob_display, self->text);
break;
}
- /* XXX draw icons */
- if (0) {
+ if (self->entry->type == OB_MENU_ENTRY_TYPE_NORMAL &&
+ self->entry->data.normal.icon_data)
+ {
XMoveResizeWindow(ob_display, self->icon, 0, 0,
self->frame->item_h, self->frame->item_h);
- /* XXX set the RGBA surface stuff */
+ self->a_icon->texture[0].data.rgba.width =
+ self->entry->data.normal.icon_width;
+ self->a_icon->texture[0].data.rgba.height =
+ self->entry->data.normal.icon_height;
+ self->a_icon->texture[0].data.rgba.data =
+ self->entry->data.normal.icon_data;
self->a_icon->surface.parent = item_a;
self->a_icon->surface.parentx = 0;
self->a_icon->surface.parenty = 0;
if (self->entry->type == OB_MENU_ENTRY_TYPE_SUBMENU) {
XMoveResizeWindow(ob_display, self->bullet,
- self->frame->text_x + self->frame->text_w, 0,
+ self->frame->text_x + self->frame->text_w
+ - self->frame->item_h, 0,
self->frame->item_h, self->frame->item_h);
self->a_bullet->surface.parent = item_a;
self->a_bullet->surface.parentx =
- self->frame->text_x + self->frame->text_w;
+ self->frame->text_x + self->frame->text_w - self->frame->item_h;
self->a_bullet->surface.parenty = 0;
RrPaint(self->a_bullet, self->bullet,
self->frame->item_h, self->frame->item_h);
gint allitems_h = 0;
gint tw, th; /* temps */
GList *it;
- gboolean has_icon = FALSE, has_bullet = FALSE;
+ gboolean has_icon = FALSE;
+ ObMenu *sub;
XSetWindowBorderWidth(ob_display, self->window, ob_rr_theme->bwidth);
XSetWindowBorder(ob_display, self->window,
text_a->texture[0].data.text.string = e->entry->data.normal.label;
RrMinsize(text_a, &tw, &th);
- /* XXX has_icon = TRUE; */
+ if (e->entry->data.normal.icon_data)
+ has_icon = TRUE;
break;
case OB_MENU_ENTRY_TYPE_SUBMENU:
- text_a->texture[0].data.text.string =
- e->entry->data.submenu.submenu->title;
+ sub = e->entry->data.submenu.submenu;
+ text_a->texture[0].data.text.string = sub ? sub->title : "";
RrMinsize(text_a, &tw, &th);
- has_bullet = TRUE;
+ tw += self->item_h;
break;
case OB_MENU_ENTRY_TYPE_SEPARATOR:
tw = 0;
self->text_w = w;
if (self->entries) {
- if (has_bullet)
- w += self->item_h;
if (has_icon) {
w += self->item_h;
self->text_x += self->item_h;
{
GList *mit, *fit;
+ menu_find_submenus(self->menu);
+
self->selected = NULL;
for (mit = self->menu->entries, fit = self->entries; mit && fit;
void menu_frame_show(ObMenuFrame *self, ObMenuFrame *parent)
{
+ GList *it;
+
+ if (g_list_find(menu_frame_visible, self))
+ return;
+
if (parent) {
if (parent->child)
menu_frame_hide(parent->child);
grab_keyboard(TRUE);
}
- if (!g_list_find(menu_frame_visible, self)) {
- menu_frame_visible = g_list_prepend(menu_frame_visible, self);
-
+ /* determine if the underlying menu is already visible */
+ for (it = menu_frame_visible; it; it = g_list_next(it)) {
+ ObMenuFrame *f = it->data;
+ if (f->menu == self->menu)
+ break;
+ }
+ if (!it) {
if (self->menu->update_func)
self->menu->update_func(self, self->menu->data);
- menu_frame_update(self);
}
+ menu_frame_visible = g_list_prepend(menu_frame_visible, self);
+ menu_frame_update(self);
+
menu_frame_move_on_screen(self);
XMapWindow(ob_display, self->window);
void menu_frame_hide(ObMenuFrame *self)
{
- menu_frame_visible = g_list_remove(menu_frame_visible, self);
+ GList *it = g_list_find(menu_frame_visible, self);
+
+ if (!it)
+ return;
+
+ menu_frame_visible = g_list_delete_link(menu_frame_visible, it);
if (self->child)
menu_frame_hide(self->child);
{
ObMenuFrame *f;
+ if (!self->entry->data.submenu.submenu) return;
+
f = menu_frame_new(self->entry->data.submenu.submenu,
self->frame->client);
menu_frame_move(f,
self->frame->area.x + self->frame->area.width
- - ob_rr_theme->menu_overlap,
+ - ob_rr_theme->menu_overlap - ob_rr_theme->bwidth,
self->frame->area.y + self->frame->title_h +
self->area.y + ob_rr_theme->menu_overlap);
menu_frame_show(f, self->frame);
void menu_entry_frame_execute(ObMenuEntryFrame *self, gboolean hide)
{
- if (self->entry->type == OB_MENU_ENTRY_TYPE_NORMAL) {
+ if (self->entry->type == OB_MENU_ENTRY_TYPE_NORMAL &&
+ self->entry->data.normal.enabled)
+ {
+ /* grab all this shizzle, cuz when the menu gets hidden, 'self'
+ gets freed */
+ ObMenuEntry *entry = self->entry;
+ ObMenuExecuteFunc func = self->frame->menu->execute_func;
+ gpointer data = self->frame->menu->data;
+ GSList *acts = self->entry->data.normal.actions;
+ ObClient *client = self->frame->client;
+
/* release grabs before executing the shit */
menu_frame_hide_all();
- if (self->frame->menu->execute_func)
- self->frame->menu->execute_func(self, self->frame->menu->data);
+ if (func)
+ func(entry, data);
else {
GSList *it;
- for (it = self->entry->data.normal.actions; it;
- it = g_slist_next(it))
+ for (it = acts; it; it = g_slist_next(it))
{
ObAction *act = it->data;
- act->data.any.c = self->frame->client;
+ act->data.any.c = client;
act->func(&act->data);
}
}