YourSearchPlugin error "this command is not supported" under IE 8.

[rouilj]

Hello all:

I am using IE 8 with:

http://tiddlywiki.abego-software.de/#YourSearchPlugin version 2.1.3
(2008-04-16) under TW 2.5.2

and when I load the TW I see the following error when I click on the
red error box under the
search box in the sidebar:

Error while executing macro <<search>>
Error: this command is not supported

However the search box works as I would expect using your search, I
can enter specific
your search syntax and get what appears to be proper search results.

My attempts to debug this have failed so far. The builtin IE debugger
isn't showing any error
and I haven't been able to get firebug lite to run at all in this
environment.

Does this issue ring any bells?

To reproduce, download empty.html from the tw site, import the plugin.
save and reload.
You should see a red error bar under the search box.

Hmm, a new data point just as I was about to post, if using the built
in debugger I change the
document mode from IE 8 standards to IE 7 standards, the error
dissapears. Also if I change the
browser mode from ie 8 to ie7 mode, the error block dissapears. Also
using IE 8 compatibility mode
seems to make the error dissapear. So I guess I have a workaround, but
having to get into the
debugger and change the mode is kind of a pain.

Thanks for any thoughts on solving it so it works in default IE 8
mode.

-- rouilj

[Tobias]

Hi Rouijo,

I am seeing the same error. But it looks like that's an "official" one
as it is not fixed on abego's site either.
See here: http://i26.tinypic.com/6y31at.png

Tobias.

On Jul 31, 5:55 am, rouilj <rou...@cs.umb.edu> wrote:
> Hello all:
>
>  I am using IE 8 with:
>

>  http://tiddlywiki.abego-software.de/#YourSearchPluginversion 2.1.3

> (2008-04-16) under TW 2.5.2
>
> and when I load the TW I see the followingerrorwhen I click on the

> rederrorbox under the

> search box in the sidebar:
>

>  Errorwhile executing macro <<search>>

>  Error: this command is not supported
>
> However the search box works as I would expect using your search, I
> can enter specific
> your search syntax and get what appears to be proper search results.
>
> My attempts to debug this have failed so far. The builtin IE debugger
> isn't showing anyerror
> and I haven't been able to get firebug lite to run at all in this
> environment.
>
> Does this issue ring any bells?
>
> To reproduce, download empty.html from the tw site, import the plugin.
> save and reload.

> You should see a rederrorbar under the search box.

>
> Hmm, a new data point just as I was about to post, if using the built
> in debugger I change the
> document mode from IE 8 standards to IE 7 standards, theerror
> dissapears. Also if I change the

> browser mode from ie 8 to ie7 mode, theerrorblock dissapears. Also

> using IE 8 compatibility mode

> seems to make theerrordissapear. So I guess I have a workaround, but

[Tobias]

As for "official" ...what I meant was, I don't think I am able to look
into it, nor would I know to what degree ie developper tools are of
any help in that respect.

;o)

Tobias.

[rouilj]

On Jul 30, 11:55 pm, rouilj <rou...@cs.umb.edu> wrote:
> Hello all:
> I am using IE 8 with:
>

>  http://tiddlywiki.abego-software.de/#YourSearchPluginversion 2.1.3

> (2008-04-16) under TW 2.5.2
>
> and when I load the TW I see the following error when I click on the
> red error box under the search box in the sidebar:
>
>   Error while executing macro <<search>>
>   Error: this command is not supported
>

It looks like this bit of code is the problem:

var txt = createTiddlyElement(place,"input",null,null,null);
if(params[0])
txt.value = params[0];
txt.onkeyup = keyHandler;
txt.onfocus = focusHandler;
txt.setAttribute("size",sizeTextbox);
txt.setAttribute("accessKey",this.accessKey);
txt.setAttribute("autocomplete","off");
if(config.browser.isSafari)
{
txt.setAttribute("type","search");
txt.setAttribute("results","5");
}
else
;//txt.setAttribute("type","text");

specifically the last commented out line. According to
https://connect.microsoft.com/IE/feedback/ViewFeedback.aspx?FeedbackID=332453
you can't change the type of an element after it has been rendered
into the DOM. Commenting out that line as shown above stops
the macro error, but it is obviously not a solution.

Does Udo or anybody from Abego software read this group? I have a
feeling this will need their efforts to fix. I will also
drop the author an email pointing to this thread.

-- rouilj

[Tobias]

Good job,

I hope Udo will read this and have a look at it.

Tobias.

On 29 Aug., 19:49, rouilj <rou...@cs.umb.edu> wrote:
> On Jul 30, 11:55 pm, rouilj <rou...@cs.umb.edu> wrote:
>
> > Hello all:
> > I am using IE 8 with:
>
> >  http://tiddlywiki.abego-software.de/#YourSearchPluginversion2.1.3
> > (2008-04-16) under TW 2.5.2
>
> > and when I load the TW I see the following error when I click on the
> > red error box under the search box in the sidebar:
>
> >   Error while executing macro <<search>>
> >   Error: this command is not supported
>
> It looks like this bit of code is the problem:
>
>         var txt = createTiddlyElement(place,"input",null,null,null);
>         if(params[0])
>                 txt.value = params[0];
>         txt.onkeyup = keyHandler;
>         txt.onfocus = focusHandler;
>         txt.setAttribute("size",sizeTextbox);
>         txt.setAttribute("accessKey",this.accessKey);
>         txt.setAttribute("autocomplete","off");
>         if(config.browser.isSafari)
>                 {
>                 txt.setAttribute("type","search");
>                 txt.setAttribute("results","5");
>                 }
>         else
>                 ;//txt.setAttribute("type","text");
>

> specifically the last commented out line. According tohttps://connect.microsoft.com/IE/feedback/ViewFeedback.aspx?FeedbackI...

[ub]

Hi,

I have no chance to tryout the stuff on IE8 but I assume the following
change (to the code block given above) should fix the issue:

var txt = createTiddlyElement(null,"input",null,null,null);

if(params[0])
txt.value = params[0];
txt.onkeyup = keyHandler;
txt.onfocus = focusHandler;
txt.setAttribute("size",sizeTextbox);
txt.setAttribute("accessKey",this.accessKey);
txt.setAttribute("autocomplete","off");
if(config.browser.isSafari)
{
txt.setAttribute("type","search");
txt.setAttribute("results","5");
}
else

txt.setAttribute("type","text");

if(place)
place.appendChild(txt);

(i.e. pass "null" as the createTiddlyElement parent and add the newly
create element to the DOM tree after it is fully created)

It would be nice if some IE8 user could verify if this fixes the
issue. I will then release a new revision of the YourSearchPlugin
soon.

Udo



If somebody could

[Tobias]

Hi Udo,

That did it! Can you explain (in short) why that was a problem in the
previous code?

Tobias.

[rouilj]

On Aug 31, 7:56 am, ub <udo.borkow...@googlemail.com> wrote:
> I have no chance to tryout the stuff on IE8 but I assume the following
> change (to the code block given above) should fix the issue:
>

> [...]

> (i.e. pass "null" as the createTiddlyElement parent and add the newly
> create element to the DOM tree after it is fully created)
>
> It would be nice if some IE8 user could verify if this fixes the
> issue. I will then release a new revision of the YourSearchPlugin
> soon.

That seems to solve the problem (I hacked the change into the
minimized code supplied with the plugin).
I checked the change using standard IE 8 and with it running in IE 7
compatibility mode.

Any idea when you can get a release? I have a deadline on the 14th and
I would like to get the updated plugin
in the release.

-- rouilj

[rouilj]

ub wrote:
> It would be nice if some IE8 user could verify if this fixes the
> issue. I will then release a new revision of the YourSearchPlugin
> soon.

Also Udo opera 10 complans about the use of:

color: none
background-color: none

in the plugin. Maybe using 'transparent' in place of none would work
(it's in css1 so should be accepted).

-- rouilj

[ub]

I just released a new revision (2.1.4) of the YourSearchPlugin that
fixes the "this command is not supported" error under IE 8:

http://tiddlywiki.abego-software.de/#YourSearchPlugin

> That did it! Can you explain (in short) why that was a problem in the
> previous code?

As Microsoft explains at https://connect.microsoft.com/IE/feedback/ViewFeedback.aspx?FeedbackID=332453
one "Cannot change an input's type after you've placed it in the
page". Therefore I now define the type of the field (using
'txt.setAttribute("type","text");') before the field is placed into
the page/added to the DOM tree.

Udo

[Tobias]

Thanks for the feedback, Udo... much appreciated. Tobias.

On 4 Sep., 13:19, ub <udo.borkow...@googlemail.com> wrote:
> I just released a new revision (2.1.4) of the YourSearchPlugin that
> fixes the "this command is not supported" error under IE 8:
>
> http://tiddlywiki.abego-software.de/#YourSearchPlugin
>
> > That did it! Can you explain (in short) why that was a problem in the
> > previous code?
>

> As Microsoft explains athttps://connect.microsoft.com/IE/feedback/ViewFeedback.aspx?FeedbackI...

[rouilj]

On Sep 4, 7:19 am, ub <udo.borkow...@googlemail.com> wrote:
> I just released a new revision (2.1.4) of the YourSearchPlugin that
> fixes the "this command is not supported" error under IE 8:

Thanks Udo. I put in in my coursebook and the problem seems
to be solved.

-- rouilj