Closed
Bug 801318
Opened 12 years ago
Closed 12 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)
SeaMonkey
Preferences
Tracking
(seamonkey2.14 fixed, seamonkey2.15 fixed)
RESOLVED
FIXED
seamonkey2.16
People
(Reporter: philip.chee, Assigned: philip.chee)
References
Details
Attachments
(2 files)
3.20 KB,
patch
|
iannbugzilla
:
review+
neil
:
approval-comm-aurora-
neil
:
approval-comm-beta-
|
Details | Diff | Splinter Review |
3.22 KB,
patch
|
philip.chee
:
review+
Callek
:
approval-comm-aurora+
Callek
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
> 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.
![]() |
Assignee | |
Comment 1•12 years ago
|
||
> -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+
![]() |
Assignee | |
Comment 3•12 years ago
|
||
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.
![]() |
Assignee | |
Comment 4•12 years ago
|
||
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?
![]() |
Assignee | |
Comment 5•12 years ago
|
||
> Regression caused by (bug #): Bug 799502.
Actually Bug 588909
Comment 6•12 years ago
|
||
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-
![]() |
Assignee | |
Comment 7•12 years ago
|
||
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?
Updated•12 years ago
|
Attachment #672799 -
Flags: approval-comm-beta?
Attachment #672799 -
Flags: approval-comm-beta+
Attachment #672799 -
Flags: approval-comm-aurora?
Attachment #672799 -
Flags: approval-comm-aurora+
Comment 8•12 years ago
|
||
status-seamonkey2.14:
--- → fixed
![]() |
Assignee | |
Comment 9•12 years ago
|
||
Pushed to comm-aurora:
http://hg.mozilla.org/releases/comm-aurora/rev/8bf74bea06a4
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
status-seamonkey2.15:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.16
You need to log in
before you can comment on or make changes to this bug.
Description
•