Menu

#90 IE7 popup position error

open
nobody
None
5
2007-03-13
2007-03-13
No

In Internet Explorer 7 using Standards Compliance Mode (using a proper doctype), the popup calendar displays in the wrong vertical position. The longer the page is vertically, the more "off" the popup will be (it will be significantly higher than it should be). This patch tells the calendar to use the standard code for positioning if IE is at least version 7.

Discussion

  • Anonymous

    Anonymous - 2007-03-13

    Patch (unified) for calendar.js to allow IE >=7 to use standard positioning code

     
  • Nobody/Anonymous

    Logged In: NO

    Hi,

    Could someone tell me how to use this patch?
    Thank you in advance.

     
  • Anonymous

    Anonymous - 2007-12-11

    Logged In: YES
    user_id=1601063
    Originator: YES

    File Added: calendar.js

     
  • Anonymous

    Anonymous - 2007-12-11

    Patched calendar.js

     
  • Nobody/Anonymous

    Thank you very much! it has been very useful. This bug in IE was driving me crazy.

    By the way, for the other comment asking how to use it, if you open the files will see there are two files to download:

    calendar.js: you can replace the original calendar.js by this one if you have not changed anything in it.
    calendar.js.patch: if you open this file with a notepad, for example, you will see a line with two @@. This line points out the lines where the change must be done. After this there are lines with a - or a + at the beginning. The - at the beginning means that line must be deleted and the + means that line must be added.

    I hope it helps and thanks pjcollier again!

     
  • Nobody/Anonymous

    Thanks so much for this patch!

     
  • vesper8

    vesper8 - 2009-11-11

    thanks for this! You are my hero! Works wonderfully!

     
  • Nobody/Anonymous

    Thank you so much, you are great!!
    Spent almost 2 days to localize the problem and find your solution...

    Laurence

     
  • Sallala

    Sallala - 2010-11-07

    Please correct the following line in the patch!

    WRONG LINE:
    Calendar.is_ie7up = ( Calendar.is_ie && parseFloat(navigator.userAgent.replace(/.*msie ([0-9]+).*/i, "$1" )) >= 7 );

    CORRECTED LINE:
    Calendar.is_ie7up = ( Calendar.is_ie && parseFloat(navigator.userAgent.replace(/.*?msie ([0-9]+).*/i, "$1" )) >= 7 );
    (there is a question mark before msie)

    This is neccesarry! For example:
    This is my useragent string, that contains multiple "MSIE" parts:

    Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.2; WOW64; Trident/4.0; Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1) ; .NET4.0C; .NET4.0E; .NET CLR 2.0.50727; .NET CLR 3.0.04506.648; .NET CLR 3.5.21022; .NET CLR 3.0.4506.2152; .NET CLR 3.5.30729)

     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.