Bug Tracker

Opened 9 years ago

Closed 6 years ago

Last modified 6 years ago

#3260 closed bug (fixed)

Weird IE7 behavior when focussing on a text input inside a div with padding

Reported by: asprouse Owned by:
Priority: low Milestone: 1.next
Component: effects Version: 1.2.6
Keywords: IE7, slide Cc: flesler, asprouse
Blocked by: Blocking:

Description

Summary

Here is a very simple example which will produce this behavior:

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
<html>
    <head>
        <title>Test</title>
            <script type="text/javascript" src="jquery-1.2.6.js"></script>
            <style type="text/css">
                #box {				
                    padding: 10px;
                }
            </style>
            <script type="text/javascript">
                function show () {
                    $("#box").slideDown("slow",function(){
                        $("#target").focus();
                    });
                }
            </script>
    </head>
    <body>
        <input type="button" value="Click Here" onClick="show()" />
            <div id="box" style="display:none">
                <input type="text" id="target" />
            </div>
    </body>
</html

What we want to happen is to use the slideDown effect to show the "box" div and then focus on the "target" input. This works fine in firefox, however IE7 starts to do the slide down effect and once the callback is performed, "target disappears and the cursor appears to the left of the button. This does not happen if the padding is removed. Another way in which this works properly is if there is something to delay the focus() function call. For example:

function show () {
    $("#box").slideDown("slow",function(){
        alert("BOO");
        $("#target").focus();
    });
}

or

function show () {
    $("#box").slideDown("slow",function(){        
        setTimeout(function(){$("#target").focus();},1);
    });
}

It seems as if there is some issue with the callback be fired while IE is still calculating the padding, but that is just a guess.

Steps to Repeat

  1. Copy the html code from above into an .html file
  2. Download the latest stable jQuery (at this time 1.2.6)
  3. Load the .html file created in step 1 into IE7
  4. Click the "Click Here" button

Attachments (3)

ie7error.jpg (1.6 KB) - added by asprouse 9 years ago.
test.html (852 bytes) - added by asprouse 9 years ago.
test-3260.html (605 bytes) - added by dmethvin 7 years ago.

Download all attachments as: .zip

Change History (9)

Changed 9 years ago by asprouse

Attachment: ie7error.jpg added

comment:1 Changed 9 years ago by asprouse

Here is a screenshot of the error in IE7:

Changed 9 years ago by asprouse

Attachment: test.html added

comment:2 Changed 9 years ago by flesler

Cc: flesler asprouse added
Component: corefx

comment:3 Changed 7 years ago by dmethvin

Attaching a test case with jQuery 1.4.2, which seems to work in IE8. Needs to be tested with IE7.

Changed 7 years ago by dmethvin

Attachment: test-3260.html added

comment:4 Changed 7 years ago by SlexAxton

Keywords: IE7 slide added
Milestone: 1.31.5
Priority: majorlow
Status: newopen

Let's test Dave's patch and see if we think it's something we can get in soon. Seems valid as an IE7 compatibility issue.

comment:5 Changed 6 years ago by timmywil

Milestone: 1.next
Resolution: fixed
Status: openclosed

I believe this has been fixed http://jsfiddle.net/timmywil/awxA9/show. Feel free to reopen and provide an updated test case if necessary.

comment:6 Changed 6 years ago by timmywil

#6767 is a duplicate of this ticket.

Note: See TracTickets for help on using tickets.