my booking form is a secret contains a hidden booking form. They don't seem to like bookings.

The reason? The booking form is within a table that is styled to be hidden:

<table class="fullwidth" id="hometable" style="clear: both; visibility:hidden;">

Another part of the code is hidden from non-IE browsers with conditional comments:

..and there is some scripting that will remove the "loading" screen and show the search form if UA is IE. Naturally since the "loading" message is hidden with conditional comments, this:

(document.getElementById("loadMessage")).style.display = "none";

fails and stops the script before the table is un-hidden. Also obviously, they don't expect any booking visitors with JavaScript disabled, since then the form would simply never appear.

"Thomsonfly – the low fares airline that goes further to confuse you."

13 thoughts on “my booking form is a secret

  1. Opera is killed by its "document.all" support. Again.IMO, that's a really good candidate for a new site preference.

  2. Well, I'm pushing for at LEAST making it deletable again so that one could use a simple user.js to fake no document.all support, would be really interesting to surf with that. (I'm pretty sure it used to be settable or deletable but that "feature" seems to have fallen through a gap somewhere.)

  3. Opera's support for document.all can be disabled with a UserJS (the variable can be turned "magic"). No separate site preference is needed….I always wondered what goes on inside the heads of authors of such web sites.

  4. Such web developers probably don't know (care) about Opera too much. I.e. they may don't know that Opera supports both document.all and document.getElementById…

  5. @Fatal: I agree with that one.document.all must evaluate to false whenever inserted in a boolean expression, yet document.all['foobar'] or document.all.foobar should work, this way not breaking the script, and prevent stupid IE browser sniffing that breaks Opera.This is the best idea possible regarding document.allI advise for the same to happen with attachEvent.Many scripts sniff for attachEvent to clean symbols and stuff defined in scripts (to prevent IE's memory leaks) which break webpages when using the back button.

  6. xErath: this will break scripts which properly use document.all (not for browser sniffing but for detection of this particular feature):if (document.all) { // if document.all is there, use it for quick retrieval e = document.all['xyzzy'];} else { // do some complex traversal}It's OK if the script does just this, but it might just refuse to work without document.all because it needs that.

  7. // do some complex traversal

    like document.getElementById ?!?document.all is mostly used for sniffing.There are very few, rare, obscure cases of websites relying on this object only, not providing a DOM branch.Opera would gain more by hiding it, than keeping the object visible.That's what webkit did.

  8. Originally posted by ale5000:

    The best idea is insert an option in site preferences to enable/disable document.all

    Most user don't have the smallest idea of what document.all is and its implications.It should be hidden by default, and visible only when masking as IE, but always present and supported.I've seen too many webpages sniffing for document.all and then trying to use ActiveXObject, using setProperty/selectNodes, or providing stupid hacks for lack of hover in other elements besides anchors.Of interest

Leave a Reply

Fill in your details below or click an icon to log in: Logo

You are commenting using your account. Log Out / Change )

Twitter picture

You are commenting using your Twitter account. Log Out / Change )

Facebook photo

You are commenting using your Facebook account. Log Out / Change )

Google+ photo

You are commenting using your Google+ account. Log Out / Change )

Connecting to %s