PolytreeDE/rofi
1
0
Fork 0
mirror of https://github.com/davatorium/rofi.git synced 2024-11-11 13:50:48 -05:00
Code Releases Activity

Revert "[I1437] Textbox make get_desired_width depend on width."

Browse source 
This reverts commit 154e03efc1.
This commit is contained in:
Dave Davenport 2021-09-06 22:47:35 +02:00
parent 965d3631cb
commit f4e5fd328a
11 changed files with 34 additions and 53 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
include/widgets/textbox.h
View file

@ -319,7 +319,7 @@ const char *textbox_get_visible_text(const textbox *tb);
* *
* @returns the desired width of the textbox. * @returns the desired width of the textbox.
*/ */
int textbox_get_desired_width(widget *wid, const int height); int textbox_get_desired_width(widget *wid);
/** /**
* @param tb Handle to the textbox * @param tb Handle to the textbox

4
include/widgets/widget-internal.h
View file

@ -80,8 +80,8 @@ struct _widget {
/** Handle mouse motion, used for dragging */ /** Handle mouse motion, used for dragging */
gboolean (*motion_notify)(struct _widget *, gint x, gint y); gboolean (*motion_notify)(struct _widget *, gint x, gint y);
int (*get_desired_height)(struct _widget *, const int width); int (*get_desired_height)(struct _widget *);
int (*get_desired_width)(struct _widget *, const int height); int (*get_desired_width)(struct _widget *);
void (*set_state)(struct _widget *, const char *); void (*set_state)(struct _widget *, const char *);

6
include/widgets/widget.h
View file

@ -318,23 +318,21 @@ gboolean widget_motion_notify(widget *wid, gint x, gint y);
/** /**
* @param wid The widget handle * @param wid The widget handle
* @param width The Widget width to get height for
* *
* Get the desired height of this widget recursively. * Get the desired height of this widget recursively.
* *
* @returns the desired height of the widget in pixels. * @returns the desired height of the widget in pixels.
*/ */
int widget_get_desired_height(widget *wid, const int width); int widget_get_desired_height(widget *wid);
/** /**
* @param wid The widget handle * @param wid The widget handle
* @param height The Widget height to get height for
* *
* Get the desired width of this widget recursively. * Get the desired width of this widget recursively.
* *
* @returns the desired width of the widget in pixels. * @returns the desired width of the widget in pixels.
*/ */
int widget_get_desired_width(widget *wid, const int height); int widget_get_desired_width(widget *wid);
/** /**
* @param wid The widget handle * @param wid The widget handle
* *

9
source/view.c
View file

@ -1026,7 +1026,7 @@ static void update_callback(textbox *t, icon *ico, unsigned int index,
list = pango_attr_list_new(); list = pango_attr_list_new();
} }
if (ico) { if (ico) {
int icon_height = widget_get_desired_height(WIDGET(ico), WIDGET(ico)->w); int icon_height = widget_get_desired_height(WIDGET(ico));
cairo_surface_t *icon = cairo_surface_t *icon =
mode_get_icon(state->sw, state->line_map[index], icon_height); mode_get_icon(state->sw, state->line_map[index], icon_height);
icon_set_surface(ico, icon); icon_set_surface(ico, icon);
@ -1667,7 +1667,7 @@ static int rofi_view_calculate_height(RofiViewState *state) {
} }
// Autosize based on widgets. // Autosize based on widgets.
widget *main_window = WIDGET(state->main_window); widget *main_window = WIDGET(state->main_window);
return widget_get_desired_height(main_window, main_window->w); return widget_get_desired_height(main_window);
} }
static WidgetTriggerActionResult textbox_button_trigger_action( static WidgetTriggerActionResult textbox_button_trigger_action(
@ -2049,10 +2049,9 @@ int rofi_view_error_dialog(const char *msg, int markup) {
} }
rofi_view_calculate_window_width(state); rofi_view_calculate_window_width(state);
// Need to resize otherwise calculated desired height is wrong. // Need to resize otherwise calculated desired height is wrong.
// widget_resize(WIDGET(state->main_window), state->width, 100); widget_resize(WIDGET(state->main_window), state->width, 100);
// resize window vertically to suit // resize window vertically to suit
state->height = state->height = widget_get_desired_height(WIDGET(state->main_window));
widget_get_desired_height(WIDGET(state->main_window), state->width);
// Calculate window position. // Calculate window position.
rofi_view_calculate_window_position(state); rofi_view_calculate_window_position(state);

19
source/widgets/box.c
View file

@ -49,7 +49,7 @@ struct _box {
static void box_update(widget *wid); static void box_update(widget *wid);
static int box_get_desired_width(widget *wid, const int height) { static int box_get_desired_width(widget *wid) {
box *b = (box *)wid; box *b = (box *)wid;
int spacing = distance_get_pixel(b->spacing, b->type); int spacing = distance_get_pixel(b->spacing, b->type);
int width = 0; int width = 0;
@ -71,10 +71,10 @@ static int box_get_desired_width(widget *wid, const int height) {
} }
active_widgets++; active_widgets++;
if (child->expand == TRUE) { if (child->expand == TRUE) {
width += widget_get_desired_width(child, height); width += widget_get_desired_width(child);
continue; continue;
} }
width += widget_get_desired_width(child, height); width += widget_get_desired_width(child);
} }
if (active_widgets > 0) { if (active_widgets > 0) {
width += (active_widgets - 1) * spacing; width += (active_widgets - 1) * spacing;
@ -86,13 +86,13 @@ static int box_get_desired_width(widget *wid, const int height) {
if (!child->enabled) { if (!child->enabled) {
continue; continue;
} }
width = MAX(widget_get_desired_width(child, height), width); width = MAX(widget_get_desired_width(child), width);
} }
} }
width += widget_padding_get_padding_width(wid); width += widget_padding_get_padding_width(wid);
return width; return width;
} }
static int box_get_desired_height(widget *wid, const int width) { static int box_get_desired_height(widget *wid) {
box *b = (box *)wid; box *b = (box *)wid;
int spacing = distance_get_pixel(b->spacing, b->type); int spacing = distance_get_pixel(b->spacing, b->type);
int height = 0; int height = 0;
@ -105,7 +105,7 @@ static int box_get_desired_height(widget *wid, const int width) {
continue; continue;
} }
active_widgets++; active_widgets++;
height += widget_get_desired_height(child, width); height += widget_get_desired_height(child);
} }
if (active_widgets > 0) { if (active_widgets > 0) {
height += (active_widgets - 1) * spacing; height += (active_widgets - 1) * spacing;
@ -117,7 +117,7 @@ static int box_get_desired_height(widget *wid, const int width) {
if (!child->enabled) { if (!child->enabled) {
continue; continue;
} }
height = MAX(widget_get_desired_height(child, width), height); height = MAX(widget_get_desired_height(child), height);
} }
} }
height += widget_padding_get_padding_height(wid); height += widget_padding_get_padding_height(wid);
@ -134,8 +134,7 @@ static void vert_calculate_size(box *b) {
iter = g_list_next(iter)) { iter = g_list_next(iter)) {
widget *child = (widget *)iter->data; widget *child = (widget *)iter->data;
if (child->enabled && child->expand == FALSE) { if (child->enabled && child->expand == FALSE) {
widget_resize(child, rem_width, widget_resize(child, rem_width, widget_get_desired_height(child));
widget_get_desired_height(child, rem_width));
} }
} }
b->max_size = 0; b->max_size = 0;
@ -202,7 +201,7 @@ static void hori_calculate_size(box *b) {
widget *child = (widget *)iter->data; widget *child = (widget *)iter->data;
if (child->enabled && child->expand == FALSE) { if (child->enabled && child->expand == FALSE) {
widget_resize(child, widget_resize(child,
widget_get_desired_width(child, rem_height), // child->w, widget_get_desired_width(child), // child->w,
rem_height); rem_height);
} }
} }

4
source/widgets/container.c
View file

@ -41,11 +41,11 @@ struct _container {
static void container_update(widget *wid); static void container_update(widget *wid);
static int container_get_desired_height(widget *widget, const int width) { static int container_get_desired_height(widget *widget) {
container *b = (container *)widget; container *b = (container *)widget;
int height = 0; int height = 0;
if (b->child) { if (b->child) {
height += widget_get_desired_height(b->child, width); height += widget_get_desired_height(b->child);
} }
height += widget_padding_get_padding_height(widget); height += widget_padding_get_padding_height(widget);
return height; return height;

6
source/widgets/icon.c
View file

@ -52,8 +52,7 @@ struct _icon {
cairo_surface_t *icon; cairo_surface_t *icon;
}; };
static int icon_get_desired_height(widget *widget, static int icon_get_desired_height(widget *widget) {
G_GNUC_UNUSED const int width) {
icon *b = (icon *)widget; icon *b = (icon *)widget;
int height = b->size; int height = b->size;
if (b->squared == FALSE) { if (b->squared == FALSE) {
@ -68,8 +67,7 @@ static int icon_get_desired_height(widget *widget,
height += widget_padding_get_padding_height(widget); height += widget_padding_get_padding_height(widget);
return height; return height;
} }
static int icon_get_desired_width(widget *widget, static int icon_get_desired_width(widget *widget) {
G_GNUC_UNUSED const int height) {
icon *b = (icon *)widget; icon *b = (icon *)widget;
int width = b->size; int width = b->size;
if (b->squared == FALSE) { if (b->squared == FALSE) {

14
source/widgets/listview.c
View file

@ -206,7 +206,7 @@ static void listview_create_row(listview *lv, _listview_row *row) {
g_list_free_full(list, g_free); g_list_free_full(list, g_free);
} }
static int listview_get_desired_height(widget *wid, const int width); static int listview_get_desired_height(widget *wid);
static void listview_free(widget *wid) { static void listview_free(widget *wid) {
listview *lv = (listview *)wid; listview *lv = (listview *)wid;
@ -319,8 +319,7 @@ static void barview_draw(widget *wid, cairo_t *draw) {
if (lv->barview.direction == LEFT_TO_RIGHT) { if (lv->barview.direction == LEFT_TO_RIGHT) {
for (unsigned int i = 0; i < max && width > 0; i++) { for (unsigned int i = 0; i < max && width > 0; i++) {
update_element(lv, i, i + offset, TRUE); update_element(lv, i, i + offset, TRUE);
int twidth = widget_get_desired_width(WIDGET(lv->boxes[i].box), int twidth = widget_get_desired_width(WIDGET(lv->boxes[i].box));
lv->element_height);
if (twidth >= width) { if (twidth >= width) {
if (!first) { if (!first) {
break; break;
@ -340,8 +339,7 @@ static void barview_draw(widget *wid, cairo_t *draw) {
for (unsigned int i = 0; for (unsigned int i = 0;
i < lv->cur_elements && width > 0 && i <= offset; i++) { i < lv->cur_elements && width > 0 && i <= offset; i++) {
update_element(lv, i, offset - i, TRUE); update_element(lv, i, offset - i, TRUE);
int twidth = widget_get_desired_width(WIDGET(lv->boxes[i].box), int twidth = widget_get_desired_width(WIDGET(lv->boxes[i].box));
lv->element_height);
if (twidth >= width) { if (twidth >= width) {
if (!first) { if (!first) {
break; break;
@ -689,8 +687,7 @@ listview *listview_create(widget *parent, const char *name,
}; };
textbox_text(row.textbox, buff); textbox_text(row.textbox, buff);
} }
// Make textbox very wide. lv->element_height = widget_get_desired_height(WIDGET(row.box));
lv->element_height = widget_get_desired_height(WIDGET(row.box), 100000);
widget_free(WIDGET(row.box)); widget_free(WIDGET(row.box));
lv->callback = cb; lv->callback = cb;
@ -880,8 +877,7 @@ void listview_nav_page_next(listview *lv) {
} }
} }
static int listview_get_desired_height(widget *wid, static int listview_get_desired_height(widget *wid) {
G_GNUC_UNUSED const int width) {
listview *lv = (listview *)wid; listview *lv = (listview *)wid;
if (lv == NULL || lv->widget.enabled == FALSE) { if (lv == NULL || lv->widget.enabled == FALSE) {
return 0; return 0;

3
source/widgets/scrollbar.c
View file

@ -40,8 +40,7 @@
static void scrollbar_draw(widget *, cairo_t *); static void scrollbar_draw(widget *, cairo_t *);
static void scrollbar_free(widget *); static void scrollbar_free(widget *);
static int scrollbar_get_desired_height(widget *wid, static int scrollbar_get_desired_height(widget *wid) {
G_GNUC_UNUSED const int width) {
// Want height we are. // Want height we are.
return wid->h; return wid->h;
} }

12
source/widgets/textbox.c
View file

@ -76,24 +76,16 @@ static void textbox_resize(widget *wid, short w, short h) {
textbox *tb = (textbox *)wid; textbox *tb = (textbox *)wid;
textbox_moveresize(tb, tb->widget.x, tb->widget.y, w, h); textbox_moveresize(tb, tb->widget.x, tb->widget.y, w, h);
} }
static int textbox_get_desired_height(widget *wid, const int width) { static int textbox_get_desired_height(widget *wid) {
textbox *tb = (textbox *)wid; textbox *tb = (textbox *)wid;
unsigned int offset = ((tb->flags & TB_INDICATOR) ? DOT_OFFSET : 0);
if ((tb->flags & TB_AUTOHEIGHT) == 0) { if ((tb->flags & TB_AUTOHEIGHT) == 0) {
return tb->widget.h; return tb->widget.h;
} }
if (tb->changed) { if (tb->changed) {
__textbox_update_pango_text(tb); __textbox_update_pango_text(tb);
} }
int old_width = pango_layout_get_width(tb->layout);
pango_layout_set_width(
tb->layout,
PANGO_SCALE *
(width - widget_padding_get_padding_width(WIDGET(tb)) - offset));
int height = int height =
textbox_get_estimated_height(tb, pango_layout_get_line_count(tb->layout)); textbox_get_estimated_height(tb, pango_layout_get_line_count(tb->layout));
pango_layout_set_width(tb->layout, old_width);
return height; return height;
} }
@ -907,7 +899,7 @@ int textbox_get_estimated_height(const textbox *tb, int eh) {
int height = tb->tbfc->height; int height = tb->tbfc->height;
return (eh * height) + widget_padding_get_padding_height(WIDGET(tb)); return (eh * height) + widget_padding_get_padding_height(WIDGET(tb));
} }
int textbox_get_desired_width(widget *wid, G_GNUC_UNUSED const int height) { int textbox_get_desired_width(widget *wid) {
if (wid == NULL) { if (wid == NULL) {
return 0; return 0;
} }

8
source/widgets/widget.c
View file

@ -646,23 +646,23 @@ int widget_padding_get_padding_width(const widget *wid) {
return width; return width;
} }
int widget_get_desired_height(widget *wid, const int width) { int widget_get_desired_height(widget *wid) {
if (wid == NULL) { if (wid == NULL) {
return 0; return 0;
} }
if (wid->get_desired_height == NULL) { if (wid->get_desired_height == NULL) {
return wid->h; return wid->h;
} }
return wid->get_desired_height(wid, width); return wid->get_desired_height(wid);
} }
int widget_get_desired_width(widget *wid, const int height) { int widget_get_desired_width(widget *wid) {
if (wid == NULL) { if (wid == NULL) {
return 0; return 0;
} }
if (wid->get_desired_width == NULL) { if (wid->get_desired_width == NULL) {
return wid->w; return wid->w;
} }
return wid->get_desired_width(wid, height); return wid->get_desired_width(wid);
} }
int widget_get_absolute_xpos(widget *wid) { int widget_get_absolute_xpos(widget *wid) {