Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-16943

Date picker broken in IE7

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: CLOSED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.5.x
    • Fix Version/s: None
    • Component/s: Account
    • Labels:
      None

      Description

      After r63219 the date picker is broken in IE7. When users click on the calendar icon the popup is never launched, this is because IE7 doesn't support adding events with the setAttribute method.

      Something on the lines of the following will fix the issue.

      Index: src/webapp/calendar/sakai-calendar.js
      ===================================================================
      — src/webapp/calendar/sakai-calendar.js (revision 66356)
      +++ src/webapp/calendar/sakai-calendar.js (working copy)
      @@ -71,8 +71,10 @@
      imageNode.id=calendar_id;
      imageNode.title='Popup date selector';
      imageNode.style.cursor="pointer";

      • imageNode.setAttribute("onClick", "popupCalendar('"inputfield_id"')");
        insertAfter(yearselectNode,imageNode);
        + document.getElementById(calendar_id).onclick = function() { + popupCalendar(inputfield_id); + }

      //Insert the hidden field node and insert it after the year select box (doesn't really matter it's hidden)
      var hiddenNode = document.createElement('input');

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  joshua.ryan@asu.edu Joshua Ryan (Inactive)
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Git Source Code