<div dir="ltr"><div>Hello,<br><br>Are there any plans on integrating the preview feature in xfce4-panel? I would be helpful for the panel tasklist to show window preview in tooltip instead of (or in addition to) the window name. That would help the user identify the correct application window.<br><br></div><div>Any thoughts on how the panel could get access to the window pixmap?<br><br></div>-Sonal<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 24, 2015 at 7:32 PM, Sonal Santan <span dir="ltr"><<a href="mailto:sonal.santan@gmail.com" target="_blank">sonal.santan@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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="HOEnZb"><div class="h5"><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><div><br>
<br>
On 15 January 2015 at 19:46, Sonal Santan <<a href="mailto:sonal.santan@gmail.com" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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>
</div></div></blockquote></div><br></div>