From: Dana Jansens Date: Thu, 7 Feb 2008 03:16:15 +0000 (-0500) Subject: give prompts a border, and fix how they are laid out. and make them use the multi... X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=84f1f2be1271bd5f5cf8f92d1cc3a56851c688e8;p=chaz%2Fopenbox give prompts a border, and fix how they are laid out. and make them use the multi-line text capabilities in render for the message --- diff --git a/openbox/prompt.c b/openbox/prompt.c index d8400928..d707e31e 100644 --- a/openbox/prompt.c +++ b/openbox/prompt.c @@ -28,8 +28,8 @@ static GList *prompt_list = NULL; static RrAppearance *prompt_a_button; static RrAppearance *prompt_a_hover; static RrAppearance *prompt_a_press; - -#define msg_appearance(self) (ob_rr_theme->osd_hilite_label) +/* we change the max width which would screw with others */ +static RrAppearance *prompt_a_msg; void prompt_startup(gboolean reconfig) { @@ -59,6 +59,9 @@ void prompt_startup(gboolean reconfig) prompt_a_button->texture[0].data.text.color = c_button; prompt_a_hover->texture[0].data.text.color = c_hover; prompt_a_press->texture[0].data.text.color = c_press; + + prompt_a_msg = RrAppearanceCopy(ob_rr_theme->osd_hilite_label); + prompt_a_msg->texture[0].data.text.flow = TRUE; } void prompt_shutdown(gboolean reconfig) @@ -66,6 +69,7 @@ void prompt_shutdown(gboolean reconfig) RrAppearanceFree(prompt_a_button); RrAppearanceFree(prompt_a_hover); RrAppearanceFree(prompt_a_press); + RrAppearanceFree(prompt_a_msg); } ObPrompt* prompt_new(const gchar *msg, const gchar *const *answers) @@ -76,15 +80,17 @@ ObPrompt* prompt_new(const gchar *msg, const gchar *const *answers) const gchar *const *c; attrib.override_redirect = TRUE; + attrib.border_pixel = RrColorPixel(ob_rr_theme->osd_border_color); self = g_new0(ObPrompt, 1); self->ref = 1; self->super.type = OB_WINDOW_CLASS_PROMPT; self->super.window = XCreateWindow(obt_display, obt_root(ob_screen), - 0, 0, 1, 1, 0, + 0, 0, 1, 1, ob_rr_theme->obwidth, CopyFromParent, InputOutput, CopyFromParent, - CWOverrideRedirect, &attrib); + CWOverrideRedirect | CWBorderPixel, + &attrib); window_add(&self->super.window, PROMPT_AS_WINDOW(self)); self->a_bg = RrAppearanceCopy(ob_rr_theme->osd_hilite_bg); @@ -154,7 +160,6 @@ void prompt_unref(ObPrompt *self) static void prompt_layout(ObPrompt *self, const Rect *area) { - RrAppearance *a_msg = msg_appearance(self); gint l, r, t, b; guint i; gint allbuttonsw, allbuttonsh, buttonx; @@ -195,10 +200,11 @@ static void prompt_layout(ObPrompt *self, const Rect *area) self->msg_wbound = MAX(allbuttonsw, area->width*3/5); /* measure the text message area */ - a_msg->texture[0].data.text.string = self->msg.text; - a_msg->texture[0].data.text.maxwidth = self->msg_wbound; - RrMinSize(a_msg, &self->msg.width, &self->msg.height); - a_msg->texture[0].data.text.maxwidth = 0; + prompt_a_msg->texture[0].data.text.string = self->msg.text; + prompt_a_msg->texture[0].data.text.maxwidth = self->msg_wbound; + RrMinSize(prompt_a_msg, &self->msg.width, &self->msg.height); + + g_print("height %d\n", self->msg.height); /* width and height inside the outer margins */ w = MAX(self->msg.width, allbuttonsw); @@ -213,7 +219,7 @@ static void prompt_layout(ObPrompt *self, const Rect *area) for (i = 0; i < self->n_buttons; ++i) { self->button[i].x = buttonx; buttonx += self->button[i].width + BUTTON_SEPARATION; - self->button[i].y = h - allbuttonsh; + self->button[i].y = t + h - allbuttonsh; self->button[i].y += (allbuttonsh - self->button[i].height) / 2; } @@ -251,15 +257,13 @@ static void render_all(ObPrompt *self) RrPaint(self->a_bg, self->super.window, self->width, self->height); - msg_appearance()->surface.parent = self->a_bg; - msg_appearance()->surface.parentx = self->msg.x; - msg_appearance()->surface.parentx = self->msg.y; + prompt_a_msg->surface.parent = self->a_bg; + prompt_a_msg->surface.parentx = self->msg.x; + prompt_a_msg->surface.parentx = self->msg.y; - msg_appearance()->texture[0].data.text.string = self->msg.text; - msg_appearance()->texture[0].data.text.maxwidth = self->msg_wbound; - RrPaint(msg_appearance(), self->msg.window, - self->msg.width, self->msg.height); - msg_appearance()->texture[0].data.text.maxwidth = 0; + prompt_a_msg->texture[0].data.text.string = self->msg.text; + prompt_a_msg->texture[0].data.text.maxwidth = self->msg_wbound; + RrPaint(prompt_a_msg, self->msg.window, self->msg.width, self->msg.height); for (i = 0; i < self->n_buttons; ++i) render_button(self, &self->button[i]);