Uploaded image for project: 'Fluid Infusion'
  1. Fluid Infusion
  2. FLUID-4762

Fluid view unit test failing (Dead man's blur test - non-excluded component one, failing test #2)

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: None
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      IE8/Win7

      Issue Links

        Activity

        Hide
        cli@ocad.ca Cindy Qi Li added a comment - - edited

        With IE8, some dead man's blur unit tests, including the one specified in the title, throw js error: Object doesn't support this property or method.

        To produce,
        1. In IE8, go to tools -> Internet Options -> Advanced tab -> Browsing section -> Turn on "Display a notification about every script error"
        2. Run http://build.fluidproject.org/infusion/tests/framework-tests/core/html/FluidView-test.html

        Show
        cli@ocad.ca Cindy Qi Li added a comment - - edited With IE8, some dead man's blur unit tests, including the one specified in the title, throw js error: Object doesn't support this property or method. To produce, 1. In IE8, go to tools -> Internet Options -> Advanced tab -> Browsing section -> Turn on "Display a notification about every script error" 2. Run http://build.fluidproject.org/infusion/tests/framework-tests/core/html/FluidView-test.html
        Hide
        anastasia Anastasia Cheetham added a comment -

        deadMansBlur() uses Date.now(), which is not supported in IE8. One suggestion is to use
        new Date().value()
        instead.

        Show
        anastasia Anastasia Cheetham added a comment - deadMansBlur() uses Date.now(), which is not supported in IE8. One suggestion is to use new Date().value() instead.
        Hide
        jobara Justin Obara added a comment -

        Added a pull request ( https://github.com/fluid-project/infusion/pull/250 ) to fix this. The change is to implement a fall back to use Date().gettime() instead of Date.now() when the latter doesn't exist.

        Show
        jobara Justin Obara added a comment - Added a pull request ( https://github.com/fluid-project/infusion/pull/250 ) to fix this. The change is to implement a fall back to use Date().gettime() instead of Date.now() when the latter doesn't exist.
        Hide
        michelle.dsouza@utoronto.ca Michelle D'Souza added a comment -

        Merged pull request https://github.com/fluid-project/infusion/pull/250 into project repo at 725fd98d6cc3ebbe92f378b8c05738b178dc6516

        Show
        michelle.dsouza@utoronto.ca Michelle D'Souza added a comment - Merged pull request https://github.com/fluid-project/infusion/pull/250 into project repo at 725fd98d6cc3ebbe92f378b8c05738b178dc6516

          People

          • Assignee:
            Unassigned
            Reporter:
            jameswy James Yoon
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: