Статьи для каждого таба, в чем проблема?

Question

[Yuriy Lyadov]

Всем привет!
Делаю правки по сайту, на странице univerokrug.hse.perm.ru/announces

window.events_page = 1;
function show_else_ann(page) {

		var tabidd = jQuery('[data-tabs="ANNOUNCE"] .current').data('tab'); 
		if ( jQuery('[data-tabs="ANNOUNCE"] #tabid' +tabidd+ '.current').data('tab') ) { 

		page = window.events_page + 1; 
		window.events_page = page; 

		console.log(window.events_page = page);

		} 

		$('.tabs_panel>A').click(function(){
			window.events_page = 1;
		});


	window.events_page = page;
    jQuery.ajax({
        url: '<?= SITE_TEMPLATE_PATH; ?>/ajax/announces.php',
        data: {
     page: page, 
     code: '<?= $IBLOCK_CODE; ?>',
     tab: jQuery('[data-tabs="ANNOUNCE"] .current').data('tab'), 
     per: <?= $PER; ?>, 
     date: 2, 
<?php
if (isset($QUESTION) and $QUESTION) { ?>
     question: 1,
<?php } ?>
     href: '/announces/' <?= (isset($_REQUEST['ID']) ? ', id: ' . (int)$_REQUEST['ID'] : ''); ?>},
        method: 'POST',
        dataType: 'JSON',
        success: function(data) {
            if (data.items) {
                var i = 0;
                for (i in data.items) {
                    var tabidfix = jQuery('[data-tabs="ANNOUNCE"] .current').data('tab');
					jQuery('#cann' +tabidfix+ ' .event_container_a').append(data.items[i]);
                }
                if (data.items.length != <?= $PER; ?>) {
                    jQuery('.show_else_a').hide();
				}
            }
            if (data.items.length == 0) {
                jQuery('.show_else_a').hide();
			} 
        }
    });

}

При открытии всех статей, кнопка "Показать еще" как положено скрывается, но при переходе на другой месяц, она не появляется больше. Я сделал что бы при нажатии на месяц, кнопка появлялась и сделал что бы статьи откатывались на первые статьи
Статья отказывается:

$('.tabs_panel>A').click(function(){
window.events_page = 1;
});

Кнопка появляется при нажатии на месяц:

jQuery('.tabs_panel>A').click(function(){ 
		jQuery('.show_else_a').show(); 
		});

но проблема если человек возвращается на месяц где открыл уже все статьи, то при нажатии на "показать еще", открываются статьи заново...

Как же мне заставить скрипт что бы кнопка появлялась и исчезала от того есть ли еще статьи или нет???

Answer 1

[Айнур Валиев]

кнопка Показать еще у тебя только в контенте активного таба. Как она покажется в другом табе, если ее там нет? Для каждого таба должна быть своя, иначе будут показываться все, даже где уже скрыто.

Comments

[Yuriy Lyadov]

Ясно... Как же её заставить работать как положено?

[Айнур Валиев]

Юрий Лядов, выводи изначально в контенте каждого таба кнопку. А скрывай только ту, к-я исчерпала себя в данном контексте.

[Yuriy Lyadov]

Айнур Валиев, Ты про этот?:

if (data.items.length == 0) {
                jQuery('.show_else_a').hide();
      }

Я просто Ajax не совсем знаю

[Айнур Валиев]

Юрий Лядов, Аякс тут непричем. если честно весь код в целом написан так себе. пересмотри логику. Воспринимай контент каждого таба как отд. стр. или фрейм - т.е. все биндинги должны быть отдельные.
1. Как я понял show_else_ann ты вызываешь при клике на show_else_a. Параметр page там излишний, раз ты его заново присваиваешь внутри функции.
page = window.events_page + 1;
2. При каждом клике ты прибавляешь номер страницы. Это норм, но надо привязать это к контексту, иначе другие табы примут этот же номер стр.
3. Привяжи номер стр. к контексту самого таба. Добавь там какой нибудь дата атрибут. Кнопку показать еще скрывай по селектору того же таба, типа этого
#cann' +tabidfix+ ' .event_container_a

[Yuriy Lyadov]

Айнур Валиев, Я смотрю и что то не понимаю... :(
Сможешь по моему коду сделать правки? Что бы я увидел свою ошибку, что ты имел введу и т.д. ?

[Айнур Валиев]

Юрий Лядов, кароч в 2-х словах - добавь селектор для show_else_a и сохраняй номер стр. в контексте.

[Yuriy Lyadov]

Айнур Валиев, До моего вмешательства был такой код:

window.events_page = 1;
function show_else_ann(page) {
    if (page == undefined) {
        page = window.events_page + 1;
    }
    window.events_page = page;
    jQuery.ajax({
        url: '<?= SITE_TEMPLATE_PATH; ?>/ajax/announces.php',
        data: {
     page: page, 
     code: '<?= $IBLOCK_CODE; ?>',
     tab: jQuery('[data-tabs="ANNOUNCE"] .current').data('tab'), 
     per: <?= $PER; ?>, 
     date: 2, 
<?php
if (isset($QUESTION) and $QUESTION) { ?>
     question: 1,
<?php } ?>
     href: '/announces/' <?= (isset($_REQUEST['ID']) ? ', id: ' . (int)$_REQUEST['ID'] : ''); ?>},
        method: 'POST',
        dataType: 'JSON',
        success: function(data) {
            if (data.items) {
                var i = 0;
                for (i in data.items) {
                    var tabidfix = jQuery('[data-tabs="ANNOUNCE"] .current').data('tab');
                    jQuery('#cann' +tabidfix+ ' .event_container_a').append(data.items[i]);
                }
                if (data.items.length != <?= $PER; ?>) {
                    jQuery('.show_else_a').hide();
                }
            }
            if (data.items.length == 0) {
                jQuery('.show_else_a').hide();
            }
        }
    });
}

[Yuriy Lyadov]

Айнур Валиев, дал id кнопке.
Вообще кнопка на каждом табе своя.
По поводу "сохраняй номер стр. в контексте." не понял.

[Айнур Валиев]

Юрий Лядов, page = window.events_page + 1; тут page (номер стр.) глобальная переменная, должна быть локальной.