Changeset dc5c303 in mainline for uspace/lib/ui/src/menubar.c

Timestamp:

2023-12-28T13:59:23Z (14 months ago)

Author:

GitHub <noreply@…>

Children:

6b66de6b

Parents:

42c2e65 (diff), f87ff8e (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

git-author:

boba-buba <120932204+boba-buba@…> (2023-12-28 13:59:23)

git-committer:

GitHub <noreply@…> (2023-12-28 13:59:23)

Message:

Merge branch 'master' into topic/packet-capture

File:

• uspace/lib/ui/src/menubar.c (modified) (25 diffs)

uspace/lib/ui/src/menubar.c

@@ -35 +35 @@
 
 #include <adt/list.h>
+#include <ctype.h>
 #include <errno.h>
 #include <gfx/color.h>
@@ -45 +46 @@
 #include <ui/control.h>
 #include <ui/paint.h>
-#include <ui/menu.h>
 #include <ui/menubar.h>
+#include <ui/menudd.h>
+#include <ui/wdecor.h>
 #include <ui/window.h>
 #include "../private/menubar.h"
 #include "../private/resource.h"
+#include "../private/wdecor.h"
+#include "../private/window.h"
 
 enum {
@@ -95 +99 @@
         mbar->ui = ui;
         mbar->window = window;
-        list_initialize(&mbar->menus);
+        list_initialize(&mbar->menudds);
+
+        if (window->mbar == NULL)
+                window->mbar = mbar;
+
         *rmbar = mbar;
         return EOK;
@@ -106 +114 @@
 void ui_menu_bar_destroy(ui_menu_bar_t *mbar)
 {
-        ui_menu_t *menu;
+        ui_menu_dd_t *mdd;
 
         if (mbar == NULL)
                 return;
 
-        /* Destroy menus */
-        menu = ui_menu_first(mbar);
-        while (menu != NULL) {
-                ui_menu_destroy(menu);
-                menu = ui_menu_first(mbar);
+        if (mbar->window->mbar == mbar)
+                mbar->window->mbar = NULL;
+
+        /* Destroy menu drop-downs */
+        mdd = ui_menu_dd_first(mbar);
+        while (mdd != NULL) {
+                ui_menu_dd_destroy(mdd);
+                mdd = ui_menu_dd_first(mbar);
         }
 
@@ -155 +166 @@
         gfx_rect_t rect;
         gfx_color_t *bg_color;
-        ui_menu_t *menu;
+        ui_menu_dd_t *mdd;
         const char *caption;
         gfx_coord_t width;
@@ -189 +200 @@
         fmt.valign = gfx_valign_top;
 
-        menu = ui_menu_first(mbar);
-        while (menu != NULL) {
-                caption = ui_menu_caption(menu);
+        mdd = ui_menu_dd_first(mbar);
+        while (mdd != NULL) {
+                caption = ui_menu_dd_caption(mdd);
                 width = ui_text_width(res->font, caption) + 2 * hpad;
                 tpos.x = pos.x + hpad;
@@ -200 +211 @@
                 rect.p1.y = mbar->rect.p1.y;
 
-                if (menu == mbar->selected) {
+                if (mdd == mbar->selected) {
                         fmt.color = res->wnd_sel_text_color;
                         fmt.hgl_color = res->wnd_sel_text_hgl_color;
@@ -223 +234 @@
 
                 pos.x += width;
-                menu = ui_menu_next(menu);
+                mdd = ui_menu_dd_next(mdd);
         }
 
@@ -241 +252 @@
  *
  * @param mbar Menu bar
- * @param menu Menu to select (or deselect if selected) or @c NULL
+ * @param mdd Menu drop-down to select (or deselect if selected) or @c NULL
  * @param openup Open menu even if not currently open
  * @param idev_id Input device ID associated with the selecting seat
  */
-void ui_menu_bar_select(ui_menu_bar_t *mbar, ui_menu_t *menu, bool openup,
+void ui_menu_bar_select(ui_menu_bar_t *mbar, ui_menu_dd_t *mdd, bool openup,
     sysarg_t idev_id)
 {
-        ui_menu_t *old_menu;
+        ui_menu_dd_t *old_mdd;
         gfx_rect_t rect;
         bool was_open;
 
-        old_menu = mbar->selected;
-
-        mbar->selected = menu;
-
-        /* Close previously open menu */
-        if (old_menu != NULL && ui_menu_is_open(old_menu)) {
+        old_mdd = mbar->selected;
+
+        mbar->selected = mdd;
+
+        /* Close previously open menu drop-down */
+        if (old_mdd != NULL && ui_menu_dd_is_open(old_mdd)) {
                 was_open = true;
-                (void) ui_menu_close(old_menu);
+                (void) ui_menu_dd_close(old_mdd);
         } else {
                 was_open = false;
@@ -270 +281 @@
                 if (openup || was_open) {
                         /*
-                         * Open the newly selected menu if either
-                         * the old menu was open or @a openup was
+                         * Open the newly selected menu drop-down if either
+                         * the old menu drop-down was open or @a openup was
                          * specified.
                          */
-                        (void) ui_menu_open(mbar->selected, &rect, idev_id);
+                        (void) ui_menu_dd_open(mbar->selected, &rect, idev_id);
                 }
-        }
+
+                mbar->active = true;
+        } else {
+                mbar->active = false;
+        }
+}
+
+/** Select first drop-down.
+ *
+ * @param mbar Menu bar
+ * @param openup @c true to open drop-down if it was not open
+ * @param idev_id Input device ID
+ */
+void ui_menu_bar_select_first(ui_menu_bar_t *mbar, bool openup,
+    sysarg_t idev_id)
+{
+        ui_menu_dd_t *mdd;
+
+        mdd = ui_menu_dd_first(mbar);
+        ui_menu_bar_select(mbar, mdd, openup, idev_id);
+}
+
+/** Select last drop-down.
+ *
+ * @param mbar Menu bar
+ * @param openup @c true to open drop-down if it was not open
+ * @param idev_id Input device ID
+ */
+void ui_menu_bar_select_last(ui_menu_bar_t *mbar, bool openup,
+    sysarg_t idev_id)
+{
+        ui_menu_dd_t *mdd;
+
+        mdd = ui_menu_dd_last(mbar);
+        ui_menu_bar_select(mbar, mdd, openup, idev_id);
+}
+
+/** Select system menu.
+ *
+ * @param mbar Menu bar
+ * @param openup @c true to open drop-down if it was not open
+ * @param idev_id Input device ID
+ */
+void ui_menu_bar_select_sysmenu(ui_menu_bar_t *mbar, bool openup,
+    sysarg_t idev_id)
+{
+        ui_wdecor_sysmenu_hdl_set_active(mbar->window->wdecor, true);
+
+        if (openup)
+                ui_window_send_sysmenu(mbar->window, idev_id);
 }
 
@@ -289 +349 @@
 void ui_menu_bar_left(ui_menu_bar_t *mbar, sysarg_t idev_id)
 {
-        ui_menu_t *nmenu;
+        ui_menu_dd_t *nmdd;
+        bool sel_sysmenu = false;
+        bool was_open;
 
         if (mbar->selected == NULL)
                 return;
 
-        nmenu = ui_menu_prev(mbar->selected);
-        if (nmenu == NULL)
-                nmenu = ui_menu_last(mbar);
-
-        if (nmenu != mbar->selected)
-                ui_menu_bar_select(mbar, nmenu, false, idev_id);
+        nmdd = ui_menu_dd_prev(mbar->selected);
+        if (nmdd == NULL) {
+                if ((mbar->window->wdecor->style & ui_wds_sysmenu_hdl) != 0) {
+                        sel_sysmenu = true;
+                } else {
+                        nmdd = ui_menu_dd_last(mbar);
+                }
+        }
+
+        was_open = mbar->selected != NULL &&
+            ui_menu_dd_is_open(mbar->selected);
+
+        if (nmdd != mbar->selected)
+                ui_menu_bar_select(mbar, nmdd, false, idev_id);
+
+        /*
+         * Only open sysmenu *after* closing the previous menu, avoid
+         * having multiple popup windows at the same time.
+         */
+        if (sel_sysmenu)
+                ui_menu_bar_select_sysmenu(mbar, was_open, idev_id);
 }
 
@@ -312 +389 @@
 void ui_menu_bar_right(ui_menu_bar_t *mbar, sysarg_t idev_id)
 {
-        ui_menu_t *nmenu;
+        ui_menu_dd_t *nmdd;
+        bool sel_sysmenu = false;
+        bool was_open;
 
         if (mbar->selected == NULL)
                 return;
 
-        nmenu = ui_menu_next(mbar->selected);
-        if (nmenu == NULL)
-                nmenu = ui_menu_first(mbar);
-
-        if (nmenu != mbar->selected)
-                ui_menu_bar_select(mbar, nmenu, false, idev_id);
+        nmdd = ui_menu_dd_next(mbar->selected);
+        if (nmdd == NULL) {
+                if ((mbar->window->wdecor->style & ui_wds_sysmenu_hdl) != 0) {
+                        sel_sysmenu = true;
+                } else {
+                        nmdd = ui_menu_dd_first(mbar);
+                }
+        }
+
+        was_open = mbar->selected != NULL &&
+            ui_menu_dd_is_open(mbar->selected);
+
+        if (nmdd != mbar->selected)
+                ui_menu_bar_select(mbar, nmdd, false, idev_id);
+
+        /*
+         * Only open sysmenu *after* closing the previous menu, avoid
+         * having multiple popup windows at the same time.
+         */
+        if (sel_sysmenu)
+                ui_menu_bar_select_sysmenu(mbar, was_open, idev_id);
 }
 
@@ -355 +449 @@
 
         if (event->key == KC_ENTER || event->key == KC_DOWN) {
-                if (mbar->selected != NULL && !ui_menu_is_open(mbar->selected)) {
+                if (mbar->selected != NULL &&
+                    !ui_menu_dd_is_open(mbar->selected)) {
                         ui_menu_bar_entry_rect(mbar, mbar->selected,
                             &rect);
-                        ui_menu_open(mbar->selected, &rect, event->kbd_id);
+                        ui_menu_dd_open(mbar->selected, &rect, event->kbd_id);
                 }
 
@@ -364 +459 @@
         }
 
-        if (event->c != '\0' && !ui_menu_is_open(mbar->selected)) {
+        if (event->c != '\0' && !ui_menu_dd_is_open(mbar->selected)) {
                 /* Check if it is an accelerator. */
                 ui_menu_bar_press_accel(mbar, event->c, event->kbd_id);
@@ -407 +502 @@
 void ui_menu_bar_press_accel(ui_menu_bar_t *mbar, char32_t c, sysarg_t kbd_id)
 {
-        ui_menu_t *menu;
+        ui_menu_dd_t *mdd;
         char32_t maccel;
 
-        menu = ui_menu_first(mbar);
-        while (menu != NULL) {
-                maccel = ui_menu_get_accel(menu);
-                if (c == maccel) {
-                        ui_menu_bar_select(mbar, menu, true, kbd_id);
+        mdd = ui_menu_dd_first(mbar);
+        while (mdd != NULL) {
+                maccel = ui_menu_dd_get_accel(mdd);
+                if ((char32_t)tolower(c) == maccel) {
+                        ui_menu_bar_select(mbar, mdd, true, kbd_id);
                         return;
                 }
 
-                menu = ui_menu_next(menu);
+                mdd = ui_menu_dd_next(mdd);
         }
 }
@@ -433 +528 @@
         gfx_coord2_t pos;
         gfx_rect_t rect;
-        ui_menu_t *menu;
+        ui_menu_dd_t *mdd;
         const char *caption;
         gfx_coord_t width;
@@ -454 +549 @@
         pos_id = event->pos_id;
 
-        menu = ui_menu_first(mbar);
-        while (menu != NULL) {
-                caption = ui_menu_caption(menu);
+        mdd = ui_menu_dd_first(mbar);
+        while (mdd != NULL) {
+                caption = ui_menu_dd_caption(mdd);
                 width = ui_text_width(res->font, caption) + 2 * hpad;
 
@@ -468 +563 @@
                         mbar->active = true;
 
-                        /* Open the menu, close if already open. */
-                        if (menu == mbar->selected)
-                                ui_menu_bar_select(mbar, NULL, false, pos_id);
-                        else
-                                ui_menu_bar_select(mbar, menu, true, pos_id);
+                        /* Open the menu, if not already open. */
+                        if (mdd != mbar->selected)
+                                ui_menu_bar_select(mbar, mdd, true, pos_id);
 
                         return ui_claimed;
@@ -478 +571 @@
 
                 pos.x += width;
-                menu = ui_menu_next(menu);
+                mdd = ui_menu_dd_next(mdd);
         }
 
@@ -487 +580 @@
  *
  * @param mbar Menu bar
- * @param menu Menu whose entry's rectangle is to be returned
+ * @param mdd Menu drop-down whose entry's rectangle is to be returned
  * @param rrect Place to store entry rectangle
  */
-void ui_menu_bar_entry_rect(ui_menu_bar_t *mbar, ui_menu_t *menu,
+void ui_menu_bar_entry_rect(ui_menu_bar_t *mbar, ui_menu_dd_t *mdd,
     gfx_rect_t *rrect)
 {
@@ -496 +589 @@
         gfx_coord2_t pos;
         gfx_rect_t rect;
-        ui_menu_t *cur;
+        ui_menu_dd_t *cur;
         const char *caption;
         gfx_coord_t width;
@@ -511 +604 @@
         pos = mbar->rect.p0;
 
-        cur = ui_menu_first(mbar);
+        cur = ui_menu_dd_first(mbar);
         while (cur != NULL) {
-                caption = ui_menu_caption(cur);
+                caption = ui_menu_dd_caption(cur);
                 width = ui_text_width(res->font, caption) + 2 * hpad;
 
@@ -520 +613 @@
                 rect.p1.y = mbar->rect.p1.y;
 
-                if (cur == menu) {
+                if (cur == mdd) {
                         *rrect = rect;
                         return;
@@ -526 +619 @@
 
                 pos.x += width;
-                cur = ui_menu_next(cur);
+                cur = ui_menu_dd_next(cur);
         }
 
@@ -544 +637 @@
         mbar->active = true;
         if (mbar->selected == NULL)
-                mbar->selected = ui_menu_first(mbar);
+                mbar->selected = ui_menu_dd_first(mbar);
 
         (void) ui_menu_bar_paint(mbar);
 }
 
+/** Deactivate menu bar.
+ *
+ * @param mbar Menu bar
+ */
 void ui_menu_bar_deactivate(ui_menu_bar_t *mbar)
 {
         ui_menu_bar_select(mbar, NULL, false, 0);
-        mbar->active = false;
 }