rem: maxLimitHeight = isPercentHeight ? 100 : maxPixels; 
add: maxLimitHeight = isPercentHeight ? 100 : maxPixels; 
context: function ValidateData(){  if ( !IsValidImage(dialog.srcInput.value )) {    ShowInputErrorMessage(GetString("MissingImageError"));    return false;  }  //TODO: WE NEED TO DO SOME URL VALIDATION HERE, E.G.:  // We must convert to "file:///" or "http://" format else image doesn't load!  var src = dialog.srcInput.value.trimString();  globalElement.setAttribute("src", src);    // Note: allow typing spaces,   // Warn user if empty string just once per dialog session  //   but don't consider this a failure  var alt = dialog.altTextInput.value;  if (doAltTextError && !alt)  {    ShowInputErrorMessage(GetString("NoAltText"));    SetTextboxFocus(dialog.altTextInput);    doAltTextError = false;    return false;  }  globalElement.setAttribute("alt", alt);    var width = "";  var height = "";  var isPercentWidth, isPercentHeight;  var maxLimitWidth, maxLimitHeight;  if (dialog.actualSizeRadio.checked)  {    width = actualWidth ? actualWidth : "";    height = actualHeight ? actualHeight : "";  }  else  {	  isPercentWidth = (dialog.widthUnitsMenulist.selectedIndex == 1);	  isPercentHeight = (dialog.heightUnitsMenulist.selectedIndex == 1);	  maxLimitWidth = isPercentWidth ? 100 : maxPixels;  // Defined in EdDialogCommon.js    width = ValidateNumberString(dialog.widthInput.value, 1, maxLimitWidth);    if (width == "")    {      if ( !SeeMore )        onMoreFewerImage();      SetTextboxFocus(dialog.widthInput);      return false;    }    if (isPercentWidth)      width = width + "%";	  maxLimitHeight = isPercentHeight ? 100 : maxPixels;  // Defined in EdDialogCommon.js    height = ValidateNumberString(dialog.heightInput.value, 1, maxLimitHeight);    if (height == "")    {      if ( !SeeMore )        onMoreFewerImage();      SetTextboxFocus(dialog.heightInput);      return false;    }    if (isPercentHeight)      height = height + "%";  }  // We always set the width and height attributes, even if same as actual.  //  This speeds up layout of pages since sizes are known before image is downloaded  // (But don't set if we couldn't obtain actual dimensions)  if (width)    globalElement.setAttribute("width", width);  else    globalElement.removeAttribute("width");  if (height)    globalElement.setAttribute("height", height);  else    globalElement.removeAttribute("height");  // spacing attributes  var amount;	if ( dialog.imagelrInput.value )	{	  amount = ValidateNumberString(dialog.imagelrInput.value, 0, maxPixels);	  if (amount == "")	    return false;    globalElement.setAttribute( "hspace", amount );  }  else    globalElement.removeAttribute( "hspace" );	if ( dialog.imagetbInput.value )	{	  var vspace = ValidateNumberString(dialog.imagetbInput.value, 0, maxPixels);	  if (vspace == "")	    return false;	  globalElement.setAttribute( "vspace", vspace );	}	else	  globalElement.removeAttribute( "vspace" );  // note this is deprecated and should be converted to stylesheets	if ( dialog.border.value )	{	  var border = ValidateNumberString(dialog.border.value, 0, maxPixels);	  if (border == "")	    return false;	  globalElement.setAttribute( "border", border );	}	else	  globalElement.removeAttribute( "border" );	// Default or setting "bottom" means don't set the attribute  // Note that the attributes "left" and "right" are opposite  //  of what we use in the UI, which describes where the TEXT wraps,  //  not the image location (which is what the HTML describes)	switch ( dialog.alignTypeSelect.data )	{	  case "top":	  case "center":	  case "right":	  case "left":      globalElement.setAttribute( "align", dialog.alignTypeSelect.data );      break;    default:      globalElement.removeAttribute( "align" );	}  return true;}
meta: 10234	/local1/tlutelli/issta_data/temp/all_js/javascript/2006_temp/2006/10234/0b2dd2b2d2fd037df917e07645fa019a963332a5/EdImageProps.js/clean/editor/ui/dialogs/content/EdImageProps.js