Check device detection and HTML editor use

Bug #1265253 reported by Kristina Hoeppner
This bug report is a duplicate of:  Bug #1539322: Remove mobile device detection. Edit Remove
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Mahara
Confirmed
Medium
Unassigned

Bug Description

Mahara 1.8 / master (1.9dev):

When device detection is turned on, mobile devices do not display TinyMCE. When you turn it off, they do and some mobile devices can actually use TinyMCE. It would be good to check the current iOS and Android version to see if we still need to make the distinction when device detection is turned on or whether both states can use TinyMCE.

I think our current lowest Android version is 2.3 which we might have to rethink as Android is now at 4.4.

Tags: manualupdate
Changed in mahara:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Stéphane (smlavoie) wrote :

Hello Kristina,

Here are results of different tests made on a Mahara 1.8.1 with device detection OFF :

- Surface RT (1st gen), Windows 8.1, Internet Explorer 11.0.9600.16476 : TinyMCE present and functional.
- Nexus 7 (2012), Android 4.4.0 (build KRT16S), Google Chrome 31.0.1650.59 (JS V8 3.21.18.8) : TinyMCE present and functional.
- iPod Touch (3rd gen), iOS 6.1.5 (10B400), Safari : TinyMCE present and functional.
- Samsung Galaxy SIII, Android 4.3, Google Chrome, Firefox Beta 27 et Opera 18 : TinyMCE present and functional.

SL

Revision history for this message
Aaron Wells (u-aaronw) wrote :

For Android at least, the important thing isn't the operating system version, but the browser type and version. For instance, on my Android phone I run Android 4.1, which includes the standard Android browser that can run TinyMCE. However most of the time I browse using Opera Mini, which most decidedly cannot run TinyMCE.

Possibly we could use MobileDetect's mobileGrade() method to determine whether or not the browser is good enough to support TinyMCE. This function is supposed to be a pass-through to JQUery Mobile's grading system, which returns:

A: Full JQuery Mobile support
B: Everything except AJAX
C: Minimal support

I don't think TinyMCE is written using JQuery, but presumably if the browser is graded B or A it can probably handle TinyMCE as well.

Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

Hello Stéphane,

Thank you for testing. I'll at least update the user manual to mention that the table refers to the standard browsers.

Cheers
Kristina

tags: added: usermanualupdate
Changed in mahara-manual:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Jono M (jonom) wrote :

MobileDetect's mobileGrade() method won't work for TinyMCE - it's too lenient. The key to TinyMCE support is contentEditable, which jQuery Mobile doesn't require. For example, contentEditable was only added to iOS in version 5, but iOS 3.1+ gets an 'A' grade in mobileGrade().
I would suggest testing in JavaScript using something like Modernizr's contentEditable test (https://github.com/Modernizr/Modernizr/blob/master/feature-detects/contenteditable.js) as IMO it would be more accurate and less complex than trying to detect it in PHP

Revision history for this message
Jono M (jonom) wrote :

Alternatively, if we need to detect it on the server-side, http://caniuse.com/#feat=contenteditable has a decent summary of which browsers have support

tags: added: manualupdate
removed: usermanualupdate
no longer affects: mahara-manual
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.