Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-34875

TinyMCE improvements in 2.4 - META

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.4
    • Component/s: HTML Editor (TinyMCE)
    • Labels:
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE

      Description

      List of TinyMCE related issues to be resolved in 2.4, the aim is to allow easy creation/distribution of custom plugins for TinyMCE in Moodle and to allow more customisation.

      This may be also partially related to filters which might be improved once we integrate new MUC caching infrastructure.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            poltawski Dan Poltawski added a comment -
            Show
            poltawski Dan Poltawski added a comment - A related 'meta' to this: http://tracker.moodle.org/browse/MDL-19211
            Hide
            skodak Petr Skoda added a comment -

            Oh, I wish we could use nested subtasks...

            Show
            skodak Petr Skoda added a comment - Oh, I wish we could use nested subtasks...
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            For self-organization, this is the ordered list of chained issues in the order I'm looking to them, grrr.

            MDL-34954 - MDL-34876 - MDL-34877 - MDL-34878 - MDL-34955 - MDL-34990 - MDL-34956 - MDL-34955 - MDL-34879

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - For self-organization, this is the ordered list of chained issues in the order I'm looking to them, grrr. MDL-34954 - MDL-34876 - MDL-34877 - MDL-34878 - MDL-34955 - MDL-34990 - MDL-34956 - MDL-34955 - MDL-34879
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Chain above halted @ MDL-34877 (lack of proper testing instructions), sorry!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Chain above halted @ MDL-34877 (lack of proper testing instructions), sorry!
            Hide
            skodak Petr Skoda added a comment - - edited

            The integration order should be the same as the order of subtasks above, it also has "blocked" links and the diff links clearly indicate the order of commits too. What else would you want Eloy, one huge issue instead?

            Show
            skodak Petr Skoda added a comment - - edited The integration order should be the same as the order of subtasks above, it also has "blocked" links and the diff links clearly indicate the order of commits too. What else would you want Eloy, one huge issue instead?
            Hide
            skodak Petr Skoda added a comment -

            here is the complete diff for all issues that should be imo integrated this week: https://github.com/skodak/moodle/compare/master...w34_MDL-34990_m24_tinymcebuttonsfix

            Show
            skodak Petr Skoda added a comment - here is the complete diff for all issues that should be imo integrated this week: https://github.com/skodak/moodle/compare/master...w34_MDL-34990_m24_tinymcebuttonsfix
            Hide
            nebgor Aparup Banerjee added a comment -

            hm, just wondering about a tinymce editor example instance within admin/settings.php?section=editorsettingstinymce ?

            it might make configurations slightly friendlier.

            Show
            nebgor Aparup Banerjee added a comment - hm, just wondering about a tinymce editor example instance within admin/settings.php?section=editorsettingstinymce ? it might make configurations slightly friendlier.
            Hide
            nebgor Aparup Banerjee added a comment -

            just adding notes here from review of subtask numbers 4 - 8:

            • (14:43:16) skodak: oh! one thing we discussed yesterday with David - there should be something like "subpluginparent = "editor_tinymce";" in version.php of each subplugin.
            • also i've seen that uninstalling of core tinymce plugins seem to break what was the concept of 'part of core' , should those ones be simply 'disable' instead of uninstall as they come part of core?
            Show
            nebgor Aparup Banerjee added a comment - just adding notes here from review of subtask numbers 4 - 8: (14:43:16) skodak: oh! one thing we discussed yesterday with David - there should be something like "subpluginparent = "editor_tinymce";" in version.php of each subplugin. also i've seen that uninstalling of core tinymce plugins seem to break what was the concept of 'part of core' , should those ones be simply 'disable' instead of uninstall as they come part of core?
            Hide
            skodak Petr Skoda added a comment -

            plugins !== core, it is fine to delete some official plugin from your install

            Show
            skodak Petr Skoda added a comment - plugins !== core, it is fine to delete some official plugin from your install
            Hide
            skodak Petr Skoda added a comment -

            Everything that was planned is now implemented, thanks everybody for feedback, support and testing.

            Show
            skodak Petr Skoda added a comment - Everything that was planned is now implemented, thanks everybody for feedback, support and testing.
            Hide
            jmvedrine Jean-Michel Vedrine added a comment -

            Thank a lot Petr for working on this !

            Show
            jmvedrine Jean-Michel Vedrine added a comment - Thank a lot Petr for working on this !
            Hide
            skodak Petr Skoda added a comment -

            If you need any more tweaks or hooks for custom plugins please create a new issue for it or comment here, thanks!

            Show
            skodak Petr Skoda added a comment - If you need any more tweaks or hooks for custom plugins please create a new issue for it or comment here, thanks!

              People

              • Votes:
                1 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  3/Dec/12