ICEmobile
  1. ICEmobile
  2. MOBI-802

position absolute causes problems in Carousel for IE10 and FF

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3 Final
    • Fix Version/s: 1.3.1
    • Component/s: Faces, JSP
    • Labels:
      None
    • Environment:
      jsf, jsp icemobile, IE, FF

      Description

      see screenshot to show how absolute positioning in class mobi-carousel-scroller affects IE10. Same thing happens with FF.
      this should be sufficient to fix for mobileshowcase, but icemobilespring has further problem after submit the carousel freezes.

        Activity

        Hide
        Philip Breau added a comment -

        Revision: 39867
        Author: philip.breau
        Date: January-31-14 8:29:13 AM
        Message:
        MOBI-802
        position absolute causes problems in Carousel for IE10 and FF
        upgraded to iScroll 5.1.1


        Modified : /icemobile/trunk/icemobile/jsf/components/component/resources/org.icefaces.component.carousel/carousel.js
        Modified : /icemobile/trunk/icemobile/resources/javascript/iscroll.js

        Show
        Philip Breau added a comment - Revision: 39867 Author: philip.breau Date: January-31-14 8:29:13 AM Message: MOBI-802 position absolute causes problems in Carousel for IE10 and FF upgraded to iScroll 5.1.1 Modified : /icemobile/trunk/icemobile/jsf/components/component/resources/org.icefaces.component.carousel/carousel.js Modified : /icemobile/trunk/icemobile/resources/javascript/iscroll.js
        Hide
        Judy Guglielmin added a comment -

        IE10 testing of iscroll demos for iscroll4 don't work, but iscroll5 beta with IE10 does work.

        Show
        Judy Guglielmin added a comment - IE10 testing of iscroll demos for iscroll4 don't work, but iscroll5 beta with IE10 does work.
        Hide
        Judy Guglielmin added a comment -

        do you want to retarget this with comments? Unless we make modifications to iscroll, I don't see a way around this (do a conditional for IE or FF that does not allow the position:absolute to be added to the dynamic style that is assigned by iscroll.js

        Show
        Judy Guglielmin added a comment - do you want to retarget this with comments? Unless we make modifications to iscroll, I don't see a way around this (do a conditional for IE or FF that does not allow the position:absolute to be added to the dynamic style that is assigned by iscroll.js
        Hide
        Judy Guglielmin added a comment -

        problem is in iscroll script. If we could get options.hasTransform to resolve to "true" this would be fine, but although I can set the userTransform to true on initialization,
        that.options.useTransform = hasTransform && that.options.useTransform;
        and since IE10 and FF are resolving options.useTransform to false, this line is adding the position:absolute....

        if (that.options.useTransform) that.scroller.style[transform] = 'translate(' + that.x + 'px,' + that.y + 'px)' + translateZ;
        else that.scroller.style.cssText += ';position:absolute;top:' + that.y + 'px;left:' + that.x + 'px';
        .....

        unless we want to modify iscroll for IE9 and FF, there isn't a simple way around this.

        Show
        Judy Guglielmin added a comment - problem is in iscroll script. If we could get options.hasTransform to resolve to "true" this would be fine, but although I can set the userTransform to true on initialization, that.options.useTransform = hasTransform && that.options.useTransform; and since IE10 and FF are resolving options.useTransform to false, this line is adding the position:absolute.... if (that.options.useTransform) that.scroller.style [transform] = 'translate(' + that.x + 'px,' + that.y + 'px)' + translateZ; else that.scroller.style.cssText += ';position:absolute;top:' + that.y + 'px;left:' + that.x + 'px'; ..... unless we want to modify iscroll for IE9 and FF, there isn't a simple way around this.
        Hide
        Philip Breau added a comment -

        We've had so many issues with iScroll that perhaps it's best to create a new jira for replacing iScroll with overthrow.js that's used with the dataView component.

        Show
        Philip Breau added a comment - We've had so many issues with iScroll that perhaps it's best to create a new jira for replacing iScroll with overthrow.js that's used with the dataView component.
        Hide
        Judy Guglielmin added a comment -

        problem is coming from iscroll which is applying that style by over-riding style of the scroller div. looking for workarounds.

        Show
        Judy Guglielmin added a comment - problem is coming from iscroll which is applying that style by over-riding style of the scroller div. looking for workarounds.
        Hide
        Judy Guglielmin added a comment -

        screenshot of IE10 carousel

        Show
        Judy Guglielmin added a comment - screenshot of IE10 carousel

          People

          • Assignee:
            Philip Breau
            Reporter:
            Judy Guglielmin
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: