Comments on: updating moopop for mootools 1.2

By: DrummerHead

DrummerHead — Wed, 28 Jan 2009 19:23:30 +0000

Hello gonchuki!

I am DrummerHead, and I am also Uruguayan :) I'm a Graphic designer who's been learning mootools and javascript to implement it in my current proyect, and this extension you've made right here really helped. Simple and straight to the point.

Good work.

By: Gavin

Gavin — Sun, 04 Jan 2009 19:41:42 +0000

I had the same problem as Danny. If I used moopop_mini.js it didn't do anything, but if I then went to moopop.js it worked.

By: gonchuki

gonchuki — Wed, 24 Dec 2008 17:39:27 +0000

You must be doing something wrong, like possibly mixing mootools with another framework that extends native objects (Prototype for example), as both "each" and "addEvent" are mootools functions and have nothing to do with my plugin.
Keep in mind that this kind of mixing is not supported by neither mootools nor prototype (or anything you could be using that also implements those or other methods) as those frameworks are designed to be the only ones present in your page using native object extension.

By: Danny Foo

Danny Foo — Wed, 24 Dec 2008 08:03:33 +0000

Just wanted to share I tried implementing this today – didn't work. Got the core builder files from MooTools and when I had all in place, I get the JS errors:
Hash.each is not a function
window.addEvent is not a function

I'm a website designer and not a developer so it's difficult to troubleshoot this on my own.

By: RayDeeA

RayDeeA — Wed, 17 Dec 2008 12:53:52 +0000

To make scrollbars work with rel="popup[800,700,r]"
change
if (size[1]) el.store('popupprops', 'width=' + size[1] + ', height=' + size[2] + (resizable[1] ? ', scrollbars=yes, resizable=yes' : ', scrollbars=yes') );

to
if (size[1]) el.store('popupprops', 'width=' + size[1] + ', height=' + size[2] + (resizable ? ', scrollbars=yes, resizable=yes' : ', scrollbars=yes') );

By: Vincent

Vincent — Fri, 14 Nov 2008 12:16:15 +0000

Thanks for this script. I was wondering about something: is it possible to check if a popup is already open? I have a popup on a site that I want to open just once. It is opened like this:

window.addEvent('domready', function(){
moopop.popup($('popup-link'));
}
});

If the popup is already open, I don't want to allow the user to open it again. Also, if the user goes to another page I don't want it to open if it is already open.

By: gonchuki

gonchuki — Wed, 17 Sep 2008 21:51:55 +0000

the popup window should automatically get focus when opened if you don't click something else, and in case you did take focus away by clicking something then that was the decision of the user and we as UI designers should not impose the window on their noses.

By: Al

Al — Thu, 21 Aug 2008 17:46:07 +0000

changing the popup function to include adding focus to the popup was very useful for me, you should think about including it:

popup: function(el) {
var w = window.open(el.get('href'), el.get('name') || ", el.retrieve('popupprops') || ");
if (window.focus)
w.focus();
}

By: Simon

Simon — Wed, 09 Jul 2008 09:50:09 +0000

Hello

Great script

I want the scipt to function on all my links. Is there any way I can achieve this without writing rel="popup" in every a href?

By: links for 2008-06-25 | iKeif

links for 2008-06-25 | iKeif — Wed, 25 Jun 2008 17:31:34 +0000

[...] updating moopop for mootools 1.2 | a pint of javascript (and general web stuff) Okay – this isn't exactly groundbreaking, but when you know you've got to do it, or have someone update them, why not make popups easy? (tags: mootools webdev popup _blank code dev example) [...]