[jira] [Resolved] (TOMAHAWK-1557) InputDate calendar popup does not render correctly on IE7

["Leonardo Uribe (JIRA)" <de...@myfaces.apache.org>]

     [ https://issues.apache.org/jira/browse/TOMAHAWK-1557?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Leonardo Uribe resolved TOMAHAWK-1557.
--------------------------------------

       Resolution: Fixed
    Fix Version/s: 1.1.11-SNAPSHOT
         Assignee: Leonardo Uribe

I think in this part it is better to replace the code with Element.setStyle() method of prototype. This one do what the proposal says internally, and has another fix of opacity property. 

> InputDate calendar popup does not render correctly on IE7
> ---------------------------------------------------------
>
>                 Key: TOMAHAWK-1557
>                 URL: https://issues.apache.org/jira/browse/TOMAHAWK-1557
>             Project: MyFaces Tomahawk
>          Issue Type: Bug
>          Components: Calendar
>         Environment: Using Tomahawk 1.2  Jar file  tomahawk12-1.1.9.jar
> Browser is IE7 on Windows XP
>            Reporter: Frank Davis
>            Assignee: Leonardo Uribe
>             Fix For: 1.1.11-SNAPSHOT
>
>
> Popup calendar does not render correctly on IE7 but renders OK of FF.  In IE7 the popup takes the full width of the browser screen.
> Following tag used to generate the popup
> 		<t:inputDate id="#{id}"  value="#{value}" renderAsPopup="#{true}" popupButtonImageUrl="images/icon_date.gif" 
> 				popupDateFormat="dd/MM/yyyy" renderPopupButtonAsImage="true" 
> 				popupCalendar="#{true}" converterMessage="#{id} is in wrong format"/>
> Issue appears to be a "feature" of IE7 where the setAttribute method in javascript does not work correctly.  In the javascript used to generate the popup the width is set in the javascript org_apache_myfaces_PopupCalendar.prototype.init  as follows
> mainTable.setAttribute("cssText", "width:" + ((this.initData.showWeekNumber == 1)?250:220) + "px;"); 
> This does not appear to set the width correctly.  The recommended workaround  (which works for both IE7 and FF) is to set the cssText directly as follows: 
> 	    mainTable.style.cssText = "width:" + ((this.initData.showWeekNumber == 1)?250:220) + "px;";
> I have tested this on both IE7 and FF and it seems to work OK.
> There are a number of other places in the javascript where the setAttribute is used to set the width and other style attributes - I assume these have the same problem - I have changed these and the popup appears to work OK

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira