PHP Edit - Adding classes/ID's

[humpingdan]

Here is the PHP code to generate a ul:

//Construct an unordered list from $navlinks
//Accessibility: heading hidden from visual browsers by default.
$navigation = get_accesshide(get_string('youarehere','access'), 'h2')." <ul>\n";
$lastindex = count($navlinks) - 1;
$i = -1; // Used to count the times, so we know when we get to the last item.
$first = true;
foreach ($navlinks as $navlink) {
$i++;
$last = ($i == $lastindex);
if (!is_array($navlink)) {
continue;
}
if (!empty($navlink['type']) && $navlink['type'] == 'activity' && !$last && $hideactivitylink) {
continue;
}
$navigation .= '<li class="first">';
if (!$first) {
$navigation .= get_separator();
}
if ((!empty($navlink['link'])) && !$last) {
$navigation .= "<a onclick=\"this.target='$CFG->framename'\" href=\"{$navlink['link']}\">";
}
$navigation .= "{$navlink['name']}";
if ((!empty($navlink['link'])) && !$last) {
$navigation .= "</a>";
}

$navigation .= "</li>";
$first = false;
}
$navigation .= "</ul>";

return(array('newnav' => true, 'navlinks' => $navigation));
}

Output:

<ul>
<li class="first">Home</li>
<li class="first">Course</li>
<li class="first">Activity</li>
</ul>

But i want this:

<ul id="nav">
<li class="navtop">Home</li>
<li class="navitem"><a href="http://www.google.com">Course</a></li>
<li class="navitem"><a href="http://www.msn.com">Activity</a></li>
<li class="navitem"><a href="http://">Web Page</a></li>
</ul>

I've alread done: Added a class=nav to the <ul>

$navigation = get_accesshide(get_string('youarehere','access'), 'h2')." <ul class=nav>\n";

Now i need to output the first link with class=navtop

Any ideas?

[edited by: eelixduppy at 9:33 pm (utc) on Jan. 12, 2009]
[edit reason] snipped url [/edit]

[g1smd]

You don't need to repeat

class=navitem

on each link.

Each of those links can already be identified by

ul.nav li a { ... }

Note the spaces (NOT commas) between the items.

Add a class name to just the first item, not all of them.

[humpingdan]

Thanks, but how do i change the php to only run the code for the first link and not the others?

[g1smd]

If the links are written out in a loop then you will need to add the class name only on the first time round the loop.

Using the other notation to style the other links will make things a lot more easy to fix.

[Fixed typos]

[edited by: g1smd at 7:58 pm (utc) on Jan. 11, 2009]

[humpingdan]

Here is the PHP:

//Construct an unordered list from $navlinks
//Accessibility: heading hidden from visual browsers by default.
$navigation = get_accesshide(get_string('youarehere','access'), 'h2')." <ul>\n";
$lastindex = count($navlinks) - 1;
$i = -1; // Used to count the times, so we know when we get to the last item.
$first = true;
foreach ($navlinks as $navlink) {
$i++;
$last = ($i == $lastindex);
if (!is_array($navlink)) {
continue;
}
if (!empty($navlink['type']) && $navlink['type'] == 'activity' && !$last && $hideactivitylink) {
continue;
}
$navigation .= '<li class="first">';
if (!$first) {
$navigation .= get_separator();
}
if ((!empty($navlink['link'])) && !$last) {
$navigation .= "<a onclick=\"this.target='$CFG->framename'\" href=\"{$navlink['link']}\">";
}
$navigation .= "{$navlink['name']}";
if ((!empty($navlink['link'])) && !$last) {
$navigation .= "</a>";
}

$navigation .= "</li>";
$first = false;
}
$navigation .= "</ul>";

return(array('newnav' => true, 'navlinks' => $navigation));
}

As posted in the first post.