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

      Attachments

        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: