Closed Bug 801318 Opened 10 years ago Closed 10 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: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.16
You need to log in before you can comment on or make changes to this bug.