Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.1
    • Fix Version/s: 2.4
    • Component/s: Tool Forum
    • Labels:
      None
    • Environment:
      Mac Safari 3.0.4

      Description

      See screenshots
      1. Picture 2.png
        60 kB
      2. Picture 3.png
        38 kB

        Activity

        Hide
        ernieg Ernie Ghiglione added a comment -
        OK, this is a bit funny though. It seems to be an issue with the way safari renders the body class="stripes".

        Just slightly modify the size of the learner window and you'll see that it renders to the full page. :-)

        The way I thought of fixing this is to add a bunch of <p>&nbsp;</p> just before </body> but that's such a stupid fix though and it just basically adds garbage to the page, so I'm not happy doing it :-)

        Show
        ernieg Ernie Ghiglione added a comment - OK, this is a bit funny though. It seems to be an issue with the way safari renders the body class="stripes". Just slightly modify the size of the learner window and you'll see that it renders to the full page. :-) The way I thought of fixing this is to add a bunch of <p>&nbsp;</p> just before </body> but that's such a stupid fix though and it just basically adds garbage to the page, so I'm not happy doing it :-)
        Hide
        ernieg Ernie Ghiglione added a comment -
        Just for the sake of documentation, I played around with the DOCTYPE to see if it might affect the way Safari would do the rendering.

        The forum page uses html4/strict.dtd where are other tools use loose.dtd or frameset.dtd.

        Even when I changed the DOCTYPE to either loose or frameset, it won't change the whitespace at the bottom of the page
        Show
        ernieg Ernie Ghiglione added a comment - Just for the sake of documentation, I played around with the DOCTYPE to see if it might affect the way Safari would do the rendering. The forum page uses html4/strict.dtd where are other tools use loose.dtd or frameset.dtd. Even when I changed the DOCTYPE to either loose or frameset, it won't change the whitespace at the bottom of the page
        Hide
        ernieg Ernie Ghiglione added a comment -
        Shelly, would you have any ideas why this might occur?
        Show
        ernieg Ernie Ghiglione added a comment - Shelly, would you have any ideas why this might occur?
        Hide
        fionam Fiona Malikoff added a comment -
        The culprit appears to the "replaceable" div, which allows for files to be attached. If I remove the javascript relating to it, then the pages work. Task List now has the same problem.
        Show
        fionam Fiona Malikoff added a comment - The culprit appears to the "replaceable" div, which allows for files to be attached. If I remove the javascript relating to it, then the pages work. Task List now has the same problem.
        Hide
        fionam Fiona Malikoff added a comment -
        More debugging: given this code in the learner page in share resources (with allow learner to add url/file turned on), I see the following behaviour on loading the page:

        function showMessage(url) {
        var area=document.getElementById("reourceInputArea");
        if(area != null){
        alert("Setting to 100%");
        area.style.width="100%";
        area.style.height="100%";
        area.src=url;
        area.style.display="block";
        }
        }


        <iframe
        onload="javascript:alert(this.contentWindow.document.body.scrollHeight);this.style.height=this.contentWindow.document.body.scrollHeight+'px'"
        id="reourceInputArea" name="reourceInputArea"
        style="width: 0px; height: 0px; border: 0px; display: none"
        frameborder="no" scrolling="no">
        </iframe>

        Firefox:
        Alert - "Setting to 100%", no iframe shown, no buttons below iframe shown. Click okay.
        White area appears on screen along with bottom buttons.
        Alert "346" and some of the Title/URL stuff appears. Click okay.
        "Add" section appears properly.

        Mac Safari
        Alert "8" and all of jsp frame white.
        Alert - "Setting to 100%", and all of jsp frame white.
        Alert "585" and all of fields have appeared onscreen, with the extra white space.

        So it appears the calculation of the iframe height isn't correct.
        Show
        fionam Fiona Malikoff added a comment - More debugging: given this code in the learner page in share resources (with allow learner to add url/file turned on), I see the following behaviour on loading the page: function showMessage(url) { var area=document.getElementById("reourceInputArea"); if(area != null){ alert("Setting to 100%"); area.style.width="100%"; area.style.height="100%"; area.src=url; area.style.display="block"; } } <iframe onload="javascript:alert(this.contentWindow.document.body.scrollHeight);this.style.height=this.contentWindow.document.body.scrollHeight+'px'" id="reourceInputArea" name="reourceInputArea" style="width: 0px; height: 0px; border: 0px; display: none" frameborder="no" scrolling="no"> </iframe> Firefox: Alert - "Setting to 100%", no iframe shown, no buttons below iframe shown. Click okay. White area appears on screen along with bottom buttons. Alert "346" and some of the Title/URL stuff appears. Click okay. "Add" section appears properly. Mac Safari Alert "8" and all of jsp frame white. Alert - "Setting to 100%", and all of jsp frame white. Alert "585" and all of fields have appeared onscreen, with the extra white space. So it appears the calculation of the iframe height isn't correct.
        Hide
        ernieg Ernie Ghiglione added a comment -
        Fixed in Safari 4
        Show
        ernieg Ernie Ghiglione added a comment - Fixed in Safari 4

          People

          • Assignee:
            ernieg Ernie Ghiglione
            Reporter:
            fionam Fiona Malikoff
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development