Добавление активного класса на вкладки из события кнопки onclick - PullRequest
0 голосов
/ 15 апреля 2020

Я пытаюсь понять событие onclick, но запутался?

У меня есть кнопка onclick внутри содержимого "вкладки". Когда я нажимаю эту кнопку, он открывает правильную вкладку с названием «Токио», но не переносит и не добавляет активный класс.

Моя кнопка в содержимом первой вкладки:

<button class="new-btn" onclick="openCity(event, 'Tokyo')">Click Me</button>

Я не понимаю, как заставить его делать то же самое событие onclick, но добавить активный класс на вкладку "Токио" из этого нового нажатия кнопки?

Вот полный код:

<style>
body {font-family: Arial;}

/* Style the tab */
.tab {
  overflow: hidden;
  border: 1px solid #ccc;
  background-color: #f1f1f1;
}

/* Style the buttons inside the tab */
.tab button {
  background-color: inherit;
  float: left;
  border: none;
  outline: none;
  cursor: pointer;
  padding: 14px 16px;
  transition: 0.3s;
  font-size: 17px;
}

/* Change background color of buttons on hover */
.tab button:hover {
  background-color: #ddd;
}

/* Create an active/current tablink class */
.tab button.active {
  background-color: #ccc;
}

/* Style the tab content */
.tabcontent {
  display: none;
  padding: 6px 12px;
  border: 1px solid #ccc;
  border-top: none;
}
</style>
<div class="tab">
  <button class="tablinks" onclick="openCity(event, 'London')" id="defaultOpen">London</button>
  <button class="tablinks" onclick="openCity(event, 'Paris')">Paris</button>
  <button class="tablinks" onclick="openCity(event, 'Tokyo')">Tokyo</button>
</div>

<div id="London" class="tabcontent">
  <h3>London</h3>
  <p>London is the capital city of England.</p>
  <button class="new-btn" onclick="openCity(event, 'Tokyo')">Click Me</button>
</div>

<div id="Paris" class="tabcontent">
  <h3>Paris</h3>
  <p>Paris is the capital of France.</p>
</div>

<div id="Tokyo" class="tabcontent">
  <h3>Tokyo</h3>
  <p>Tokyo is the capital of Japan.</p>
</div>

<script>
function openCity(evt, cityName) {
  var i, tabcontent, tablinks;
  tabcontent = document.getElementsByClassName("tabcontent");
  for (i = 0; i < tabcontent.length; i++) {
    tabcontent[i].style.display = "none";
  }
  tablinks = document.getElementsByClassName("tablinks");
  for (i = 0; i < tablinks.length; i++) {
    tablinks[i].className = tablinks[i].className.replace(" active", "");
  }
  document.getElementById(cityName).style.display = "block";
  evt.currentTarget.className += " active";
}

// Get the element with id="defaultOpen" and click on it
document.getElementById("defaultOpen").click();
</script>

Ответы [ 2 ]

1 голос
/ 15 апреля 2020

Вы добавляете активный класс к своей кнопке вместо активной вкладки. Вы можете выбрать правильную активную вкладку по идентификатору или использовать индекс вкладки вместо названия города.

document.getElementById(cityName + "-tab").classList.add("active");
// evt.currentTarget.className += " active";

Например: https://jsfiddle.net/90h5nwds/

0 голосов
/ 15 апреля 2020

Вы можете получить нажатие кнопки с помощью event.target. Тогда вы просто добавите к нему свой класс, используя classList.add(). Вы не хотите устанавливать значение атрибута всего класса с помощью className, потому что это уничтожает другие классы.

...