Fixes #24 - The IFrame Shim for IE produced some error and did not fi…

…nish loading the IFrame
arian · Nov 5, 2010 · 1096e95 · 1096e95
1 parent e93d221
commit 1096e95
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/Source/datepicker.js b/Source/datepicker.js
@@ -315,7 +315,7 @@ var DatePicker = new Class({
 	shim: function(){
 		var coords = this.picker.setStyle('zIndex', 1000).getCoordinates();
 		var frame = this.frame = new Element('iframe', {
-			src: 'javascript:false;document.write("");',
+			src: 'javascript:false;document.write("");document.close()',
 			styles: {
 				position: 'absolute',
 				zIndex: 999,
@@ -326,7 +326,7 @@ var DatePicker = new Class({
 		frame.style.filter = 'progid:DXImageTransform.Microsoft.Alpha(style=0,opacity=0)';
 
 		this.addEvent('close', function(){
-			frame.destroy()
+			if (frame && frame.destroy) frame.destroy()
 		});
 
 		if (this.dragger){

diff --git a/Test/index.html b/Test/index.html
@@ -135,7 +135,10 @@ <h2>Demo and Examples</h2>
 
 <p>
 <label>Datepicker with Vista skin:</label><br />
-<input type="text" value="" class="date demo_vista" />
+<input type="text" value="" class="date demo_vista" /><br />
+<select>
+	<option>IE should shim this!</option>
+</select>
 </p>
 
 <p>