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

Remove Collapsed Topics contributed plugin code (js & css) from MyMobile theme

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide
      1. Select MyMobile theme via theme selector
      2. With mobile device or simulator TEST to see that the MyMobile works OK in normal course pages (weeks/topic/ etc.,) but NOT necessarily contributed plugins as these should be fixed in the plugins themselves.
      Show
      Select MyMobile theme via theme selector With mobile device or simulator TEST to see that the MyMobile works OK in normal course pages (weeks/topic/ etc.,) but NOT necessarily contributed plugins as these should be fixed in the plugins themselves.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:

      Description

      Historically CSS & Javascript was added to MyMobile theme to fix a problem with Collapsed Topics, a contributed plugin. However with the rapid development of Moodle 2.3 and 2.4 this became increasingly difficult, and so it was agreed that the changes should be with Collapsed Topics rather than the MyMobile theme itself.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              poltawski Dan Poltawski added a comment -

              Giving this one Damyon, since hes integrating the blocking issue.

              Show
              poltawski Dan Poltawski added a comment - Giving this one Damyon, since hes integrating the blocking issue.
              Hide
              damyon Damyon Wiese added a comment -

              Waiting for feedback on the blocking issue.

              Show
              damyon Damyon Wiese added a comment - Waiting for feedback on the blocking issue.
              Hide
              damyon Damyon Wiese added a comment -

              Moving this one out of integration - can be resubmitted at the same time as the blocking issue.

              Show
              damyon Damyon Wiese added a comment - Moving this one out of integration - can be resubmitted at the same time as the blocking issue.
              Hide
              cibot CiBoT added a comment -

              Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.

              Show
              cibot CiBoT added a comment - Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.
              Hide
              lazydaisy Mary Evans added a comment -

              Because of the current issues with MDL-33934 I am submitting this issue first. Then deal with MDL-33934 after this is integrated.

              Show
              lazydaisy Mary Evans added a comment - Because of the current issues with MDL-33934 I am submitting this issue first. Then deal with MDL-33934 after this is integrated.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

              TIA and ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
              Hide
              lazydaisy Mary Evans added a comment -

              REBASED

              Show
              lazydaisy Mary Evans added a comment - REBASED
              Hide
              lazydaisy Mary Evans added a comment -

              I am very worried that nothing seems to be happening here although looking at the activity history it appears to be in integration. I just don't want to lose time with this as it is critical if I am to fix MyMobile theme.

              Can someone tell me what's happening?

              Thanks

              Show
              lazydaisy Mary Evans added a comment - I am very worried that nothing seems to be happening here although looking at the activity history it appears to be in integration. I just don't want to lose time with this as it is critical if I am to fix MyMobile theme. Can someone tell me what's happening? Thanks
              Hide
              nebgor Aparup Banerjee added a comment -

              This has been integrated into master, sorry if this fell off any radar. (i'd like to know how though)

              ps: could you link any discussion here about the removal of support if available just ftr. (it does make sense)

              Show
              nebgor Aparup Banerjee added a comment - This has been integrated into master, sorry if this fell off any radar. (i'd like to know how though) ps: could you link any discussion here about the removal of support if available just ftr. (it does make sense)
              Hide
              lazydaisy Mary Evans added a comment - - edited

              Just added an link Aparup, to MDL-33934 where there has been a lot of discussion, testing, rejection which MDL-38072 is currently blocking progress.

              Thanks for clarifying the state of this with this issue.
              Much appreciated.

              Show
              lazydaisy Mary Evans added a comment - - edited Just added an link Aparup, to MDL-33934 where there has been a lot of discussion, testing, rejection which MDL-38072 is currently blocking progress. Thanks for clarifying the state of this with this issue. Much appreciated.
              Hide
              lazydaisy Mary Evans added a comment -

              Aparup, I have just found the original tracker MDL-33115 where all the works was done.

              Show
              lazydaisy Mary Evans added a comment - Aparup, I have just found the original tracker MDL-33115 where all the works was done.
              Hide
              fred Frédéric Massart added a comment -

              Test passed, thanks!

              Show
              fred Frédéric Massart added a comment - Test passed, thanks!
              Hide
              nebgor Aparup Banerjee added a comment -

              Thanks for the linking them up Mary

              Show
              nebgor Aparup Banerjee added a comment - Thanks for the linking them up Mary
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              This is valid for unlimited entries to the, soon to be unveiled, Moodle Codebase Gardens. It includes free access to all facilities.

              Personal and non-transferable to all assignees, reviewers and testers in this issue. Valid until switching to Blackboard (100000€ penalization will be applied).

              Thanks, closing as fixed!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - This is valid for unlimited entries to the, soon to be unveiled, Moodle Codebase Gardens. It includes free access to all facilities. Personal and non-transferable to all assignees, reviewers and testers in this issue. Valid until switching to Blackboard (100000€ penalization will be applied). Thanks, closing as fixed!
              Hide
              lazydaisy Mary Evans added a comment -

              Show
              lazydaisy Mary Evans added a comment -

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  5 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    14/May/13