Editing highest in row (ol li)

This edit will create a new revision.

Your details (optional)

Name

Email (won’t be displayed; might be used for Gravatar)

URL

Test case details

Title *

Published (uncheck if you want to fiddle around before making the page public)

Description (in case you feel further explanation is needed)(Markdown syntax is allowed)

Are you a spammer? (just answer the question)

Preparation code

Preparation code HTML (this will be inserted in the <body> of a valid HTML5 document in standards mode) (useful when testing DOM operations or including libraries)

Include JavaScript libraries as follows: <script src="//cdn.ext/library.js"></script>

Define setup for all tests (variables, functions, arrays or other objects that will be used in the tests) (runs before each clocked test loop, outside of the timed code region) (e.g. define local test variables, reset global variables, clear canvas, etc.) (see FAQ)

Define teardown for all tests (runs after each clocked test loop, outside of the timed code region) (see FAQ)

Code snippets to compare

Test 1

Title

Async (check if this is an asynchronous test)

Code

establishheights();
function establishheights(){
	ape = 0;
	servicesheight= 0;
	
	$('#featured-work li').each(function(i,item){//going through each li measuring the highest height, then applying that height to all li's (if viewport > 480), BvL
		var $services = $(this).find('.services');
		
		if ( $services.length  && $services.css('display') != 'none' ) {
			servicesheight = $services.outerHeight();
		}else { servicesheight = 0 }
	
		aper =  $(this).find('h3').outerHeight() + $(this).find('.thumb').outerHeight() + $(this).find('p').outerHeight() + servicesheight;
		
		if($(window).width() > 480) {
			if(ape < aper){
				ape = aper;
				}
		}else {
				$(this).css('height', aper );
				}
	});
	if($(window).width() > 480) { $('#featured-work li').css('height',ape);};// if window size is smaller then 480 #featured-work li is one column and doesn't the highest height, BvL
};

Test 2

Title

Async (check if this is an asynchronous test)

Code

establishheights();
function establishheights(){
	// calculating highest li within row (number per row can vary), BvL
	rownumber = 0;
	liheight = 0;
	lihighest = 0;
	highestofrow = [];
	$('#featured-work li').each(function(){
		$(this).css('height', '10px');
		//thisclass = $(this).attr('class');
		//$(this).removeClass(thisclass);
		$(this).removeClass();
		

		//console.log($(this).position().top);
		if($(this).next().length > 0) {
			if($(this).position().top == $(this).next().position().top) {
				$(this).addClass('row'+rownumber);
				var $services = $(this).find('.services');
				if ( $services.length  && $services.css('display') != 'none') {
					servicesheight = $services.outerHeight();
				}else { servicesheight = 0 }
				liheight =  $(this).find('h3').outerHeight() + $(this).find('.thumb').outerHeight() + $(this).find('p').outerHeight() + servicesheight;
				if(liheight > lihighest) {lihighest = liheight;}
			}
			else { 
				$(this).addClass('row'+rownumber);
				var $services = $(this).find('.services');
				if ( $services.length  && $services.css('display') != 'none') {
					servicesheight = $services.outerHeight();
				}else { servicesheight = 0 }
				liheight =  $(this).find('h3').outerHeight() + $(this).find('.thumb').outerHeight() + $(this).find('p').outerHeight() + servicesheight;
				if(liheight > lihighest) {lihighest = liheight;}
				highestofrow.push(lihighest);
				rownumber = rownumber +1; 
				lihighest = 0;
			}
		}
		else {	
			$(this).addClass('row'+rownumber);
			var $services = $(this).find('.services');
			if ( $services.length  && $services.css('display') != 'none') {
				servicesheight = $services.outerHeight();
			}else { servicesheight = 0 }
			liheight =  $(this).find('h3').outerHeight() + $(this).find('.thumb').outerHeight() + $(this).find('p').outerHeight() + servicesheight;
			if(liheight > lihighest) {lihighest = liheight;}

			highestofrow.push(lihighest);
			

		};
	});
	count = 0;
	$('#featured-work li').each(function(){
		if($(this).hasClass('row'+count)){
			//console.log(highestofrow[count]);
			$(this).css('height', highestofrow[count]+'px');
		}else{
			count++;
			$(this).css('height', highestofrow[count]+'px');
		};
	});

};