Issue Details (XML | Word | Printable)

Key: MDL-19211
Type: Task Task
Status: Open Open
Priority: Major Major
Assignee: Petr Skoda
Reporter: Petr Skoda
Votes: 0
Watchers: 2
Operations

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

Text editors cleanup and improvements META

Created: 17/May/09 01:34 AM   Updated: 19/Jun/09 09:21 PM
Return to search
Component/s: HTML Editor
Affects Version/s: 2.0
Fix Version/s: 2.0

Issue Links:
Relates
 

Participants: Marc Grober, Mauno Korpelainen and Petr Skoda
Security Level: None
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE

Sub-Tasks  All   Open   
 Sub-Task Progress: 

 Description  « Hide
* upgrade tinymce
* make editors pluggable again
* improve lang strings handling
* upgrade DragMath
* etc.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Marc Grober added a comment - 16/Jun/09 10:52 AM
As the person nominally identified as the Moodle maintainer for DragMath, might I inquire as to:
  • how changing what is in Moodle HEAD to 0.7.7, which was out of date when the changes were made, constitutes "upgrading"
  • why DragMath has now been placed, contrary to the prior discussion, not in /lib/editor/common nor in /lib/common, but in /lib/dragmath
  • DragMath having been placed in /lib/dragmath, what is the intended abode of asciimathml and shouldn;t the locations demonstrate consistency, as per the MDLs that address dragmath and asciimath
  • Since you have been fiddling HEAD vis-a-vis DragMath, have you also now changed adminlib.php as previously suggested (forum postings seem to suggest that you did this but I don't see the line in adminlib.php)
  • who will close the half dozen MDL's regarding dragmath that this MDL has rendered moot.... it would seem that at least things should be linked and tagged as closed since those tracking the existing MDL's would have no clue that things took a hard right.....

Petr Skoda added a comment - 16/Jun/09 04:07 PM
1/ there can not be "common" in editor directory because we have editor plugins there, whoever invented it forgot how our plugins work
2/ lib is the right place for imported libraries, it helps with maintenance
3/ I did not want to be maintainer of editor integrations but unfortunately nobody was working on editor integrations
4/ my current priority is to finalize editor embedding and only then start dealing with editor extensions, sorry if I broke anything
5/ each imported library MUST have readme_moodle.txt which includes list of all changes, import steps but also the maintainer!
6/ I do not understand what is your problem with 0.7.7, I took it from http://www.dragmath.bham.ac.uk/ which was at the time the latest version, the previous version was much older

Marc Grober added a comment - 18/Jun/09 02:30 AM
In fact I initially argued for dragmath and asciimath to be located in /lib/common but was overruled by Matthieu... There were also some editor recommendations and demonstrations by Mauno (pretty exciting stuff) that were set aside at the time (gee, almost a year ago now??).... i.e. there were people working on editor integrations.....

if dragmath is in lib/dragmath, then can we expect that asciimath will be in lib/asciimath?

Dragmath 0.7.7 was broken (a new fix in the code to address implied multiplication apparently broke another function), Alex released 0.7.7.1 to resolve the issue and I created a fresh zip for moodlers. 0.7.8 was released to address java console errors MK was seeing as he developed SEE, his integration of dragmath, asciimath, and svg with tinyMCE. Have you reviewed his work?


Mauno Korpelainen added a comment - 19/Jun/09 09:21 PM
Hi Marc and Petr,

you have written so many new tracker issues during my 2 holiday weeks that it is hard to follow all of them - anyway some comments in http://tracker.moodle.org/browse/MDL-19516 ...

I'm not worried about different plugin structure of moodle 2.0 and previous versions - or different integration methods, different editors etc...

Marc,

I'm really pleased to see Petr taking care of editor code and plugins - I suppose he was not aware of your role and we have had so many different test integrations (trials) during the last few years (starting from Janne's work, my versions, Glen's version, OU version, Mathieu's moodle 2.0 version etc) that most likely nobody has tested all of them - not to mention math plugins etc.

I am happy with any system that is stable and flexible enough for creating and using great features of moodle - both old and new ones... and I can change my files according to each version if needed...