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

Timestamp:

2023-12-28T13:59:23Z (2 years 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/list.c (modified) (9 diffs)

uspace/lib/ui/src/list.c

@@ -149 +149 @@
         list->cb = cb;
         list->cb_arg = arg;
+}
+
+/** Get UI list callback argument.
+ *
+ * @param list UI list
+ * @return Callback argument
+ */
+void *ui_list_get_cb_arg(ui_list_t *list)
+{
+        return list->cb_arg;
 }
 
@@ -701 +711 @@
 }
 
-/** Delete UI list entry.
+/** Destroy UI list entry.
+ *
+ * This is the quick way, but does not update cursor or page position.
  *
  * @param entry UI list entry
  */
-void ui_list_entry_delete(ui_list_entry_t *entry)
+void ui_list_entry_destroy(ui_list_entry_t *entry)
 {
         if (entry->list->cursor == entry)
@@ -718 +730 @@
 }
 
+/** Delete UI list entry.
+ *
+ * If required, update cursor and page position and repaint.
+ *
+ * @param entry UI list entry
+ */
+void ui_list_entry_delete(ui_list_entry_t *entry)
+{
+        ui_list_t *list = entry->list;
+
+        /* Try to make sure entry does not disappear under cursor or page */
+        if (entry->list->cursor == entry)
+                ui_list_cursor_up(entry->list);
+        if (entry->list->cursor == entry)
+                ui_list_cursor_down(entry->list);
+        if (entry->list->page == entry)
+                ui_list_scroll_up(entry->list);
+        if (entry->list->page == entry)
+                ui_list_scroll_down(entry->list);
+
+        ui_list_entry_destroy(entry);
+
+        /*
+         * But it could still happen if there are not enough entries.
+         * In that case just move page and/or cursor to the first
+         * entry.
+         */
+        if (list->page == NULL) {
+                list->page = ui_list_first(list);
+                list->page_idx = 0;
+        } else {
+                /*
+                 * Entry index might have changed if earlier entry
+                 * was deleted.
+                 */
+                list->page_idx = ui_list_entry_get_idx(list->page);
+        }
+
+        if (list->cursor == NULL) {
+                list->cursor = ui_list_first(list);
+                list->cursor_idx = 0;
+        } else {
+                /*
+                 * Entry index might have changed if earlier entry
+                 * was deleted.
+                 */
+                list->cursor_idx = ui_list_entry_get_idx(list->cursor);
+        }
+}
+
 /** Get entry argument.
  *
@@ -728 +790 @@
 }
 
+/** Get containing list.
+ *
+ * @param entry UI list entry
+ * @return Containing list
+ */
+ui_list_t *ui_list_entry_get_list(ui_list_entry_t *entry)
+{
+        return entry->list;
+}
+
+/** Change list entry caption.
+ *
+ * @param entry UI list entry
+ * @param caption New caption
+ *
+ * @return EOK on success, ENOMEM if out of memory
+ */
+errno_t ui_list_entry_set_caption(ui_list_entry_t *entry, const char *caption)
+{
+        char *dcaption;
+
+        dcaption = str_dup(caption);
+        if (dcaption == NULL)
+                return ENOMEM;
+
+        free(entry->caption);
+        entry->caption = dcaption;
+
+        (void)ui_list_entry_paint(entry, ui_list_entry_get_idx(entry));
+        return EOK;
+}
+
 /** Clear UI list entry list.
  *
@@ -738 +832 @@
         entry = ui_list_first(list);
         while (entry != NULL) {
-                ui_list_entry_delete(entry);
+                ui_list_entry_destroy(entry);
                 entry = ui_list_first(list);
         }
@@ -858 +952 @@
 {
         return list->cursor;
+}
+
+/** Set new cursor position.
+ *
+ * O(N) in list size, use with caution.
+ *
+ * @param list UI list
+ * @param entry New cursor position
+ */
+void ui_list_set_cursor(ui_list_t *list, ui_list_entry_t *entry)
+{
+        size_t idx;
+
+        idx = ui_list_entry_get_idx(entry);
+        ui_list_cursor_move(list, entry, idx);
 }
 
@@ -912 +1021 @@
                                 /* Find first page entry (go back rows - 1) */
                                 e = entry;
-                                for (i = 0; i < rows - 1; i++) {
+                                for (i = 0; i + 1 < rows; i++) {
                                         e = ui_list_prev(e);
                                 }
@@ -1098 +1207 @@
         ui_list_entry_t *prev;
 
+        if (list->page == NULL)
+                return;
+
         prev = ui_list_prev(list->page);
         if (prev == NULL)
@@ -1120 +1232 @@
         size_t i;
         size_t rows;
+
+        if (list->page == NULL)
+                return;
 
         next = ui_list_next(list->page);