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

×
×
  • 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