Ticket #2100 (closed bug: fixed)

Opened 4 years ago

Last modified 4 years ago

Layer.TMS tests fail in IE8

Reported by: tschaub Owned by:
Priority: minor Milestone: 2.8 Release
Component: Layer.TMS Version: 2.8 RC2
Keywords: Cc:
State: Complete

Description


Attachments

2100.patch Download (2.6 KB) - added by tschaub 4 years ago.
indexOf is not a cross-browser array method

Change History

Changed 4 years ago by tschaub

indexOf is not a cross-browser array method

Changed 4 years ago by tschaub

  • state set to Review

This broke with r8932. Tests now pass (and I suspect this fixes the TMS layer in IE).

Thanks for any review.

Changed 4 years ago by tschaub

  • component changed from tests to Layer.TMS

Changed 4 years ago by crschmidt

Good catch. Please commit.

note that the tms layer would have worked before -- it just wouldn't have worked with the serverResolution option, so this fixes that. (I did run through the examples and confirrm that Layer.TMS worked in at least some cases.)

Changed 4 years ago by crschmidt

  • state changed from Review to Commit

Changed 4 years ago by tschaub

  • keywords pullup added
  • state changed from Commit to Pullup

(In [9403]) Using util method for array indexOf in TMS layer. r=crschmidt (pullup #2100)

Changed 4 years ago by crschmidt

  • keywords pullup removed
  • status changed from new to closed
  • state changed from Pullup to Complete
  • resolution set to fixed

(In [9406]) Pullups for OL 2.8 RC3.

jQuery lib fix (Closes #1391) getRenderedSize regression (Closes #1906) element.scrolls error with panzoombar (Closes #2054) createUrlObject bug (Closes #2060) google layer in late rendered maps (Closes #2075) IE6/Lang.nb bug (Closes #2093) Layer.TMS/TileCache bugs (Closes #2099) (Closes #2100) Graphic names issues (Closes #2101)

Note: See TracTickets for help on using tickets.