harootun:
Is there any downside to not having that javascript behavior in the <body> tag?
There are actually two ways of how to place a self.close JavaScript code over the window. You are right, the first one is like now, when this event is placed over the whole body. Some browsers consider to close such window also when you click the right mouse button. But for pop-ups like we have in this add-on, it is more suitable, because then it allows to place a background image very precisely.
I already tried to implement the other way of how the self.close event is placed, in <a href...> tag over the picture itself. In that case it will work only when you press the left mouse button. However there is another trouble, because such code would work ideally only with wrong HTML definition. If we define XHTML transitional, it will add some scrollbars to the pop-up window, and it all will start to look terrible.
So there will be no changes regarding this case... I suppose, users who need to save pictures, could do it anyway in another browser, or as mentioned, just set up the pop-ups so they close "regular" way.