Changeset 68d68e9 in mainline for uspace/app/taskbar/wndlist.c

Timestamp:

2022-11-23T12:50:27Z (2 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

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

Children:

c4a53280

Parents:

6e91475

Message:

Vary window button size to fit

If they get too narrow, we stop adding more buttons.

File:

• uspace/app/taskbar/wndlist.c (modified) (8 diffs)

uspace/app/taskbar/wndlist.c

@@ -35 +35 @@
 #include <gfx/coord.h>
 #include <stdbool.h>
+#include <stddef.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -65 +66 @@
 
 enum {
-        /** X distance between left edges of two consecutive buttons */
-        wndlist_button_pitch = 145,
-        /** X distance between left edges of two consecutive buttons (text) */
-        wndlist_button_pitch_text = 17,
+        /** Min. X distance between left edges of two consecutive buttons */
+        wndlist_button_pitch_min = 85,
+        /** Max. X distance between left edges of two consecutive buttons (text) */
+        wndlist_button_pitch_min_text = 10,
+        /** Min. X distance between left edges of two consecutive buttons */
+        wndlist_button_pitch_max = 165,
+        /** Max. X distance between left edges of two consecutive buttons (text) */
+        wndlist_button_pitch_max_text = 17,
         /** Padding between buttons */
         wndlist_button_pad = 5,
@@ -197 +202 @@
         wndlist_entry_t *entry = NULL;
         ui_resource_t *res;
+        wndlist_entry_t *e;
         errno_t rc;
 
@@ -217 +223 @@
         entry->visible = false;
 
-        /* Set the button rectangle and add it to layout, if applicable */
-        wndlist_set_entry_rect(wndlist, entry);
+        /*
+         * Update rectangles for all entries, including @a entry, adding
+         * it to the layout, if applicable.
+         */
+        e = wndlist_first(wndlist);
+        while (e != NULL) {
+                wndlist_set_entry_rect(wndlist, e);
+                e = wndlist_next(e);
+        }
 
         /* Set button callbacks */
         ui_pbutton_set_cb(entry->button, &wndlist_button_cb, (void *)entry);
 
-        if (paint && entry->visible) {
-                rc = ui_pbutton_paint(entry->button);
-                if (rc != EOK)
-                        goto error;
-        }
+        if (paint)
+                return wndlist_repaint(wndlist);
 
         return EOK;
@@ -249 +259 @@
     bool paint)
 {
-        wndlist_entry_t *next;
+        wndlist_entry_t *e;
         assert(entry->wndlist == wndlist);
 
-        next = wndlist_next(entry);
-
-        ui_fixed_remove(wndlist->fixed, ui_pbutton_ctl(entry->button));
+        if (entry->visible)
+                ui_fixed_remove(wndlist->fixed, ui_pbutton_ctl(entry->button));
         ui_pbutton_destroy(entry->button);
         list_remove(&entry->lentries);
         free(entry);
 
-        /* Update positions of the remaining entries */
-        while (next != NULL) {
-                wndlist_set_entry_rect(wndlist, next);
-                next = wndlist_next(next);
+        /* Update positions of the all entries */
+        e = wndlist_first(wndlist);
+        while (e != NULL) {
+                wndlist_set_entry_rect(wndlist, e);
+                e = wndlist_next(e);
         }
 
@@ -307 +317 @@
         ui_resource_t *res;
         gfx_coord_t pitch;
+        gfx_coord_t pitch_max;
+        gfx_coord_t pitch_min;
         gfx_coord_t pad;
         size_t idx;
+        size_t nbuttons;
 
         /* Determine entry index */
@@ -322 +335 @@
 
         if (ui_resource_is_textmode(res)) {
-                pitch = wndlist_button_pitch_text;
+                pitch_max = wndlist_button_pitch_max_text;
+                pitch_min = wndlist_button_pitch_min_text;
                 pad = wndlist_button_pad_text;
         } else {
-                pitch = wndlist_button_pitch;
+                pitch_max = wndlist_button_pitch_max;
+                pitch_min = wndlist_button_pitch_min;
                 pad = wndlist_button_pad;
         }
+
+        /* Compute pitch that fits all buttons perfectly */
+        nbuttons = wndlist_count(wndlist);
+        pitch = (wndlist->rect.p1.x - wndlist->rect.p0.x + pad) / nbuttons;
+        if (pitch < pitch_min)
+                pitch = pitch_min;
+        if (pitch > pitch_max)
+                pitch = pitch_max;
 
         rect.p0.x = wndlist->rect.p0.x + pitch * idx;
@@ -478 +501 @@
 }
 
+/** Get number of window list entries.
+ *
+ * @param wndlist Window list
+ * @return Number of entries
+ */
+size_t wndlist_count(wndlist_t *wndlist)
+{
+        return list_count(&wndlist->entries);
+}
+
 /** Repaint window list.
  *