two questions about rendering a navigation

[lecrackffm]

Hello everyone, 

i have written a simple function to render the Navigation for my "Onepager"

Here is the code: 

<?php
function renderOnepagenav($onepageroot) {
	$sections = $onepageroot->children;
	echo "<ul>";
	$id = '';
	foreach ($sections as $item) {
		$id ++;
		echo "<li><a href='#section-$id'>$item->title</a></li>";
	}
	echo "</ul>";
}

Every Sektion is a child-page of home in the backend.

It works fine but i would like to add two modifications, where i need your help:

1.  i would like to skip the first  (child-)page.

2. I would like to add a class only to the first  rendered <li>  Element.

 

All kind of advice is highly appreciated.

Thank you, 

Gregor

[adrian]

Typically the way to do this is define $i=0; before the foreach loop and then at the end (just inside) do: $i++ 

This lets you check which iteration of the loop you are at, so if $i==0 then "continue;"

Hope that helps - it's not a full code example, but hopefully it will help you to understand what is required.

[Harmen]

Hi,

I think this is not that difficult, you work already with an id which you can use now:

37 minutes ago, lecrackffm said:

1.  i would like to skip the first  (child-)page.

You could add a if statement for the very first id in your foreach loop:

$id = 0; // set it to zero
	foreach ($sections as $item) {
		if ($id == 0) continue
		else {
		echo "<li><a href='#section-$id'>$item->title</a></li>";
		}
		$id ++;
	}

NOTE: Haven't tested this

37 minutes ago, lecrackffm said:

2. I would like to add a class only to the first  rendered <li>  Element.

As you want to skip the first element, which id is equal to 0, the first rendered <li> will have id 1, so if you add another if statement in your foreach loop for id ==1 you can filter that item out and the rest gets the normal tags. See code below:

$id = 0; // set it to zero
	foreach ($sections as $item) {
		if ($id == 0) continue
		else {
			if ($id == 1){
				echo "<li class='yourclassforfirstitem'><a href='#section-$id'>$item->title</a></li>";
			}
		echo "<li><a href='#section-$id'>$item->title</a></li>";
		}
		$id ++;
	}

Haven't tested it so far, but I expect this works (maybe with some adjustments)

Edited January 27, 2017 by Harmen
edited the code ($i++ to the end)

[lecrackffm]

Thank you very much for the fast help, both of you.  I will test it asap as i am back at my desktop!

Have a nice weekend!

[horst]

On 27.1.2017 at 9:10 PM, Harmen said:

 

$id = 0; // set it to zero
	foreach ($sections as $item) {
		if ($id == 0) continue
		else {
		echo "<li><a href='#section-$id'>$item->title</a></li>";
		}
		$id ++;
	}

 

 

@Harmen you must ensure that you count up the $id before continue.

$id = -1; // set it -1, as we first count it up +1, (it will be 0 on its first check then!)
foreach($sections as $item) {
	$id++; // count up +1
	if($id == 0) continue; // skip the first child
	if($id == 1) {
		echo "<li class='yourclassforfirstitem'><a href='#section-$id'>$item->title</a></li>";
	} else {
		echo "<li><a href='#section-$id'>$item->title</a></li>";
	}
}

 

[Robin S]

1 hour ago, horst said:

@Harmen you must asure that you count up the $id before continue.

Isn't it just a stylistic preference whether you increment the counter at the beginning or the end of your operations?

[lecrackffm]

The solution @horst posted worked for me. I tried with $id++ at the end as well. But then it did not render anything.

Thank you for your support!

[Harmen]

5 hours ago, horst said:

you must asure that you count up the $id before continue.

You're right, but as I said. I did not tested it, and wrote this piece of code in a rush. Thanks for the addition.

3 hours ago, Robin S said:

Isn't it just a stylistic preference whether you increment the counter at the beginning or the end of your operations?

I think it is, most of the time I use it at the end of operations.

[horst]

5 hours ago, Robin S said:

Isn't it just a stylistic preference whether you increment the counter at the beginning or the end of your operations?

Sure! Only important thing is to do it!  -- before you continue

If $id is 0 and the first thing you do is: if(0 == $id) continue; nothing ever will happen.

[lecrackffm]

@horstThanks for the explanation.