Result Todo:余計なコードがあります。もっと短く出来ますね。 jQuery$(document).ready(function() { //a要素をクリックでイベント発火 $('ul#filter a').click(function() { $(this).css('outline','none'); $('ul#filter .current').removeClass('current'); $(this).parent().addClass('current'); //小文字/大文字の処理 var filterVal = $(this).text().toLowerCase().replace(' ','-'); if(filterVal == 'all') { //マッチしない要素を隠す $('ul#portfolio li.hidden').slideDown('sl