Remove IE6 related code

Bug #1308305 reported by Yuliya Bozhko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
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.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/3267

Revision history for this message
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
Revision history for this message
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.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

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)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.