Можно ли применить onclick="ChangeColor( )", чтобы изменить цвет двух элементов одновременно?

Question

[andreypodkovyrov]

В javascript не силен, нужна подсказка. Хочу по клику изменить цвет двух ячеек в таблице.

Как пробую сделать:

<th id="sto">100</th>
<td onclick="ChangeColor( this, '#FEEC88' ); ChangeColor( sto, '#FEEC88' )">&nbsp;</td>

Не работает. По отдельности всё ок.

Answer 1

[Илья Лесных]

<th id="sto">100</th>
<td onclick="ChangeColor( this, '#FEEC88' ); ChangeColor( document.getElementById("sto"), '#FEEC88' )">&nbsp;</td>

без изменения JS. Либо так:

<th id="sto">100</th>
<td onclick="ChangeColor( [this, document.getElementById("sto")], '#FEEC88' )">&nbsp;</td>

но тогда заменяем и JS:

var last_change = [];
var last_color = [];
function ChangeColor( elements, color ){
   if (typeof element !== 'array') {
      elements = [elements];
   }
   for (var i = elements.length; --i;) {
      var element = elements[i];
      if( last_change[element.id] != null ){
          last_change[element.id].style.backgroundColor = last_color[element.id];
      }
      last_change[element.id] = element;
      last_color[element.id] = element.style.backgroundColor;
      element.style.backgroundColor = color;
   }
}

Answer 2

[Rsa97]

ChangeColor(document.getElementById("sto"), "#FEEC88");

Comments

[andreypodkovyrov]

это изменяет цвет одного элемента "sto", а мне нужно двух одновременно "sto" и текущий this

Answer 3

[4ikist]

А почему бы и нет, только с передачей элемента у вас наверное стоит пофиксить, а лучше сделайте поддержку массива элементов:

ChangeColor([this, '#sto'], '#FEEC88' )

Comments

[andreypodkovyrov]

Спасибо, похоже это то что нужно. Но не могли бы вы еще подсказать. Взял готовый код, для выделения одной ячейки работает. Когда делаю так как вы сказали — нет. Возможно нужно поправить вот эту строчку function ChangeColor( element, color ) ?

<script type="text/javascript">
<!-- //
var last_change = null;
var last_color = null;
function ChangeColor( element, color ){
   if( last_change != null ){
       last_change.style.backgroundColor = last_color;
   }
   last_change = element;
   last_color = element.style.backgroundColor;
   element.style.backgroundColor = color;
}
// -->
</script>

[Илья Лесных]

Вам дали зачем-то пример для jQuery селекторов, если делать все нативно - то как в совете от @Rsa97

[4ikist]

@Aliance наличие "#" ещё не делает гарантию выбора jQuery.