Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

listview refresh creates small artifact after divider for new dynamic elements #1576

Closed
mfn opened this Issue May 6, 2011 · 5 comments

Comments

Projects
None yet
4 participants

mfn commented May 6, 2011

Given the following example:

<!DOCTYPE html>
<html>
<head>
    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, minimum-scale=1, maximum-scale=1">
    <link rel="stylesheet"  href="jquery/jquery.mobile-1.0a4.1.min.css" />
    <script type="text/javascript" src="jquery/jquery-1.5.2.min.js"></script>
    <script type="text/javascript" src="jquery/jquery.mobile-1.0a4.1.min.js"></script>
</head>
<body>
<div id="index" data-role="page">
    <div data-role="content">
        <ul id="findme" data-role="listview" data-inset="true">
            <li data-role="list-divider">Test</li>
        </ul>
        <p>
            <a id="add-entry" href="#">Add entry</a>
        </p>
    </div>
</div>
<script type="text/javascript">
    $('#add-entry').click( function() {
        var li = $('<li></li>');
        li.text("some text");
        $('#findme').append(li);
        $('#findme').listview('refresh');
    });
</script>
</body>
</html>

after clicking the link, the element is added but there will be an artifact left on the list-divider, see http://i.imgur.com/3lkFs.png?7150

I tested this reproducible on FF4 and current Fennec. I compared the DOM with Firebug and figured that the class ui-corner-bottom was added which shouldn't be there in this case.

When I add this line to the last JS block it is displayed correctly:

$('#findme li:first-child').removeClass('ui-corner-bottom');
Contributor

hakanson commented May 18, 2011

Looks similar to jquery/jquery-mobile#1470

Contributor

toddparker commented Aug 8, 2011

Is this still and issue with Beta 2 and latest? If so, please provide a jsBin for testing.

Member

agcolom commented Feb 12, 2012

This now looks fixed to me: http://jsfiddle.net/agcolom/UZQsR/

mfn commented Feb 12, 2012

Confirmed, tested with Fennec -> fixed.

@mfn mfn closed this Feb 12, 2012

Member

agcolom commented Feb 12, 2012

Thanks :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment