Timezone selector

[André Miranda]

Hi Viktor,
I just tested your work, just a small makefile dependency issue while 
building libxfce4ui (see attachment), after that no trouble.

While using I found few points that could be improved:
- The timezone entry should be aligned with Layout and Tooltip format, 
perhaps they all should expand.
- As it takes a couple of seconds to open the "World Map" dialog, an 
impatient user (me) might click more than once thus opening multiples 
dialogs. You can make the dialog modal or at least block the button 
while the dialog is open.
- The button that opens the dialog has no tooltip.
- The dialog title is "xfce4-panel".
- When the user selects a location, no feedback is given, the dialog 
covers its parent, so it's not possible to see that the timezone entry 
has changed. I think that just closing the dialog is enough.

Other than that, this feature is a nice addition, from the user 
perspective (I didn't evaluate the source), IMHO it's almost ready to be 
merged, I hope panel maintainers are able to review it soon.

Thanks your contributions!

Cheers,
Andre Miranda

On 09/03/2017 08:30 PM, Viktor Semykin wrote:
> On 16/08/17 21:19, Viktor Semykin wrote:
>> On 16/08/17 12:59, Viktor Odintsev wrote:
>>> Hi.
>>>
>>> I tested your patches. They are very good but I found some issues.
>>>
>>> 1. libxfce4ui builds both for gtk2 and gtk3. Your custom widget is 
>>> not compatible to gtk2 so now libxfce4ui can't be built without 
>>> --disable-gtk2.
>>> 2. libxfce4ui is not builable with --enable-gtk-doc.
>>
>> Good point. Going to fix it.
>
> If anyone is still interested, this is fixed.
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: diff.patch
Type: text/x-patch
Size: 635 bytes
Desc: not available
URL: <https://mail.xfce.org/pipermail/xfce4-dev/attachments/20170906/60779b3e/attachment.bin>