Moodle
  1. Moodle
  2. MDL-30857

Clicking on comments in the community viewer block doesn't display any comments in IE8

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.2.6, 2.3.3
    • Component/s: Blocks, Hub
    • Labels:
    • Environment:
      Windows 7 Home Basic, Internet Explorer 8.0.7600.16385
    • Testing Instructions:
      Hide

      This issue should be tested in different browsers, specially in IE family.

      1. Login as an admin user
      2. Add the "Community finder" block to moodle frontpage
      3. Click "Search" on the community finder block
      4. Browse through the databases until you find a course with comments (I've found courses with comments in MOOCH + find: courses I can download + keywords: water)
      5. Click on the "Comments (N)" you SHOULD be able to see a emerging overlay with the size of the text it contains
      Show
      This issue should be tested in different browsers, specially in IE family. Login as an admin user Add the "Community finder" block to moodle frontpage Click "Search" on the community finder block Browse through the databases until you find a course with comments (I've found courses with comments in MOOCH + find: courses I can download + keywords: water) Click on the "Comments (N)" you SHOULD be able to see a emerging overlay with the size of the text it contains
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-30857_master
    • Rank:
      33857

      Description

      Make sure that you are using Internet Explorer 8

      1) Add the community finder block.
      2) Do a search (MOOCH is good).
      3) Select either courses I can enroll in or courses I can download then search.
      4) Scroll down and find a course with a comment. Click and view.

      In Internet Explorer 8 a box pops up but no comments are displayed.

        Issue Links

          Activity

          Hide
          David Monllaó added a comment -

          Hi Jerome,

          I've discovered the issue while working on another issue, is still present in master, addding pull branches with proposed patch

          Show
          David Monllaó added a comment - Hi Jerome, I've discovered the issue while working on another issue, is still present in master, addding pull branches with proposed patch
          Hide
          Jérôme Mouneyrac added a comment - - edited

          If it works on other browsers too, you can submit to integration. Thanks.

          Show
          Jérôme Mouneyrac added a comment - - edited If it works on other browsers too, you can submit to integration. Thanks.
          Hide
          David Monllaó added a comment -

          Thanks Jerome, submitting for integration, tested with:

          • Windows
            • Chrome 21.0.1180.89
            • IE 8
            • Firefox 15.0
            • Opera 12.02
            • Safari 5.1.7
          • Linux
            • Firefox 15.0
            • Chrome 21.0.1180.89
          Show
          David Monllaó added a comment - Thanks Jerome, submitting for integration, tested with: Windows Chrome 21.0.1180.89 IE 8 Firefox 15.0 Opera 12.02 Safari 5.1.7 Linux Firefox 15.0 Chrome 21.0.1180.89
          Hide
          Sam Hemelryk added a comment -

          Thanks David, this has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks David, this has been integrated now
          Hide
          Tim Barker added a comment -

          Tested on IE8 and 9, Safari, Chrome and Firefox. It's all awesome

          Show
          Tim Barker added a comment - Tested on IE8 and 9, Safari, Chrome and Firefox. It's all awesome
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Gutta cavat lapidem, non vi sed saepe cadendo - Ovidio

          This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

          Thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Gutta cavat lapidem, non vi sed saepe cadendo - Ovidio This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads). Thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: