<span>Hi Viktor,</span><div><br></div><div>your widget looks and feels pretty awesome - nice work!</div><div>Just to avoid misunderstandings: don't make the widget compatible with Gtk2, fixing the compilation is enough.</div><div><br></div><div>Not sure why and how you would include theming there and I would even (as a themer) say it's not necessary. </div><div><br></div><div>On another note, since you're already working on custom widgetry for the clock, the standard Gtk+ calendar widget is really really bad (even the codebase) and nobody upstream cares about it because gnome has its own thing in the panel. So if you're motivated, that'd be a widget I'd gladly be able to replace (it's also pretty unthemeable and looks really bad with Adwaita by default).</div><div><br></div><div>Cheers</div><div>Simon</div><div><br></div><div><br></div><div><br><br><div class="gmail_quote"><div dir="ltr">On Do., 17. Aug. 2017, 02:25 Viktor Semykin <<a href="mailto:thesame.ml@gmail.com">thesame.ml@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 16/08/17 12:59, Viktor Odintsev wrote:<br>
> Hi.<br>
><br>
> I tested your patches. They are very good but I found some issues.<br>
><br>
> 1. libxfce4ui builds both for gtk2 and gtk3. Your custom widget is not<br>
> compatible to gtk2 so now libxfce4ui can't be built without --disable-gtk2.<br>
> 2. libxfce4ui is not builable with --enable-gtk-doc.<br>
<br>
Good point. Going to fix it.<br>
<br>
> There is also a small GUI issue. I found that you can scroll and zoom<br>
> the widget. That's great, but I can overscroll the widget too much.<br>
> I think that image should never be scrolled over its corners. Is it<br>
> possible to fix? Of course it's not so important as the issues described<br>
> above but it would be very nice to see this fixed too.<br>
<br>
I tried a lot of approaches to scrolling and zooming while creating the<br>
widget. Any restriction to scroll and zoom breaks the behaviour when the<br>
map point under cursor remains still while mouse-wheel-zoom. Although<br>
overscroll restriction may visually look better because the map is never<br>
scrolled out of screen, this also feels totally crappy from UX point of<br>
view. It feels like "oh my why things are moving away?". And also this<br>
is very consistent with every 2GIS system I've tried to use before.<br>
<br>
There's another thing I'm worrying about. The map definitely needs<br>
styling capabilities because it should match user's theme. I have no<br>
idea of how this is achieved and how to make it work on gtk2/3 edge.<br>
<br>
> Everything else looks good for me, good job.<br>
><br>
> Regards,<br>
> Viktor<br>
><br>
> On 11/08/17 06:56, Viktor Semykin wrote:<br>
>> Hello xfce team.<br>
>><br>
>> I have implemented nice timezone selection widget and added it to<br>
>> libxfce4ui and then modified clock plugin in xfce4-panel to use this<br>
>> widget. What is the way to get this code accepted/rejected into<br>
>> mainline xfce code?<br>
>><br>
>> This is my libxfce4ui: [1], this is my xfce4-panel: [2] and this is<br>
>> YouTube video demonstrating the selector in action: [3]<br>
>><br>
>> [1] <a href="https://bitbucket.org/thesame/libxfce4ui/branch/timezone" rel="noreferrer" target="_blank">https://bitbucket.org/thesame/libxfce4ui/branch/timezone</a><br>
>> [2] <a href="https://bitbucket.org/thesame/xfce4-panel/branch/timezone" rel="noreferrer" target="_blank">https://bitbucket.org/thesame/xfce4-panel/branch/timezone</a><br>
>> [3] <a href="https://youtu.be/elHEXriC8oo" rel="noreferrer" target="_blank">https://youtu.be/elHEXriC8oo</a><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" rel="noreferrer" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br>
<br>
--<br>
best, Viktor.<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" rel="noreferrer" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a></blockquote></div></div>