[tabacco]

Just a heads-up for anyone who cares... the reviews page no longer has a iframe. Everything is now taken care of using AJAX (which updates the content without having to refresh the page). W00t.

[Intrepid Homoludens]

Much better! Thanks, tabs.

[RLacey]

My irrational loathing of frames is sated .

[fov]

Err... I kind of liked the frame...

*runs*

[Fairygdmther]

Looks good and is easy to navigate! You need to fix the title on Salammbo - it got mucked up, though.

Lynsie

[fov]

Thanks for the heads up. I went to fix it but it's correct in the database. Apparently whatever Doug did doesn't like the ô with the little hat on it.

[Jeysie]

Erm. I get the feeling I'm missing something... when I click on the letters or sort links, nothing happens. I can't get anything other than the first page of reviews to load.

I'm running Opera 8.01 with javascript enabled, if that helps.

Personally I'd rather just have one plain, boring ol' list, but maybe that's just me... can someone tell me what the link is to the "All" version so I can just bookmark that?

Peace & Luv, Liz

[Dale Baldwin]

It does seem somewhat less responsive than the old iframe version.

[tabacco]

Originally Posted by Jeysie

Erm. I get the feeling I'm missing something... when I click on the letters or sort links, nothing happens. I can't get anything other than the first page of reviews to load.

I'm running Opera 8.01 with javascript enabled, if that helps.

Personally I'd rather just have one plain, boring ol' list, but maybe that's just me... can someone tell me what the link is to the "All" version so I can just bookmark that?

Peace & Luv, Liz

Hmm... apparently opera doesn't support AJAX properly. Figures. I'll see what I can do later tonight.

[tabacco]

Originally Posted by Dale Baldwin

It does seem somewhat less responsive than the old iframe version.

Shouldn't be. The same basic thing is taking place under the hood, in that you click a link, Javascript forms a URL, requests the URL, loads a new page, and then your browser renders it. The only difference is that instead of the html returned being dumped into an iframe, it's being dumped into a div.

[Dale Baldwin]

Maybe I'm just being impatient with it and/or my router's having a hissy fit with the number of connections the reviews list opens (requesting the large number of ratings stars).

I've never played with the page this much before.

[tabacco]

Well, to make it a little clearer I added a "Loading..." message that will appear while the request is happening. That way it won't seem like you click and nothing happens if the request is slow.

[BenjaminBunny]

I love you Tabacco. *hugs AG web master*

Thank you! Those frames were pretty annoying.

[tabacco]

Okay, here's your nightly Opera Update:

Opera does not support xmlHttpRequest.overrideMimeType(), along with several other methods of xmlHttpRequest. It would seem that for 8.0 the Opera developers implemented only the methods needed to make gmail work, and nothing more. This == bullshit. End result, I got it working in Opera, but reordering the list causes the page background to go white. That's pretty much how it's going to stay, unless some enterprising Opera user wants to dig through our Javascript and find a solution. Sorry to leave it at that, but Opera represents <3% of our browser share, and I have lots of other stuff to get done

For the truly bored:

function xmlhttpPost(strURL, strSubmit, strResultFunc) {

	var xmlHttpReq = false;

	// Mozilla/Safari/Opera/Everything other than ****ing IE

	if (window.XMLHttpRequest) {

		xmlHttpReq = new XMLHttpRequest();

		//xmlHttpReq.overrideMimeType('text/xml');

	}

	// IE

	else if (window.ActiveXObject) {

		xmlHttpReq = new ActiveXObject("Microsoft.XMLHTTP");

	}

        xmlHttpReq.open('POST',strURL,true);

        xmlHttpReq.setRequestHeader('Content-Type', 'application/x-www-form-urlencoded');

	xmlHttpReq.onreadystatechange = function() {

		if (xmlHttpReq.readyState == 4) {

			strResponse = xmlHttpReq.responseText;

			switch (xmlHttpReq.status) {

			   case 404:

				   alert('The server returned a 404 on ' + strURL);

				   break;

			   case 403:

				   alert('The server returned a 403 on ' + strURL);

				   break;

			   default:

				   if (strResponse.indexOf('Error:') > -1 || strResponse.indexOf('Debug:') > -1) {

					   alert(strResponse);

				   } else {

					   eval(strResultFunc + '(strResponse);');

				   }

				   break;

			}

		}

	}

	xmlHttpReq.send(strSubmit);

}

... Oh.

[Jeysie]

Thank you. Considering that 1. the important bit works, 2. Opera's been improving their xmlHttpRequest support with each new version anyway, and 3. you didn't tell me to flip off and download a different browser, I'm pretty happy.

Peace & Luv, Liz

[tabacco]

Well, don't get me wrong, I still think you should get a better browser

[Jeysie]

If there was a better browser to get, I would.

Seriously, Firefox is all right, but I'd have to download a ton of extensions to get all the stuff I'm used to in Opera, so I can live with the (very) occasional rendering quirk. Plus IE and FF both run rather like molasses on my creaky old computer.

Not to mention that Opera supports pretty much everything else (all the W3C standards, at any rate)...

I'm an "Anything but IE" sort of person, anyway.

Peace & Luv, Liz