我想在点击时交换图像,但是如果我从向下图像向上点击向上,但如果点击该图像,则再次点亮图像... 发现一些错误 - x.attr(“src”)未定义
$(document).ready(function () {
$('#sortdivlink').click(function () {
var x = $("img[src$='../Images/Sort_down.png'][name='stCodeDSC']");
if (x.attr("SRC") == "../Images/Sort_down.png") {
x.attr('src', x.attr('src').replace("../Images/Sort_down.png", "../Images/sort_up.png"));
}
else {
x.attr('src', x.attr('src').replace("../Images/sort_up.png", "../Images/Sort_down.png"));
}
});
});
答案 0 :(得分:8)
我认为这是由于你用来分配x
的选择器,它可能会返回null。我会考虑在图像中添加Id或CSS类,然后将选择器简化为:
var x = $('img.sortDirectionImage');
或者,如果您知道sortdivlink
元素只有一个图像,则只需在该元素的上下文中搜索img
元素,例如:
// within the click function scope
var x = $('img', this);
修改强>
此外,我建议使用更多可识别的变量名称;你可能知道x
是什么,但下一个阅读你代码的人可能不会。
答案 1 :(得分:2)
您可以将<div>
与带有background-image
和尺寸的css类一起使用,并使用$(div).toggleClass(“new-image”)来使用更新后的background-image
这将大大简化您的代码。
[编辑]
<html>
<head>
<script src="http://ajax.googleapis.com/ajax/libs/jquery/1.7.2/jquery.min.js" type="text/javascript"></script>
<style>
.first-image
{
background-image: url( 'http://www.google.com/press/zeitgeist2001/google_logo.gif' );
width:218px;
height:90px;
background-position:center;
background-repeat:no-repeat;
}
.second-image
{
background-image: url( 'http://www.google.com/doodle4google/2008/images/regional_doodles/CO-80106-16a267f8-3.jpg' );
width:350px;
height:224px;
background-position:center;
background-repeat:no-repeat;
}
</style>
<script>
$(document).ready(function( )
{
$("#target-div").click( function( )
{
$("#target-div").toggleClass("second-image");
});
});
</script>
</head>
<body>
<div id="target-div" class="first-image"></div>
</body>
</html>
答案 2 :(得分:1)
看看我很久以前写的脚本(没有任何jQuery):
function changeImg(element) {
var img = document.getElementById(element);
if (img.src == "../images/expand.png") {
// collapse
img.src = "../images/collapse.png";
}
else {
// expand
img.src = "../images/expand.png";
}
}
普通HTML标记:
<span onclick="changeImg('ansprechpartner');return false;">...</span>
也许它会对你有用......