Changeset 96c6a00 in mainline for uspace/lib/ui/src

Timestamp:

2022-03-10T13:44:10Z (3 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

master, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

c88d7f99

Parents:

ca2680d

git-author:

Jiri Svoboda <jiri@…> (2022-03-09 18:44:00)

git-committer:

Jiri Svoboda <jiri@…> (2022-03-10 13:44:10)

Message:

Menu accelerators

Open a menu by pressing F10, then accelerator key (e.g. 'F').

Location:

uspace/lib/ui/src

Files:

• accel.c (added)
• menu.c (modified) (2 diffs)
• menubar.c (modified) (4 diffs)
• paint.c (modified) (4 diffs)

uspace/lib/ui/src/menu.c

@@ -43 +43 @@
 #include <stdlib.h>
 #include <str.h>
+#include <uchar.h>
+#include <ui/accel.h>
 #include <ui/control.h>
 #include <ui/paint.h>
@@ -251 +253 @@
         ui_menu_get_geom(menu, spos, &geom);
         *rect = geom.outer_rect;
+}
+
+/** Get menu accelerator character.
+ *
+ * @param menu Menu
+ * @return Accelerator character (lowercase) or the null character if
+ *         the menu has no accelerator.
+ */
+char32_t ui_menu_get_accel(ui_menu_t *menu)
+{
+        return ui_accel_get(menu->caption);
 }
 

uspace/lib/ui/src/menubar.c

@@ -252 +252 @@
         old_menu = mbar->selected;
 
-        if (mbar->selected != menu)
-                mbar->selected = menu;
-        else
-                mbar->selected = NULL;
+        mbar->selected = menu;
 
         /* Close previously open menu */
@@ -333 +330 @@
 {
         gfx_rect_t rect;
+        ui_menu_t *menu;
+        char32_t maccel;
 
         if (event->key == KC_F10) {
@@ -361 +360 @@
 
                 return ui_claimed;
+        }
+
+        if (event->c != '\0' && !ui_menu_is_open(mbar->selected)) {
+                /* Check if it is an accelerator. */
+
+                menu = ui_menu_first(mbar);
+                while (menu != NULL) {
+                        maccel = ui_menu_get_accel(menu);
+                        if (event->c == maccel) {
+                                ui_menu_bar_select(mbar, menu, true);
+                                return ui_claimed;
+                        }
+
+                        menu = ui_menu_next(menu);
+                }
         }
 
@@ -428 +442 @@
                     gfx_pix_inside_rect(&ppos, &rect)) {
                         mbar->active = true;
-                        ui_menu_bar_select(mbar, menu, true);
+
+                        /* Open the menu, close if already open. */
+                        if (menu == mbar->selected)
+                                ui_menu_bar_select(mbar, NULL, false);
+                        else
+                                ui_menu_bar_select(mbar, menu, true);
+
                         return ui_claimed;
                 }

uspace/lib/ui/src/paint.c

@@ -1 +1 @@
 /*
- * Copyright (c) 2021 Jiri Svoboda
+ * Copyright (c) 2022 Jiri Svoboda
  * All rights reserved.
  *
@@ -39 +39 @@
 #include <gfx/render.h>
 #include <gfx/text.h>
+#include <ui/accel.h>
 #include <ui/paint.h>
 #include <stdlib.h>
@@ -566 +567 @@
 }
 
-/** Process text with '~' markup.
- *
- * Parse text with tilde markup into a list of strings. @a *rbuf is set
- * to point to a newly allocated buffer containing consecutive null-terminated
- * strings.
- *
- * Each part between two '~' becomes one string. '~~' is translated into
- * a literal '~' character. @a *endptr is set to point to the first character
- * beyond the end of the list.
- *
- * @param str String with tilde markup
- * @param rbuf Place to store pointer to newly allocated buffer.
- * @param endptr Place to store end pointer (just after last character)
- * @return EOK on success or an error code
- */
-static int ui_text_process(const char *str, char **rbuf, char **endptr)
-{
-        const char *sp;
-        char *dp;
-        char *buf;
-
-        buf = malloc(str_size(str) + 1);
-        if (buf == NULL)
-                return ENOMEM;
-
-        /* Break down string into list of (non)highlighted parts */
-        sp = str;
-        dp = buf;
-        while (*sp != '\0') {
-                if (*sp == '~') {
-                        if (sp[1] == '~') {
-                                sp += 2;
-                                *dp++ = '~';
-                        } else {
-                                ++sp;
-                                *dp++ = '\0';
-                        }
-                } else {
-                        *dp++ = *sp++;
-                }
-        }
-
-        *dp++ = '\0';
-        *rbuf = buf;
-        *endptr = dp;
-
-        return EOK;
-}
-
 /** Compute UI text width.
  *
@@ -664 +616 @@
 
         /* Break down string into list of (non)highlighted parts */
-        rc = ui_text_process(str, &buf, &endptr);
+        rc = ui_accel_process(str, &buf, &endptr);
         if (rc != EOK)
                 return rc;