Page MenuHomePhabricator

ResourceLoader: Increase coverage of isCompatible() in startup.js
Closed, ResolvedPublic

Description

I'm sure I already reported this but, but I can't find it, so either I have a serious problem to remember things or the database.

./resources/startup.js should load jQuery and everything else only for compatible browsers. But currently only IE < 6 is blocked, everything else is still only a TODO comment. In the meantime jQuery has changed, so is no longer compatible with some old browsers [1].

You should block:

  • FF < 2
  • Safari < 3
  • Opera < 10 (this has changed, see also bug 31691)
  • Chrome < 1

[1] http://docs.jquery.com/Browser_Compatibility


Version: 1.17.x
Severity: enhancement

Details

Reference
bz35906

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 12:14 AM
bzimport set Reference to bz35906.

Related URL: https://gerrit.wikimedia.org/r/60254 (Gerrit Change I2fdd0da8c17553a3465ef27c46a4632e135c4405)

Related URL: https://gerrit.wikimedia.org/r/62213 (Gerrit Change I2fdd0da8c17553a3465ef27c46a4632e135c4405)

Related URL: https://gerrit.wikimedia.org/r/62605 (Gerrit Change I2fdd0da8c17553a3465ef27c46a4632e135c4405)

Included in MediaWiki 1.22.0:
(Ia54dd738b) startup: Lower blacklist for Firefox from 4 to 3
(If7743c476) startup: Blacklist incompatible mobile browsers