<div dir="ltr"><div><div>Hello Olivier,<br><br></div>You beat me to it :-) In the implementation I was working on I was trying to use XRender to scale the image making it much more complex. Your implementation is much cleaner.<br><br></div><div>Cheers,<br></div>-Sonal<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 24, 2015 at 1:53 PM, Olivier Fourdan <span dir="ltr"><<a href="mailto:fourdan@gmail.com" target="_blank">fourdan@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi<br>
<br>
Meanwhile I've pushed in git master the implementation I was trying to<br>
explain, it seems to work quite well.<br>
<br>
Cheers,<br>
Olivier<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On 15 January 2015 at 19:46, Sonal Santan <<a href="mailto:sonal.santan@gmail.com">sonal.santan@gmail.com</a>> wrote:<br>
> I hope to look into it soon.<br>
><br>
> -Sonal<br>
><br>
><br>
> On Wed, Jan 14, 2015 at 10:52 AM, Olivier Fourdan <<a href="mailto:fourdan@gmail.com">fourdan@gmail.com</a>> wrote:<br>
>><br>
>> Sonal,<br>
>><br>
>> The implementation I pointed out is actually much simpler than yours,<br>
>> and you get GdkPixbuf as well, I reckon it's much better and efficient<br>
>> than using XGetImage().<br>
>><br>
>> Cheers<br>
>> Olivier<br>
>><br>
>><br>
>> On 13 January 2015 at 15:52, Olivier Fourdan <<a href="mailto:fourdan@gmail.com">fourdan@gmail.com</a>> wrote:<br>
>> > Nope, sorry.<br>
>> ><br>
>> > Olivier<br>
>> ><br>
>> > On 13 January 2015 at 15:08, Sonal Santan <<a href="mailto:sonal.santan@gmail.com">sonal.santan@gmail.com</a>><br>
>> > wrote:<br>
>> >> Hello,<br>
>> >><br>
>> >> IMHO this implementation fits very well with the existing<br>
>> >> infrastructure in<br>
>> >> tabwin which expects a GdkPixbuf to draw onto the window buttons,<br>
>> >> requiring<br>
>> >> minimal changes overall. I do agree that in an ideal world we would not<br>
>> >> use<br>
>> >> XGetImage to capture the contents of the preview pixmap but directly<br>
>> >> render<br>
>> >> the preview over a picture provided by the preview client. But that<br>
>> >> would<br>
>> >> require more changes. Also, I believe this implementation is inline<br>
>> >> with my<br>
>> >> original proposal of using XImage as a mechanism to exchange previews<br>
>> >> to<br>
>> >> clients. The images are small so the overhead should not be huge.<br>
>> >><br>
>> >> Can we go ahead with the current design for now while I will look at<br>
>> >> optimizing the implementation as a future step?<br>
>> >><br>
>> >> -Sonal<br>
>> >><br>
>> >> On Mon, Jan 12, 2015 at 10:10 AM, Olivier Fourdan <<a href="mailto:fourdan@gmail.com">fourdan@gmail.com</a>><br>
>> >> wrote:<br>
>> >>><br>
>> >>> Hi<br>
>> >>><br>
>> >>> I haven't looked much at the code, but using XGetImage and XImage<br>
>> >>> conversion seem inappropriate here.<br>
>> >>><br>
>> >>> Using a compositor, all client windows are unredirected, meaning they<br>
>> >>> are rendered to an offscreen pixmap already, and the compositor has<br>
>> >>> all that's required already, it "knows" about all this so all you need<br>
>> >>> to do is to add hooks in the compositor to retrieve this and use that<br>
>> >>> picture, that's all. Metacity's compositor was based originally on<br>
>> >>> xfwm4's and had that feature up until not too long ago (not sure it's<br>
>> >>> still there), you should look there for an example.<br>
>> >>><br>
>> >>> Also, please note that the gtk_ prefix is for gtk alone.<br>
>> >>><br>
>> >>> Cheers,<br>
>> >>> Olivier<br>
>> >><br>
>> >><br>
>> >><br>
>> >> _______________________________________________<br>
>> >> Xfce4-dev mailing list<br>
>> >> <a href="mailto:Xfce4-dev@xfce.org">Xfce4-dev@xfce.org</a><br>
>> >> <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br>
>> _______________________________________________<br>
>> Xfce4-dev mailing list<br>
>> <a href="mailto:Xfce4-dev@xfce.org">Xfce4-dev@xfce.org</a><br>
>> <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> Xfce4-dev mailing list<br>
> <a href="mailto:Xfce4-dev@xfce.org">Xfce4-dev@xfce.org</a><br>
> <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br>
_______________________________________________<br>
Xfce4-dev mailing list<br>
<a href="mailto:Xfce4-dev@xfce.org">Xfce4-dev@xfce.org</a><br>
<a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br>
</div></div></blockquote></div><br></div>