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

        Hide
        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
        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
        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
        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
        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: