Fixed in changeset:1474

I am re-opening this ticket since it was only partially implemented. I have submitted a patch for the proposed changes. I have tested it and it should be 100% backwards compatible. It does not result increase the code overhead either.

The ticket originally suggested refactoring the Popup code so it could be reused. There were two suggestions within this:

1) To refactor Popup.create, to allow elements other than 'ul' to be created in a popup. Additionally to allow custom ids and classes. This has already been implemented.

2) Refactor Popup.show so that the code used for positioning the popup can be reused. I believe this was accidentally overlooked, and the patch submitted implements this via a new function Popup.pos

Amongst other things, I have been using the refactored code and the new Popup.pos function for positioning lightboxes, tooltips and mouseover popups. If there was a reason for not implementing this, that I have overlooked, I apologize. If so, what is it?

patch for proposed changes.

Please note that the Popup.pos function accepts an optional 'offset' argument that is an object like {x: 20; y: 5}. This can be used to tweak the exact position of the popup. For instance, if you want it to appear 5 pixels below the root and 10 to the right, pass offset as {x:10, y:0}. There is minimal code overhead involved but it makes the function much more versatile. Since this argument is optional, the function remains perfectly backwards compatible.

Fixed in changeset:2313