Jump to content
  • 0

Прокомментируйте код


warpig
 Share

Question

Прокомментируйте код, это css и javascript, ваши комментарии будут паказаны автору кода в качестве убеждения тех или иных позиций, сам автор неможет зайти по технико/религиозным причинам

/* pagination */

.pagesWrapper { width: 100%; float: left; }
#pages { padding: 0 10px; margin: 0 0 10px; color: #91969c; font: 12px Verdana, Geneva, sans-serif; }
table#pages_table { width: 100%; border-collapse: collapse; table-layout: auto; }
table#pages_table td { }
table#pages_table td .count_result, table#pages_table td .count { font-weight: bold; }
table#pages_table td .count_result {}
table#pages_table td .count {}
table#pages_table td .paginator { display: block; float: left; padding: 2px 0 0; }
table#pages_table td a.navi_link { display: block; float: left; padding: 0 5px; color: #91969c; text-decoration: none; }
table#pages_table td .current_page { display: block; float: left; padding: 0 5px; font-weight: bold; }
a#ipage_prev, a#ipage_next {
display: block;
float: left;
width: 11px;
height: 14px;
margin: 0 5px 1px;
background: url(/images/admin_paginator-arrows-sprite.png) no-repeat;
text-decoration: none;
color: #91969c;
}
a#ipage_prev { background-position: 0 1px; }
a#ipage_next { background-position: 0 -14px; }

function setControls() {
$('#pageControls li a').each(function() {
$(this).addClass('icons')
.addClass('disabled')
.attr('title', $(this).text())
.text('')
.click(function() {
if ($(this).hasClass('disabled'))
return false
});
if ($(this).hasClass('dependent')) {
if ($(this).attr('title').toLowerCase().indexOf('edit') > -1 || $(this).attr('title').toLowerCase().indexOf('view') > -1) {
$(this).addClass('edit');
} else if ($(this).attr('title').toLowerCase().indexOf('delete') > -1) {
$(this).addClass('delete');
} else if ($(this).attr('title').toLowerCase().indexOf('approve') > -1 || $(this).attr('title').toLowerCase().indexOf('confirm') > -1) {
$(this).addClass('approve');
} else if ($(this).attr('title').toLowerCase().indexOf('decline') > -1 || $(this).attr('title').toLowerCase().indexOf('cancel') > -1) {
$(this).addClass('decline');
}
} else if ($(this).hasClass('independent')) {
$(this).removeClass('disabled');
if ($(this).attr('title').toLowerCase().indexOf('add') > -1 ||
$(this).attr('title').toLowerCase().indexOf('new') > -1 ||
$(this).attr('title').toLowerCase().indexOf('create' )> -1) {
$(this).addClass('add');
} else if ($(this).attr('title').toLowerCase().indexOf('list') > -1) {
$(this).addClass('list');
}
}
});
}




function initPageNavi() {
if ($('#pageNavi', '#rightColumn .content').length > 0) {
$('#pageNavi', '#rightColumn .content').clone().appendTo('#tabsPanel');
$('#pageNavi', '#tabsPanel').removeAttr('class');
$('#pageNavi', '#rightColumn .content').remove();
$('#pageNavi li.tab', '#tabsPanel').each(function(i) {
if ( i < $('#pageNavi li.tab', '#tabsPanel').length-1 ) {
$('<li class="tabSeparator"></li>').insertBefore($(this));
} else {
$('<li class="tabSeparator"></li>').insertBefore($(this));
$('<li class="tabSeparator"></li>').insertAfter($(this));
}
});
var document_link = '';
$('#pageNavi li a', '#tabsPanel').each(function() {
if ( $.browser.msie && $.browser.version.substr(0,2) == '7.' ) {
document_link = document.location.href;
} else {
document_link = document.location.pathname;
}
if ( document_link == $(this).attr('href') ) {
$(this).parent().addClass('activeTab');
$(this).parent().prev().addClass('left');
$(this).parent().next().addClass('right');
}
})
}
}

function initViewFilterButton() {
if ( $('.tableFilterWrapper').length > 0 ) {
//init filter if no empty
if ( $('.tableFilterWrapper').find('input[type="text"]').length > 0 || $('.tableFilterWrapper').find('select').length > 0 ) {
$('.tableFilterWrapper').jNice();
$('.tableWrapper:first').prepend('<div id="filterBottomWrapper"><div id="filterBottom"><div><div></div></div></div></div>')
.css({'padding':'10px 0 0'});
$('#rightColumnHead').append('<div id="tableFilterControl" title="Show filter"></div>');
$('#filter_button').val('');
//bind show-filter button
$('#tableFilterControl').bind({
'mouseover': function() {
$(this).addClass('hover');
},
'mousedown': function() {
$(this).removeClass('hover').addClass('push');
},
'mouseup': function() {
$(this).removeClass('push');
},
'mouseout': function() {
if($(this).hasClass('push')) {
$(this).removeClass('push');
}
if($(this).hasClass('hover')) {
$(this).removeClass('hover');
}
},
'click': function() {

if ($('.tableFilterWrapper').css('display') == 'block') {
$('#tableFilterControl').attr({'title':'Show filter'});
$('.tableFilterWrapper').slideUp('fast');
$('#filterBottomWrapper').remove()
$('.tableWrapper:first').css({'padding':'0'})
.removeAttr('style');
} else {
$('#tableFilterControl').attr({'title':'Hide filter'});
$('.tableFilterWrapper').slideDown('fast').jNice();
$('.tableWrapper:first').prepend('<div id="filterBottomWrapper"><div id="filterBottom"><div><div></div></div></div></div>')
.css({'padding':'10px 0 0'});
$('#filter_button').val('');
}

if ( $(this).hasClass('expanded') ) {
$(this).addClass('hover').removeClass('expanded');
} else {
$(this).addClass('hover').addClass('expanded');
}
}
});
} else {
$('.tableFilterWrapper').hide();
}
}
}

Это части кода CSS и javascript код с использованием jQuery, ведра для сблёва под windows правом верхнем углу ;),

Link to comment
Share on other sites

6 answers to this question

Recommended Posts

  • 0
А что не так то? Код как код, ничего "такого" в нем не вижу. Ну я противник, чтоб id в css так рьяно использовали...

Ну во первых

if ($('#pageNavi', '#rightColumn .content').length > 0) {
$('#pageNavi', '#rightColumn .content').clone().appendTo('#tabsPanel');
$('#pageNavi', '#tabsPanel').removeAttr('class');
$('#pageNavi', '#rightColumn .content').remove();
$('#pageNavi li.tab', '#tabsPanel').each(function(i) {

Повторение кода, безсмысленного и безпошадного

Использование диких конуструкций типо

else if ($(this).attr('title').toLowerCase().indexOf('approve') > -1

это больше похоже на парсер на javascript чем создание интерфейса

Link to comment
Share on other sites

  • 0

Я с jQuery не работаю, так что...

Если я правильно понимаю, то в jQuery поиск происходит так: $('что_ищем', 'где_ищем (контекст)'). Тогда повторения кода не вижу:

if ($('#pageNavi', '#rightColumn .content').length > 0) {
$('#pageNavi', '#rightColumn .content').clone().appendTo('#tabsPanel');
$('#pageNavi', '#tabsPanel').removeAttr('class');
$('#pageNavi', '#rightColumn .content').remove();
$('#pageNavi li.tab', '#tabsPanel').each(function(i) {

#pageNavi ищется в разном контексте. Ладно пара строк может и лишняя...

Такие конструкции для jQuery вроде нормальны... Я не прав?

$(this).attr('title').toLowerCase().indexOf('approve')

Link to comment
Share on other sites

  • 0

По коду какого-то супер идиотизма не вижу... Не понятна задача, вполне возможно что для достижения какой-то вашей цели, нет необходимости использовать скажем 50% кода... Возможно вообще намного выиграшней будет использовать AJAX или заранее генерировать какую то часть контента или может HTML структура идиотская ну и т.д.

Edited by stars
Link to comment
Share on other sites

  • 0

Прокомментируйте код, это css и javascript, ваши комментарии будут паказаны автору кода в качестве убеждения тех или иных позиций, сам автор неможет зайти

по технико/религиозным причинам

 

извините, а что такое технико/религиозные причины? это как?

/* pagination */.pagesWrapper { width: 100%; float: left; }#pages { padding: 0 10px; margin: 0 0 10px; color: #91969c; font: 12px Verdana, Geneva, sans-serif; }	table#pages_table { width: 100%; border-collapse: collapse; table-layout: auto; }		table#pages_table td {  }			table#pages_table td .count_result, table#pages_table td .count { font-weight: bold; }			table#pages_table td .count_result {}			table#pages_table td .count {}			table#pages_table td .paginator { display: block; float: left; padding: 2px 0 0; }				table#pages_table td a.navi_link { display: block; float: left; padding: 0 5px; color: #91969c; text-decoration: none; }				table#pages_table td .current_page { display: block; float: left; padding: 0 5px; font-weight: bold; }				a#ipage_prev, a#ipage_next { 					display: block;					float: left;					width: 11px;					height: 14px;					margin: 0 5px 1px;					background: url(/images/admin_paginator-arrows-sprite.png) no-repeat;					text-decoration: none;					color: #91969c;				}				a#ipage_prev { background-position: 0 1px; }				a#ipage_next { background-position: 0 -14px; }

function setControls() {	$('#pageControls li a').each(function() {		$(this).addClass('icons')			   .addClass('disabled')			   .attr('title', $(this).text())			   .text('')			   .click(function() {					if ($(this).hasClass('disabled'))						return false				});		if ($(this).hasClass('dependent')) {			if ($(this).attr('title').toLowerCase().indexOf('edit') > -1 || $(this).attr('title').toLowerCase().indexOf('view') > -1) {				$(this).addClass('edit');			} else if ($(this).attr('title').toLowerCase().indexOf('delete') > -1) {				$(this).addClass('delete');			} else if ($(this).attr('title').toLowerCase().indexOf('approve') > -1 || $(this).attr('title').toLowerCase().indexOf('confirm') > -1) {				$(this).addClass('approve');			} else if ($(this).attr('title').toLowerCase().indexOf('decline') > -1 || $(this).attr('title').toLowerCase().indexOf('cancel') > -1) {				$(this).addClass('decline');			}		} else if ($(this).hasClass('independent')) {			$(this).removeClass('disabled');			if ($(this).attr('title').toLowerCase().indexOf('add') > -1 ||				$(this).attr('title').toLowerCase().indexOf('new') > -1 ||				$(this).attr('title').toLowerCase().indexOf('create' )> -1) {				$(this).addClass('add');			} else if ($(this).attr('title').toLowerCase().indexOf('list') > -1) {				$(this).addClass('list');			}		}	});}function initPageNavi() {	if ($('#pageNavi', '#rightColumn .content').length > 0) {		$('#pageNavi', '#rightColumn .content').clone().appendTo('#tabsPanel');		$('#pageNavi', '#tabsPanel').removeAttr('class');		$('#pageNavi', '#rightColumn .content').remove();		$('#pageNavi li.tab', '#tabsPanel').each(function(i) {			if ( i < $('#pageNavi li.tab', '#tabsPanel').length-1 ) {				$('<li class="tabSeparator"></li>').insertBefore($(this));			} else {				$('<li class="tabSeparator"></li>').insertBefore($(this));				$('<li class="tabSeparator"></li>').insertAfter($(this));			}		});		var document_link = '';		$('#pageNavi li a', '#tabsPanel').each(function() {			if ( $.browser.msie && $.browser.version.substr(0,2) == '7.' ) {				document_link = document.location.href;			} else {				document_link = document.location.pathname;			}			if ( document_link == $(this).attr('href') ) {				$(this).parent().addClass('activeTab');				$(this).parent().prev().addClass('left');				$(this).parent().next().addClass('right');			}		})	}}function initViewFilterButton() {	if ( $('.tableFilterWrapper').length > 0 ) {		//init filter if no empty		if ( $('.tableFilterWrapper').find('input[type="text"]').length > 0 || $('.tableFilterWrapper').find('select').length > 0 ) {			$('.tableFilterWrapper').jNice();			$('.tableWrapper:first').prepend('<div id="filterBottomWrapper"><div id="filterBottom"><div><div></div></div></div></div>')									.css({'padding':'10px 0 0'});			$('#rightColumnHead').append('<div id="tableFilterControl" title="Show filter"></div>');			$('#filter_button').val('');			//bind show-filter button			$('#tableFilterControl').bind({				'mouseover': function() {					$(this).addClass('hover');				},				'mousedown': function() {					$(this).removeClass('hover').addClass('push');				},				'mouseup': function() {					$(this).removeClass('push');				},				'mouseout': function() {					if($(this).hasClass('push')) {						$(this).removeClass('push');					}					if($(this).hasClass('hover')) {						$(this).removeClass('hover');					}				},				'click': function() {										if ($('.tableFilterWrapper').css('display') == 'block') {						$('#tableFilterControl').attr({'title':'Show filter'});						$('.tableFilterWrapper').slideUp('fast');						$('#filterBottomWrapper').remove()						$('.tableWrapper:first').css({'padding':'0'})												.removeAttr('style');					} else {						$('#tableFilterControl').attr({'title':'Hide filter'});						$('.tableFilterWrapper').slideDown('fast').jNice();						$('.tableWrapper:first').prepend('<div id="filterBottomWrapper"><div id="filterBottom"><div><div></div></div></div></div>')												.css({'padding':'10px 0 0'});						$('#filter_button').val('');					}										if ( $(this).hasClass('expanded') ) {						$(this).addClass('hover').removeClass('expanded');					} else {						$(this).addClass('hover').addClass('expanded');					}				}			});		} else {			$('.tableFilterWrapper').hide();		}	}}

Это части кода CSS и javascript код с использованием jQuery, ведра для сблёва под windows правом верхнем углу ;),

 

Link to comment
Share on other sites

Guest
This topic is now closed to further replies.
 Share

  • Обсуждения

    • Актуальные контакты: Telegram: @Nikker_web E-Mail:   tarasevich.email@gmail.com Портфолио https://www.behance.net/d4d4186e Разрабатываю дизайн групп в соц сетях, сайтов, приложений, другой дизайн под заказ    
    • Актуальные контакты: Telegram: @Nikker_web E-Mail:   tarasevich.email@gmail.com   Разрабатываю дизайн групп в соц сетях, сайтов, приложений, другой дизайн под заказ   Портфолио https://www.behance.net/d4d4186e
    • Доброго всем времени суток. Прошу помощи. Научите принципу изменения футера. Движок Xenforo. Версия 2.2.10. Стиль дефолтный. Что именно нужно в итоге на фото примере. Мой шаблон app.footer less имеет следующее значение.  .p-footer { .xf-publicFooter(); a { .xf-publicFooterLink(); } } .p-footer-inner { .m-pageWidth(); .m-pageInset(); padding-top: @xf-paddingMedium; padding-bottom: @xf-paddingLarge; } .p-footer-row { .m-clearFix(); margin-bottom: -@xf-paddingLarge; } .p-footer-row-main { float: left; margin-bottom: @xf-paddingLarge; } .p-footer-row-opposite { float: right; margin-bottom: @xf-paddingLarge; } .p-footer-linkList { .m-listPlain(); .m-clearFix(); > li { float: left; margin-right: .5em; &:last-child { margin-right: 0; } a { padding: 2px 4px; border-radius: @xf-borderRadiusSmall; &:hover { text-decoration: none; background-color: fade(@xf-publicFooterLink--color, 10%); } } } } .p-footer-rssLink { > span { position: relative; top: -1px; display: inline-block; width: 1.44em; height: 1.44em; line-height: 1.44em; text-align: center; font-size: .8em; background-color: #4682B4; border-radius: 2px; } .fa-rss { color: white; } } .p-footer-copyright { margin-top: @xf-elementSpacer; text-align: center; font-size: @xf-fontSizeSmallest; } .p-footer-debug { margin-top: @xf-paddingLarge; text-align: right; font-size: @xf-fontSizeSmallest; .pairs > dt { color: inherit; } } @media (max-width: @xf-responsiveMedium) { .p-footer-row-main, .p-footer-row-opposite { float: none; } .p-footer-copyright { text-align: left; padding: 0 4px; // aligns with other links } }  
    • Нужны сайты с примерами верстки, типа https://css-tricks.com/. Типовые приемы и нестандартные на все случаи жизни. Накидайте ссылок.
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy