simonbaird / mGSD

The TiddlyWiki powered GTD system formerly known as MonkeyGTD
http://mgsd.tiddlyspot.com/
68 stars 19 forks source link

Error in macro <<search>> in IE8 #18

Open simonbaird opened 14 years ago

simonbaird commented 14 years ago

And check for any other other IE problems.

Jobirder commented 14 years ago

It's not a bug, it's a feature --> use Chrome! =P

rgallion commented 12 years ago

I duplicated this issue in IE8 and mGSD 3.1.9 beta. The error does not occur if the bundled 'YourSearchPlugin 2.1.3' plugin is disabled. The error is occuring because the plugin is attempting to set the 'type' of the search 'input' element to 'text'. Since the default 'type' for an 'input' element in IE8 is 'text', this is not necessary so the following code:

``` txt.onkeyup=_163;txt.onfocus=_166;txt.setAttribute(&quot;size&quot;,_16a);txt.setAttribute(&quot;accessKey&quot;,this.accessKey);txt.setAttribute(&quot;autocomplete&quot;,&quot;off&quot;);if(config.browser.isSafari){txt.setAttribute(&quot;type&quot;,&quot;search&quot;);txt.setAttribute(&quot;results&quot;,&quot;5&quot;);}else{txt.setAttribute(&quot;type&quot;,&quot;text&quot;);} can be changed to: txt.onkeyup=_163;txt.onfocus=_166;txt.setAttribute(&quot;size&quot;,_16a);txt.setAttribute(&quot;accessKey&quot;,this.accessKey);txt.setAttribute(&quot;autocomplete&quot;,&quot;off&quot;);if(config.browser.isSafari){txt.setAttribute(&quot;type&quot;,&quot;search&quot;);txt.setAttribute(&quot;results&quot;,&quot;5&quot;);}

See the 'Remarks' section of http://msdn.microsoft.com/en-us/library/ms534700(v=vs.85).aspx for details. I will contact the owner of the YourSearchPlugin with this information.

abego commented 12 years ago

Hi,

I just released YourSearchPlugin v2.1.6:

 http://tiddlywiki.abego-software.de/#%5B%5BYourSearchPlugin%20v2.1.6%20released%5D%5D

This should fix the issue.

Udo