Marc358 / gwt-cal

Automatically exported from code.google.com/p/gwt-cal
0 stars 0 forks source link

Not working in IE6 #92

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
The calendar doesn't work in Internet Explorer 6 (a lot of people still run 
this browser unfortunately, see 
http://marketshare.hitslink.com/browser-market-share.aspx?qprid=2).

To reproduce this problem run the demo on this site 
(http://google.latest.gwt-web-calendar.appspot.com/) in IE6 and click between 
day view and month view. Then the calendar isn't rendered properly.

This is a critical bug!

/Tobias

Original issue reported on code.google.com by lovestad...@gmail.com on 13 Oct 2010 at 6:34

GoogleCodeExporter commented 8 years ago
I have modified the setView method of the CalendarWidget class to include some 
code for IE 6 only.

I've moved the "this.setStyleName(this.view.getStyleName());" line before 
"this.view.attach(this);" to apply the style before attaching the new view to 
calendar widget.

Now, the calendar works in IE 6.

  public final void setView(CalendarView view) {
        this.getRootPanel().clear();
        this.view = view;

        String appVersion = com.google.gwt.user.client.Window.Navigator.getAppVersion().toLowerCase();
        if (appVersion.indexOf("msie 6.") > 0) {
            this.getRootPanel().setVisible(false);
            this.setStyleName(this.view.getStyleName());
            this.getRootPanel().setVisible(true);
        } else {
            this.setStyleName(this.view.getStyleName());
        }

        this.view.attach(this);
        this.refresh();
    }

Original comment by alin.par...@gmail.com on 10 Feb 2011 at 1:50

Attachments:

GoogleCodeExporter commented 8 years ago
As seen in http://www.w3schools.com/browsers/browsers_explorer.asp the IE6 is 
almost not used nowadays.

To apply this fix I need to do it blindly since I don't have access to any real 
IE6 (IE9 has some emulation mode), so if no one complains I vote to simply 
close it.

Original comment by ctasada on 3 Jan 2012 at 4:19

GoogleCodeExporter commented 8 years ago
Since IE6 is no longer a relevant browser.

Original comment by ctasada on 5 Jan 2012 at 9:28