Remove IE6 related code

Bug #1308305 reported by Yuliya Bozhko on 2014-04-16
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Low
Yuliya Bozhko

Bug Description

A lot of it is in javascript and should probably go.

Code like: if (isIE6) {...}

Not sure about IE7. Maybe it's worth to get rid of that too.

Yuliya Bozhko (yuliya.bozhko) wrote :

To test you need to make sure that everything still works (I guess) :) Especially if it is possible to check iPad/iPhone as I was not quite sure about some code that I removed.

Changed in mahara:
status: Triaged → In Progress
importance: Undecided → Low
Aaron Wells (u-aaronw) wrote :

If it would make your life easier to get rid of IE7 compatibility code, go for it.

According to the README, Mahara 1.9 and 1.10 officially support IE 9, 10, and 11. We've also been trying to not break compatibility with IE8 where possible.

Reviewed: https://reviews.mahara.org/3267
Committed: http://gitorious.org/mahara/mahara/commit/920d299786be753963aaafd6337ed3cf49970321
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 920d299786be753963aaafd6337ed3cf49970321
Author: Yuliya Bozhko <email address hidden>
Date: Wed Apr 16 14:41:11 2014 +1200

Removed IE6 and IE7 related code (Bug #1308305)

Change-Id: I1e81f836797e482427c03a005c35d47d1eae8c27
Signed-off-by: Yuliya Bozhko <email address hidden>

Changed in mahara:
status: In Progress → Fix Committed
milestone: none → 1.10.0
Aaron Wells (u-aaronw) on 2014-10-21
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers