FLUID-2046: Renderer version of pager throws an error; using IE

Description

Renderer version of pager throws an error; using IE

Steps to reproduce:

1) Open the renderer version of pager
http://build.fluidproject.org/fluid/sample-code/pager/renderer/pager-render.html

Notice that an error is thrown as the page loads

Line: 24
File: pager-render-example.js

Environments

IE 6, IE 7 (Win XP)
IE 7 (Win Vista)

Comments

• Justin Obara commented 2009-01-06T12:06:58.000-0500

  Bug Parade 0.7 release

• Jacob Farber commented 2009-01-08T12:53:49.000-0500

  removed trailing commas

• Antranig Basman commented 2009-01-08T15:06:55.000-0500

  As well as the trailing commas, there is a really fundamental issue that innerHTML is corrupt in IE in an entirely new bizarre and destructive way.

  The retrieved text for parsing reads like this on IE: "<LI class=page-link><A href="#">1</A>
  <LI class=page-link><A href="#">2</A> </LI>" - that is, the closing </LI> tag is completely missing. This is apparently a "well-known and long-standing" issue with all current versions of IE (from IE 5 through IE7) which is attested at these links:

  http://objectmix.com/javascript/120675-ul-innerhtml-ex.html

  http://www.quirksmode.org/bugreports/archives/explorer_7_beta_2/index.html

  The parser will need to be hacked to specially invent the required closing tags.

• Antranig Basman commented 2009-01-09T16:16:17.000-0500

  Fixed at revision 6292 with testcase in RendererTests

• Michelle D'Souza commented 2009-01-13T11:06:26.000-0500

  I reviewed the code commits - they look reasonable.

• Justin Obara commented 2009-01-13T11:24:39.000-0500

  Verified fix using:

  IE 6, IE 7 (Win XP)
  IE 7 (Win Vista)