Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Forms Library, Usability
    • Labels:
    • Testing Instructions:
      Hide
      1. Add a new URL resource
      2. Make sure
        • The URL field is in a section called Content
        • The content section is expanded by default
        • The file picker opens normally when you click "Choose a link..."
        • The Parameters section does not contain advanced settings

      (The description field is collapsed in the screenshots, but that is part of another issue, please ignore it)

      Show
      Add a new URL resource Make sure The URL field is in a section called Content The content section is expanded by default The file picker opens normally when you click "Choose a link..." The Parameters section does not contain advanced settings (The description field is collapsed in the screenshots, but that is part of another issue, please ignore it)
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-38612-master-int

      Description

      Reformat URL resource settings form

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            fred Frédéric Massart added a comment -

            Screenshots of the suggested solution.

            Show
            fred Frédéric Massart added a comment - Screenshots of the suggested solution.
            Hide
            tsala Helen Foster added a comment - - edited

            Good work making parameters not advanced settings.

            Just noting that we have to keep other advanced settings such as pop-up width and height, since they are set by an admin for the site.

            Also noting that perhaps it's better to keep the URL setting in a section called 'Content' for consistency with other resources https://docs.google.com/spreadsheet/ccc?key=0AmamVlPpCnn4dGx6SlEyMXQ3RzdJYjZCWnFya1ZtN1E#gid=1 Apologies for changing my mind...

            Show
            tsala Helen Foster added a comment - - edited Good work making parameters not advanced settings. Just noting that we have to keep other advanced settings such as pop-up width and height, since they are set by an admin for the site. Also noting that perhaps it's better to keep the URL setting in a section called 'Content' for consistency with other resources https://docs.google.com/spreadsheet/ccc?key=0AmamVlPpCnn4dGx6SlEyMXQ3RzdJYjZCWnFya1ZtN1E#gid=1 Apologies for changing my mind...
            Hide
            fred Frédéric Massart added a comment -

            Thanks Helen. That's fixed.

            Show
            fred Frédéric Massart added a comment - Thanks Helen. That's fixed.
            Hide
            salvetore Michael de Raadt added a comment -

            A nice simple fix.

            [Y] Syntax
            [Y] Output
            [Y] Whitespace
            [-] Language
            [-] Databases
            [Y] Testing
            [-] Security
            [-] Documentation
            [Y] Git
            [Y] Sanity check

            I ammended the testing instructions slightly. I hope that is OK.

            Show
            salvetore Michael de Raadt added a comment - A nice simple fix. [Y] Syntax [Y] Output [Y] Whitespace [-] Language [-] Databases [Y] Testing [-] Security [-] Documentation [Y] Git [Y] Sanity check I ammended the testing instructions slightly. I hope that is OK.
            Hide
            damyon Damyon Wiese 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.

            Thanks!

            Show
            damyon Damyon Wiese 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. Thanks!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Success!

            Show
            salvetore Michael de Raadt added a comment - Test result: Success!
            Hide
            poltawski Dan Poltawski added a comment -

            Did you remember to call thankDevelopers() for 'this_weeks_work'? Defaulting to PARAM_SHODDY thanking.

            line 1289 of \lib\changes.php: call to debugging()
            line 281 of \lib\are.php: call to moodleform->detectMissingThanks()
            line 202 of \lib\now.php: call to moodleform->_is_poor_form()
            line 73 of \course\upstream.php: call to moodleform->forgetingToThank()

            Show
            poltawski Dan Poltawski added a comment - Did you remember to call thankDevelopers() for 'this_weeks_work'? Defaulting to PARAM_SHODDY thanking. line 1289 of \lib\changes.php: call to debugging() line 281 of \lib\are.php: call to moodleform->detectMissingThanks() line 202 of \lib\now.php: call to moodleform->_is_poor_form() line 73 of \course\upstream.php: call to moodleform->forgetingToThank()

              People

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

                Dates

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