Closed Bug 801318 Opened 8 years ago Closed 8 years ago

Override the UA string for Moodle to get the rich text editor rather than plain textareas (Port Bug 799502).

Categories

(SeaMonkey :: Preferences, defect)

defect
Not set
normal

Tracking

(seamonkey2.14 fixed, seamonkey2.15 fixed)

RESOLVED FIXED
seamonkey2.16
Tracking Status
seamonkey2.14 --- fixed
seamonkey2.15 --- fixed

People

(Reporter: philip.chee, Assigned: philip.chee)

References

Details

Attachments

(2 files)

> Steps to reproduce:
> 
> Example site to use can be http://demo.moodle.net/user/edit.php?id=5&course=1 
> where after using demo login as student, the "description" field should be rich 
> text.
> 
> Actual results:
> 
> The textarea which should have progressively enhanced stays as a textarea.
> 
> Expected results:
> 
> It should have shown all the rich text buttons and things.

References:

Bug 799502 - Override the UA string for moodle to get the rich text editor rather than plain textareas.
  Bug 797703 - moodle serves plain textarea instead of rich text editor.
Bug 797363 - Remove aol.com user agent override since mail.aol.com is fixed.
> -pref("general.useragent.override.aol.com", "Gecko/[^ ]*#Gecko/20100101");
This is from Bug 797363 - Remove aol.com user agent override since mail.aol.com is fixed.
Attachment #671113 - Flags: review?(iann_bugzilla)
Comment on attachment 671113 [details] [diff] [review]
Patch v1.0 Moodle UA Override.

r=me though probably needs someone familiar with using Moodle to fully test it.
Attachment #671113 - Flags: review?(iann_bugzilla) → review+
From Bug 797703 - moodle serves plain textarea instead of rich text editor

1. Go to http://demo.moodle.net/login/index.php
2. Login as student/demo
3. The Description textarea should turn into a rich text editor.
Comment on attachment 671113 [details] [diff] [review]
Patch v1.0 Moodle UA Override.

Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/06cde2eba4e8

Base bug landed on mozilla-beta and mozilla-aurora. See Bug 799502 Comment 10.

[Approval Request Comment]
Regression caused by (bug #): Bug 799502.
User impact if declined: plain textarea instead of rich text editor on moodle (worse user experience).
Testing completed (on m-c, etc.): Base bug tested on m-c, this patch tested on c-c.
Risk to taking this patch (and alternatives if risky): reasonably low risk, and the User Agent override for moodle can be disabled via a pref.
String changes made by this patch: none.
Attachment #671113 - Flags: approval-comm-beta?
Attachment #671113 - Flags: approval-comm-aurora?
> Regression caused by (bug #): Bug 799502.
Actually Bug 588909
Comment on attachment 671113 [details] [diff] [review]
Patch v1.0 Moodle UA Override.

Need to make a branch-safe patch because contains() doesn't exist on beta and got turned off on aurora by bug 793781.
Attachment #671113 - Flags: approval-comm-beta?
Attachment #671113 - Flags: approval-comm-beta-
Attachment #671113 - Flags: approval-comm-aurora?
Attachment #671113 - Flags: approval-comm-aurora-
Regression caused by (bug #): Bug 588909/Bug 799502.
User impact if declined: plain textarea instead of rich text editor on moodle (worse user experience).
Testing completed (on m-c, etc.): Base bug tested on m-c, this patch tested on c-c.
Risk to taking this patch (and alternatives if risky): reasonably low risk, and the User Agent override for moodle can be disabled via a pref.
String changes made by this patch: none.
Attachment #672799 - Flags: review+
Attachment #672799 - Flags: approval-comm-beta?
Attachment #672799 - Flags: approval-comm-aurora?
Attachment #672799 - Flags: approval-comm-beta?
Attachment #672799 - Flags: approval-comm-beta+
Attachment #672799 - Flags: approval-comm-aurora?
Attachment #672799 - Flags: approval-comm-aurora+
Pushed to comm-aurora:
http://hg.mozilla.org/releases/comm-aurora/rev/8bf74bea06a4
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.16
You need to log in before you can comment on or make changes to this bug.