Code review request about gmapcatcher HOT 14 CLOSED

First test on Windows 
  After updating the files the Main "form" will not show up :(, I have not look into
the code yet, will try to find out in a few...

It seem that windows does not like this line:
 gtk.gdk.threads_init()

some more info about this:
 http://www.daa.com.au/pipermail/pygtk/2007-July/014049.html
 http://www.nabble.com/Application-crashes-when-appending-text-to-gtk.ComboBox-while-it-got-the-focus-(MS-Windows)-td198791.html
 http://faq.pygtk.org/index.py?req=show&file=faq14.023.htp

Yes, I've already found it, and now looking for a workaround.

To be completely honest I do not like the way it looks with the download button
(DwnBtn.PNG) 
What about a less invasive approach, like a new menu item (DwnMnu.PNG) in that 
way we
could even send the coordinates to the "DLWindow"

 Also it will be a best to take the "class DLWindow(gtk.Window)" out of maps.py into
it's own file, something similar to mapTools.py

Added a couple of crutches for win32, now it seems to work - see rev.249

I agree that the class DLWindow should be moved into a separate file, but it 
doesn't
look urgent. Moreover, the downloader thread itself should be extracted into a
separate class and reused in the main window and in download.py 

As for a popup menu, it's not a good idea because it hides functionality from 
users.
Actually, until today I didn't even know that popup menu exists in the current 
version.

I totally agree with you! 

 - The "downloader thread" should indeed be a separate class
 - The PopUp menu might be good as a anther way to open the "DLWindow", but we
definitely want to make the download functionality very visible!

I'm looking to see if the download button looks better a line below, with the 
check
boxes, it seems that the combo box that holds the layers is a lot bigger than 
what it
needs...

This could be another option...

I'm not a great guru in GUI, all positions of a download button look equivalent 
for me.

If all the position are equivalent for you, then I will put it in the same row 
of the
check boxes, in that way the "Entry" box is wider. 

Just do it ;)

I been testing it for a while and it all looks ready to merge! 

Merge complete