2009-08-03 3 views

ответ

38

Лучше практика отделить интерфейс от логики, в вашем случае:

$("#textboxid").addClass("aClass"); 

Если вам действительно нужно это ваш путь, то сделайте следующее:

$("#textboxid").css({"background-color": "color"}); 

Заменить #textboxid с нужный селектор и цвет с требуемым цветом.

Обратите внимание на следующем делает то же самое для одного свойства:

$("#textboxid").css("background-color", "color"); 
+1

Нет необходимости жестко кодированное представление в js. Используйте Css для стиля, js для поведения. – redsquare

10
$('#txtBoxID').css('background-color', '#ffff00'); 
8

Лучше, чтобы добавить имя класса к входу, а не жестким стилям кодирования в ваши JS. Стили презентации должны находиться в css не js.

$('#inputId').addClass('someCssClass'); 
+0

И это то, что вы делаете сейчас. Я бы сказал, что вы должны добавить семантический класс, а не backgroundRed, я вижу это как антипаттерн. И, возможно, здесь код UI. – Dykam

+0

это был всего лишь образец ... конечно, у вас не должно быть классов для каждого цвета. geee – redsquare

+0

Вы все еще не должны жестко писать стиль кода в js. Что происходит, когда ему нужна граница, а затем другой шрифт. Кончаться с уродливым беспорядком js. – redsquare

0

Еще лучше, не используйте jQuery. Я предполагаю, что вы хотите сделать это в центре внимания или размытия, так что вы можете сделать:

#textboxid {background-color:defaultcolor;} 
#textboxid:focus {background-color:selectedcolor;} 
Смежные вопросы