Re: [zathura] [PATCH] Center pages

2014-01-20 Thread Sebastian Ramacher
On 2014-01-19 20:29:40, akbjker wrote: > >From a first look at it, this leaks all old GtkAlignment instances on every > >page_widget_set_mode call. > > Whoops, you're right. I misunderstood the purpose of gtk_widget_reparent. > > Here's the patch v2. Thank you. I've pushed it for now. We probab

Re: [zathura] [PATCH] Center pages

2014-01-20 Thread akbjker
>From a first look at it, this leaks all old GtkAlignment instances on every >page_widget_set_mode call. Whoops, you're right. I misunderstood the purpose of gtk_widget_reparent. Here's the patch v2. diff --git a/zathura.c b/zathura.c index 75aabc5..49a78b0 100644 --- a/zathura.c +++ b/zathura.c

Re: [zathura] [PATCH] Center pages

2014-01-19 Thread Sebastian Ramacher
On 2014-01-18 02:08:04, akbjker wrote: > diff --git a/zathura.c b/zathura.c > index 1f5f4fd..5335def 100644 > --- a/zathura.c > +++ b/zathura.c > @@ -943,10 +943,6 @@ document_close(zathura_t* zathura, bool keep_monitor) > >/* remove widgets */ >gtk_container_foreach(GTK_CONTAINER(zathura

Re: [zathura] [PATCH] Center pages

2014-01-19 Thread Sebastian Ramacher
On 2014-01-19 17:12:09, akbjker wrote: > Basically, all of the ZathuraPages get resized to the largest one, and > GtkDrawingArea's default background color gets drawn in the leftover > space of the smaller pages. > It's more noticeable when pages are grossly different sizes, like in > this attached

Re: [zathura] [PATCH] Center pages

2014-01-19 Thread Sebastian Ramacher
On 2014-01-18 02:08:04, akbjker wrote: > Quick patch to center-align pages that have different sizes. > Also fixes a bug that forces all the ZathuraPages to the size of the > largest one, therefore causing GTK's default background color to be > drawn instead of the one set in zathurarc. > > This f

[zathura] [PATCH] Center pages

2014-01-19 Thread akbjker
Quick patch to center-align pages that have different sizes. Also fixes a bug that forces all the ZathuraPages to the size of the largest one, therefore causing GTK's default background color to be drawn instead of the one set in zathurarc. This feels kind of hacky, though. In the long term, it mi