[MDL-42476] Completion report should use feature detection rather than browser sniffing - Moodle Tracker

XML

Word

Printable

Testing Instructions:
Hide

In as many browsers as possible (at least IE>9, IE8, firefox, chrome, mobilesafari)

Enable completion tracking in advanced features

In a course, enable completion tracking in the course settings

Go to "completion tracking" and enable self completion and completion by date

Go to course -> reports > course completion

Enrol a user in the course

VERIFY: no JS errors occur

VERIFY: the text is rotated if the browser supports SVG (IE9+ and all the rest)

VERIFY: the text is not rotated if the browser doesn't support SVG (IE8).
Show
In as many browsers as possible (at least IE>9, IE8, firefox, chrome, mobilesafari) Enable completion tracking in advanced features In a course, enable completion tracking in the course settings Go to "completion tracking" and enable self completion and completion by date Go to course -> reports > course completion Enrol a user in the course VERIFY: no JS errors occur VERIFY: the text is rotated if the browser supports SVG (IE9+ and all the rest) VERIFY: the text is not rotated if the browser doesn't support SVG (IE8).
Affected Branches:
MOODLE_24_STABLE, MOODLE_25_STABLE
Fixed Branches:
MOODLE_24_STABLE, MOODLE_25_STABLE
Pull from Repository:
https://github.com/srynot4sale/moodle.git
Pull Master Branch:
~~MDL-42476~~
Pull Master Diff URL:
https://github.com/srynot4sale/moodle/commit/d2cd4acab13928347181b9e5d7c19103283b5165

Same issue as ~~MDL-39964~~ but with completion report.

Browser sniffing is problematic and should be avoiding. Since this feature uses JS to do its work, we should be able to use feature detection.

Participants:: Aaron Barnes, Damyon Wiese, Dan Poltawski, Jason Fowler, Michael de Raadt, Rossiani Wijaya

Component watchers:: Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona), Andrew Nicols, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze