jQuery addClass и removeClass onchange проблема - PullRequest
Новая
       6

jQuery addClass и removeClass onchange проблема

2 голосов
/ 19 октября 2011

Я пытаюсь заставить addQlass и removeClass jQuery работать в среде выбора форм с небольшим успехом.

Я хочу изменить класс родительского div и пытался передать идентификатор в функцию, но всемои попытки были тщетными.Любое направление и помощь будет высоко ценится. 

<div id="MaxAge" class="SearchFormStyle">
                <select name="maxage" onchange="addClass(MaxAge);">
                    <option value="0">Age</option>
                    <option value="1" >1 Year Old</option>
                    <option value="2" >2 Years Old</option>
                </select>
                    <span class="ThisValue">
                        <a href="#" onclick="RemoveClass(MaxAge)">VALUE ENTERED HERE</a>
                    </span>
                </div>

<script>
    function addClass (id) { 
        div = "#" + id; 
        $(div).addClass("InputSet"); 
    }
    function RemoveClass (id) { 
        div = "#" + id; 
        $(div).removeClass("InputSet"); 
    }
</script>

Ответы [ 3 ]

3 голосов
/ 19 октября 2011

addClass(MaxAge) передаст значение переменной MaxAge в addClass, а не в строку 'MaxAge'.

Кроме того, вам действительно следует использовать jQuery для привязки обработчиков событий вместо встроенных обработчиков событий: 

$("select[name='maxage']").change(function() {
    addClass('MaxAge');
});

$('.ThisValue a').click(function(event) {
    RemoveClass('MaxAge');
    event.preventDefault();
});

Чтобы узнать больше об обработчиках событий, я рекомендую прочитать статьи в quirksmode.org .

Есть и другие проблемы, например, переменная div в addClass и RemoveClass, являющаяся global .Не забывайте var, когда вы объявляете переменные.Кроме того, наименование этой функции несовместимо add против Remove.

Руководство по MDN JavaScript очень полезно для изучения основ JavaScript.

0 голосов
/ 19 октября 2011

Вы никогда не создаете экземпляр переменной div.

Попробуйте это: 

var div = "#" + id;
0 голосов
/ 19 октября 2011

вы почти все делаете правильно.Вот что я бы сделал (с некоторыми комментариями для вас) 

// onchange event of the selectbox with id = maxage
$("#maxage").change(function() {
    $("#MaxAge").addClass("InputSet");
});

// the click event for the a element within span with class ThisValue
$("span.ThisValue a").click(function(e) {
    // prevent doing default behaviour like going to another location...
    e.preventDefault();
    $("#MaxAge").removeClass("InputSet");
});
...