Webmaster Forum / HTML, CSS, Scripts / JavaScript / October 2005

...
var controls = thisForm.elements;
var minPrice = controls.minPrice;
var maxPrice = controls.maxPrice;
...

Try to change it to:

with (thisForm) {
var minPrice = elements['minPrice'];
var maxPrice = elements['maxPrice'];
}

may help / may not. If still not then try element indexes instead of
names:
var minPrice = document.forms[formIndex].elements[elementIndex];

Safary seems to have rather *custom* ideas what's wrong and what's
right in DOM and scripting.

You cannot install Safary or Safary emulator w/o MacOS, I tried.

Works for me, Safari 1.0.3
Mick

It's not unreasonable for the user to be entering commas, which your
function will fail without any explanatory message.

This modified function strips the input of anything but digits and
commas then checks that any commas are positioned correctly.

The input field is modified to show the value under consideration.

Please not that the parameter to this function must be  formElement not
formElement.value as yours uses.

(Watch for line wrap)

function isNumber(n)
{
var str=n.value=n.value.replace(/[^0-9\,]/g,'');

return !( !str.length ||
/^\,|\d{4,}\,|\,\d{4,}|\,\d{0,2}(\D|$)/.test(str) );
}

--
S.C.