Fluid Infusion

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

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.5
  • Fix Version/s: None
  • Component/s: Framework
  • Labels:
    None
  • Environment:
    IE8/Win7
  • Similar Issues:
    Show 10 results 

Issue Links

depends on

Bug - A problem which impairs or prevents the functions of the product. FLUID-4781 The rich inline edit demo fails in IE8

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.
is depended on by

Bug - A problem which impairs or prevents the functions of the product. FLUID-4781 The rich inline edit demo fails in IE8

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Improvement - An improvement or enhancement to an existing feature or task. VP-254 Video player IE8 support

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
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
Permalink
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 Cheetham added a comment - deadMansBlur() uses Date.now(), which is not supported in IE8. One suggestion is to use     new Date().value() instead.
Hide
Permalink
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
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
Permalink
Michelle D'Souza added a comment -
Merged pull request https://github.com/fluid-project/infusion/pull/250 into project repo at 725fd98d6cc3ebbe92f378b8c05738b178dc6516
Show
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:
    James Yoon
Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: