Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Course
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a course and set the format to topics and layout to a single section per page
      2. Turn on editing
      3. In a topic header add an image via file upload and insert with file picker.
      4. Save.
      5. Image should be visible.
      6. Turn off editing
      7. Click short name in navigation bar to go back to summary page
      8. Image should be visible
      9. Switch course mode to multiple topics on one page
      10. Image should be visible
      11. Turn on editing
      12. Image should be visible
      13. Turn off editing
      14. image should be visible
      Show
      Create a course and set the format to topics and layout to a single section per page Turn on editing In a topic header add an image via file upload and insert with file picker. Save. Image should be visible. Turn off editing Click short name in navigation bar to go back to summary page Image should be visible Switch course mode to multiple topics on one page Image should be visible Turn on editing Image should be visible Turn off editing image should be visible
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
    • Rank:
      39921

      Description

      Select single section mode
      In topic header section add image via file upload and insert with file picker.
      Save. note image is visible
      Click short name in navigation bar.
      Note:
      1. Image not visible
      2. Section does not expand to fit image (it does with an embedded youtube video).

        Issue Links

          Activity

          Hide
          Michael de Raadt added a comment -

          There are issues with images in labels and headings and their affect on the flow of the page. I'll link to an issue that may be relevant.

          However, images in these headings should be appearing consistently.

          Show
          Michael de Raadt added a comment - There are issues with images in labels and headings and their affect on the flow of the page. I'll link to an issue that may be relevant. However, images in these headings should be appearing consistently.
          Hide
          Dan Poltawski added a comment -

          Hi Ray,

          I'm not seeing this problem at all, for me both the image is available and the section is expanding on firefox and chrome.

          Trying with IE now.

          Show
          Dan Poltawski added a comment - Hi Ray, I'm not seeing this problem at all, for me both the image is available and the section is expanding on firefox and chrome. Trying with IE now.
          Hide
          Dan Poltawski added a comment -

          Ah, found it - the missing step was switching off editing and viewing in 'summary mode'. Updated testing instructions and going to fix now

          Show
          Dan Poltawski added a comment - Ah, found it - the missing step was switching off editing and viewing in 'summary mode'. Updated testing instructions and going to fix now
          Hide
          Dan Poltawski added a comment -

          Submitting for integration.

          Thanks a lot for the report Ray!

          Show
          Dan Poltawski added a comment - Submitting for integration. Thanks a lot for the report Ray!
          Hide
          Ray Lawrence added a comment -

          My pleasure.

          Cheers.

          Show
          Ray Lawrence added a comment - My pleasure. Cheers.
          Hide
          Sam Hemelryk added a comment -

          Thanks Dan, this has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks Dan, this has been integrated now
          Hide
          Adrian Greeve added a comment -

          Works as directed. No problems found during testing.
          Thanks.

          Show
          Adrian Greeve added a comment - Works as directed. No problems found during testing. Thanks.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome?

          Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome? Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: