ICEmobile
  1. ICEmobile
  2. MOBI-286

Carousel not working in IE9

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.0 Beta
          • Fix Version/s: 1.1 Final
          • Component/s: Faces, Spring
          • Labels:
            None
          • Environment:
            n/a

            Description

            Carousel is not working in IE9 for both the JSF and JSP tags. Swiping the JSF component appears to change the bottom pager, but the images do not move.

              Activity

              Ascending order - Click to sort in descending order
              4 older comments
              Hide
              Permalink
              Judy Guglielmin added a comment -

              rev 30384 has the new iscroll-4.2 libs checked in and javascript and renderers are modified appropriately. New features include a "next" and "prev" pagination (see mobitest example test page), has "zoom" feature built in
              The pagination features will still have to be styled.
              all mobitest examples pass on Chrome, Safari, IE9 on desktop. and iPhone and iPad.

              mediacast:- problem here is that the upload somehow removes the update script tag. Have to refresh or reload the page to get it back again.

              mobileshowcase:- testing shows that removal of position:absolute; from the contentPane styling makes this work correctly. So Patrick or Phillip I"m sure can come up with something.

              iPad theme seems to be rendering differently this component on desktop browsers and iPad. If I had more time, I would implement a "width" attribute as I think this would help in the rendering (esp IE9).

              Show
              Judy Guglielmin added a comment - rev 30384 has the new iscroll-4.2 libs checked in and javascript and renderers are modified appropriately. New features include a "next" and "prev" pagination (see mobitest example test page), has "zoom" feature built in The pagination features will still have to be styled. all mobitest examples pass on Chrome, Safari, IE9 on desktop. and iPhone and iPad. mediacast:- problem here is that the upload somehow removes the update script tag. Have to refresh or reload the page to get it back again. mobileshowcase:- testing shows that removal of position:absolute; from the contentPane styling makes this work correctly. So Patrick or Phillip I"m sure can come up with something. iPad theme seems to be rendering differently this component on desktop browsers and iPad. If I had more time, I would implement a "width" attribute as I think this would help in the rendering (esp IE9).
              Hide
              Permalink
              Ted Goddard added a comment -

              Changing mobitest to "Production" breaks the carousel as observed in mediacast earlier.

              A fix for the script update issue is checked in: the script is now a child of the carousel div, causing it to be re-executed if the carousel div is replaced. Further, the iscroll scroller.parent is tested to see if iscroll is still attached to the DOM. If not, it is rebuilt. Likely it needs to be rebuilt every time it is updated, but that requires more change to the code.

              Show
              Ted Goddard added a comment - Changing mobitest to "Production" breaks the carousel as observed in mediacast earlier. A fix for the script update issue is checked in: the script is now a child of the carousel div, causing it to be re-executed if the carousel div is replaced. Further, the iscroll scroller.parent is tested to see if iscroll is still attached to the DOM. If not, it is rebuilt. Likely it needs to be rebuilt every time it is updated, but that requires more change to the code.
              Hide
              Permalink
              Ted Goddard added a comment -

              Please verify carousel using IE9 in http://labs.icesoft.com/mediacast/mediacast.jsf and close if successful.

              Show
              Ted Goddard added a comment - Please verify carousel using IE9 in http://labs.icesoft.com/mediacast/mediacast.jsf and close if successful.
              Hide
              Permalink
              Ted Goddard added a comment -

              CarouselRenderer is no longer using iscroll-min.js. This file was out of date and is not minified by our compressor, it was simply checked in. Minification is an optimization that is not strictly necessary.

              Show
              Ted Goddard added a comment - CarouselRenderer is no longer using iscroll-min.js. This file was out of date and is not minified by our compressor, it was simply checked in. Minification is an optimization that is not strictly necessary.
              Hide
              Permalink
              Ted Goddard added a comment -

              Verified working on IE9.

              Show
              Ted Goddard added a comment - Verified working on IE9.

                People

                • Assignee:
                  Patrick Corless
                  Reporter:
                  Philip Breau
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: