Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #1 (permalink)  
Старый 17.02.2022, 18:28
Интересующийся
Отправить личное сообщение для ColT Посмотреть профиль Найти все сообщения от ColT
 
Регистрация: 14.02.2020
Сообщений: 16

Оптимизировать скрипт
Всем привет.
В общем у меня есть кусок кода, он рабочий но вот гугл кричит что он не оптимизирован и почему то в мобильной версии из-за этого кода не грузится почему то css. Может можно как то оптимизировать это?

<div class="pleerbac">  

<div class="tab">
 <button class="tablinks active" onclick="openCity(event, 'mm1');update_iframe();update_iframe2();update_iframe3(); return false;"><span style="color: #f59c12;">Кнопка 1</span></button>

<button class="tablinks" onclick="openCity(event, 'mm2');update_iframe();update_iframe2();update_iframe3(); return false;"><span style="color: #f59c12;">Кнопка 2</span></button>
 
<button class="tablinks" onclick="openCity(event, 'mm3');update_iframe();update_iframe2();update_iframe3(); return false;"><span style="color: #f59c12;">Кнопка 3</span></button>
</div>

<div id="mm1" class="tabcontent active">
<div style="text-align:center;"><iframe id="FrameID1" allowfullscreen="" src="https://turksezon.com/playlist/745" width="100%" height="430" frameborder="0" scrolling="no"></iframe></div>
</div>

<div id="mm2" class="tabcontent">
<div style="text-align:center;"><iframe id="FrameID2" allowfullscreen="" src="https://turksezon.com/playlist/745" width="100%" height="430" frameborder="0" scrolling="no"></iframe></div>
</div>

<div id="mm3" class="tabcontent">
<div style="text-align:center;"><iframe id="FrameID3" allowfullscreen="" src="https://turksezon.com/playlist/745" width="100%" height="430" frameborder="0" scrolling="no"></iframe></div>
</div>
  
	</div>


и сам JS

<script type="text/javascript">
function update_iframe(){
	var iframe = document.getElementById('FrameID');
	iframe.src = iframe.src;
}
function update_iframe2(){
	var iframe = document.getElementById('FrameID2');
	iframe.src = iframe.src;
}
function update_iframe3(){
	var iframe = document.getElementById('FrameID3');
	iframe.src = iframe.src;
}
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";
}
</script>


Прошу помочь мне и заранее благодарен за ваше время!
Ответить с цитированием
  #2 (permalink)  
Старый 17.02.2022, 18:52
Аватар для Aetae
Тлен
Отправить личное сообщение для Aetae Посмотреть профиль Найти все сообщения от Aetae
 
Регистрация: 02.01.2010
Сообщений: 6,590

Скрипт прост как палка, ни на что влиять не должен. Если гугл на что-то может жаловаться - так это на фреймы. Каждый фрейм - это по сути отдельный сайт. Три фрейма типа как загрузка 4х сайтов одновременно.
__________________
29375, 35
Ответить с цитированием
  #3 (permalink)  
Старый 17.02.2022, 18:53
Аватар для ksa
ksa ksa вне форума
CacheVar
Отправить личное сообщение для ksa Посмотреть профиль Найти все сообщения от ksa
 
Регистрация: 19.08.2010
Сообщений: 14,228

Сообщение от ColT
onclick="openCity(event, 'mm1');update_iframe();update_iframe2();update_iframe3(); return false;"
Такое определенно можно оптимизировать...

И все
Сообщение от ColT
function update_iframe*() {}
Ответить с цитированием
  #4 (permalink)  
Старый 17.02.2022, 20:31
Аватар для voraa
Профессор
Отправить личное сообщение для voraa Посмотреть профиль Найти все сообщения от voraa
 
Регистрация: 03.02.2020
Сообщений: 2,750

Гугл не ругается на код js. Там оптимизировать нечего. А если и есть чего, то это все микросекунды.
Надо точно смотреть, что он считает неоптимальным. Там есть более подробные отчеты.

А CSS в коде вообще не видно, где он там должен грузиться
Ответить с цитированием
Ответ



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Скрипт в скрипт :) Sakyra Общие вопросы Javascript 6 22.05.2013 09:58
Помогите вставить скрипт на страницу alexsio Работа 7 22.04.2013 18:19
Не работает скрипт :( VladimirV Javascript под браузер 5 21.12.2010 14:26
Люди, помогите адаптировать скрипт под Оперу KiLLk Opera, Safari и др. 1 01.06.2009 01:05
добавление окошка в скрипт подсчета слов Гость Общие вопросы Javascript 10 11.03.2008 17:07