Issue Details (XML | Word | Printable)

Key: MDL-8417
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Critical Critical
Assignee: Nicolas Connault
Reporter: Urs Hunkler
Votes: 0
Watchers: 1
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

Extend check_browser_version() to include other A-Grade YUI browsers

Created: 04/Feb/07 02:11 AM   Updated: 01/Mar/07 04:52 PM
Return to search
Component/s: AJAX
Affects Version/s: 1.7.1, 1.8
Fix Version/s: 1.7.2, 1.8

File Attachments: 1. Text File 01-03-07_8417.patch (16 kB)


Participants: Martin Dougiamas, Nicolas Connault, Urs Hunkler and Vy-Shane Sin Fat
Security Level: None
Resolved date: 01/Mar/07
Affected Branches: MOODLE_17_STABLE, MOODLE_18_STABLE
Fixed Branches: MOODLE_17_STABLE, MOODLE_18_STABLE


 Description  « Hide
Why does the function check_browser_version() exclude Opera?

I do not see any reason. Opera 9 is an A-Grad Browser and fully supported by YUI. Please change the Opera Opera 9 exclusion for 1.7 and 1.8. Thank you very much.

Please see http://developer.yahoo.com/yui/articles/gbs/gbs_browser-chart.html

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Urs Hunkler added a comment - 04/Feb/07 08:53 PM
And Safari 2 is an A-Grad Browser and is rejcted when I read the function right. Safari 2 should be allowed too.

Martin Dougiamas added a comment - 05/Feb/07 09:05 AM
We're being conservative and adding platforms as they start working 100% (much harder to remove them than add them!)

However, Vy, can you test Opera 9 and Safari 2? Do they work?


Urs Hunkler added a comment - 05/Feb/07 03:33 PM
Martin, being conservative for the Moodle part is the best way to go.

But developersd/designers can and do use YUI for added interface enhancements or flexibility. These efforts should be supported because they add the opportunity that Moodle will benefit from YUI much faster. If these projects work with all A-Grade Browsers they should be possible in Moodle too.

A way around this dilemma - as I see it in the moment - would be to rework the loading function in a way that:

1) YUI is used for Moodle core functionality like Drag&Drop with Moodle tested Browser support
2) YUI can transparently be used from developers/designers without any Moodle internal filtering for all A-Grade Browsers

One solution may be that the Moodle Ajax option works only for the Moodle Drag&Drop and other internal Ajax.

What do you think?


Vy-Shane Sin Fat added a comment - 05/Feb/07 04:20 PM
There are still issues with the AJAX course editing for Safari 2 (MDL-6381) and Opera 9 (MDL-7598).

Martin Dougiamas added a comment - 05/Feb/07 04:41 PM
I'm confused ... we aren't stopping developers using AJAX at all!

If they choose to implement it then they can (we'll fix the detection to address a wider range of browsers).

However if the user or admin decides to switch ajax off then moodle must still function (for accessibility).


Martin Dougiamas added a comment - 06/Feb/07 11:55 AM
I'm working on adding checks for all grade-A browsers to ajaxenabled, but also adding support for a parameter which will be an array of browsers supported.

This allows the developer to restrict ajax enablement to only those browsers they know works, just for that specific page.


Martin Dougiamas added a comment - 28/Feb/07 02:40 PM
Nick's finishing this for me.

Nicolas Connault added a comment - 01/Mar/07 09:00 AM
Enabled YUI on Opera 9.0 and Safari 2.0. Tested Opera (very slow!), needs Safari testing.

Nicolas Connault added a comment - 01/Mar/07 09:38 AM
See Version Control for comments. This task is complete, but YUI support for Safari and Opera isn't very usable. Try drag&drop for an example.

Martin Dougiamas added a comment - 01/Mar/07 09:59 AM
Yes, that's why we need the second part of this bug to disable those specific browsers again on the course pages.

Nicolas Connault added a comment - 01/Mar/07 10:18 AM
Second part is complete. See lib/simpletest/testajaxlib.php for an example

Martin Dougiamas added a comment - 01/Mar/07 10:44 AM
OK, now it's complete , the final part was to add those new bits to the course formats.

Urs Hunkler added a comment - 01/Mar/07 04:52 PM
I checked a feature using YUI in Opera 9 and it works.

Great and thanks a lot Nicolas and Martin