Improving subject line

In testing the changes to the ResponseWriters (ICE-5664), the compatible version of Component Showcase was showing a couple of major issues.

One was that the Calendar demo would not render properly when the menu link was clicked. This only affected WebKit based browsers (Chrome and Safari). Firefox worked fine. I could see that the update was being retrieved but not applied. Debugging showed that WebKit considered the update to be malformed XML. I guessed that it might be the   entities and removed them from the SelectInputDateRenderer class and that worked.

The other was that Google Maps throwing a JavaScript exception on the initial load of the first page. Some Googling turned up a blog (http://www.jroller.com/HazemBlog/entry/uncaught_typeerror_object_a_document) that stated the following regarding the exact same error:

Uncaught TypeError: Object #<a Document#> has no method 'write'

Error Conditions:
This issue happens when you use GMaps4JSF with Google Chrome (also Safari) in Facelets XHTML files (The default PDL for JSF 2.0). Actually this issue generally happens when you make make any include to Google Maps inside XHTML files:
<script src="http://maps.google.com/maps?file=api&v=2&sensor=false&key=ABQIAAAAxrVS1QxlpJHXxQ2Vxg2bJBQdkFK-tWRbPPQS4ACM1pq_e-PltxQXeyH20wQuqDaQ_6EM5UeGGVpnIw"> </script>

Solution:
Just set the content type to "text/html":
<f:view contentType="text/html">

Adding the contentType to the f:view tag did solve the problem. Basically, it overrides the main issue outlined in the original problem description - that some browsers "prefer" application-xhtml and will therefore have that sent back as the content type. Since xhtml is stricter about many things, I've adjusted what I could to make things more compliant. However, GMaps won't work without this setting. I've applied it to the page-template.xhtml file of the Component Showcase and GMaps does work again. This is likely something we'll need to note as a general solution to problems with strict XHTML adherence.

For educational purposes, here's the diff of the changes that I had made to the renderer that allowed it to be XHTML compliant (remove the   entities).

Index: components/src/main/java/com/icesoft/faces/component/selectinputdate/SelectInputDateRenderer.java
===================================================================
— components/src/main/java/com/icesoft/faces/component/selectinputdate/SelectInputDateRenderer.java (revision 21788)
+++ components/src/main/java/com/icesoft/faces/component/selectinputdate/SelectInputDateRenderer.java (working copy)
@@ -1030,7 +1030,7 @@
table.appendChild(tr1);
}

• writeCell(domContext, facesContext, writer, inputComponent, " ",
  + writeCell(domContext, facesContext, writer, inputComponent, " ",
  null, inputComponent.getDayCellClass(), tr1, null,
  (weekStartsAtDayIndex + i) % 7,
  timeKeeper, months, weekdaysLong, converter);
  @@ -1136,7 +1136,7 @@
  if ((columnIndexCounter != 0) && (tr2 != null)) {
  for (int i = columnIndexCounter; i < weekdays.length; i++) {
  writeCell(domContext, facesContext, writer,
• inputComponent, " ", null,
  + inputComponent, " ", null,
  inputComponent.getDayCellClass(), tr2, null,
  (weekStartsAtDayIndex + i) % 7,
  timeKeeper, months, weekdaysLong, converter);

I've adjusted the DOMUtils class where we were handling the logic to translate coded entities that are what are generally expected for XHTML (e.g.  ) to the friendly HTML versions of the entities:

Index: src/main/java/org/icefaces/util/DOMUtils.java
===================================================================
— src/main/java/org/icefaces/util/DOMUtils.java (revision 21794)
+++ src/main/java/org/icefaces/util/DOMUtils.java (revision 21795)
@@ -546,11 +546,11 @@
} else if (ch == '\'')

else if (ch == '"')

else if (ch >= 0xA0 && ch <= 0xff)

else if (ch == 0x20AC)

else

We do this because now that we adjust to the proper content type, WebKit browsers can be very strict about the XHTML and most HTML entities are not supported. A summary of the issue can be found here:

http://techtrouts.com/webkit-entity-nbsp-not-defined-convert-html-entities-to-xml/

Where possible, I've adjusted the demo pages and the code to be more XHTML compliant so that WebKit browsers will play nicely with ICEfaces applications. The fallback for those applications can't be adjusted to provide stricter XHTML content is to mark the contentType as text/html as noted in a previous comment.

Re-opening the case.

To fix ICE-5891, it was necessary to back out some changes that originally helped with XHTML compliance. They related to the case of tag names (e.g. DIV vs div) in some of the bridge code. With the changes applied, some components stopped working properly in component showcase. There was no in-depth analysis to find out why changing the case to lowercase affected only a few of these components (Menu, Split Pane Panel). At this point, Component Showcase is currently marked to be output as "text/html" and the changes have been reverted.

Attaching the patch of the original changes that were made to the bridge code to adjust the case of various tags. The changes the scriptaculous code may have been unnecessary as the library itself may have been accomodating the case properly depending on how it was used.

Assigning to Ken for re-assignment.

As an example, one of the component related changes that I didn't make but alluded to in the notes was to the SelectInputDateRenderer. Similar to the adjustments we made for the DOMUtils for entities:

writeCell(domContext, facesContext, writer, inputComponent, " ",

becomes

writeCell(domContext, facesContext, writer, inputComponent, " ",

Revision: 23190

Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/ext/renderkit/TableRenderer.java
Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/outputprogress/OutputProgress.java
Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/outputprogress/OutputProgressRenderer.java
Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/paneldivider/PanelDividerRenderer.java
Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/selectinputdate/SelectInputDateRenderer.java

Re-open to also include all the sample apps.

Revision: 23202

Modified : /icefaces2/trunk/icefaces/compat/component-metadata/src/main/java/com/icesoft/metadata/test/DefaultValueTest.java
Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/paneltabset/PanelTabSetRenderer.java
Modified : /icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/context/effects/JavascriptContext.java
Modified : /icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/jasper/Constants.java
Modified : /icefaces2/trunk/icefaces/compat/samples/auctionMonitor/src/com/icesoft/applications/faces/auctionMonitor/beans/UserBean.java
Modified : /icefaces2/trunk/icefaces/compat/samples/auctionMonitor/web/WEB-INF/faces-config.xml
Modified : /icefaces2/trunk/icefaces/compat/samples/auctionMonitor/web/auctionMonitor.html
Modified : /icefaces2/trunk/icefaces/compat/samples/compat-basic/web/icefaces-subtree.xhtml
Modified : /icefaces2/trunk/icefaces/compat/samples/compat-basic/web/icefaces.xhtml
Modified : /icefaces2/trunk/icefaces/compat/samples/compat-basic/web/nonicefaces.xhtml