Sitenav li Klasse leer?

[Cohnina]

Ich wollte gerne wieder farbige Trennstriche in meiner Seitennavigation haben und hab dafür in der CSS Datei (blue von bulletproof; ja ich weiß, ich muss mir mal eins selbst erstellen) mein CSS Gewuschel eingefügt (für #sbsitenav li, border-right: 1px solid #99cdff;) - nachdem ich es hochgeladen und aktualisiert hatte, ist mir im Quelltext aufgefallen, dass aber gar keine Klasse angegeben ist:
Da steht nur li class="" - in der index.tpl von bulletproof steht dazu:

Code: Select all

<li class="{if $currpage==$navlink.href}currentpage{/if}{if $smarty.foreach.navbar.first} navlink_first{/if}{if $smarty.foreach.navbar.last} navlink_last{/if}">

Ist das so richtig? Wenn nicht wie bekomm ich das berichtigt?

[garvinhicking]

Hi!

Das ist richtig so. Nur für den ersten und letzten Link wird ein eigenes Class verwendet über dass Du das erste und letzte speziell ansprechen kannst. Ansonsten sind die li's über #sbsitenav li {...} ansprechbar.

Grüße,
Garvin

[Cohnina]

Oh gut.

Über #sbsitenav li {...} hatte ich es probiert - ich bilde mir auch ein, dass die einzelnen Links auseinander gerutscht sind - mal sehen wie ich meinen Rand da rein bekomme. Wahrscheinlich steht dann noch etwas anderes da, was entgegengesetzt wirkt

[yellowled]

Code: Select all

<li class="{if $currpage==$navlink.href}currentpage{/if}{if $smarty.foreach.navbar.first} navlink_first{/if}{if $smarty.foreach.navbar.last} navlink_last{/if}">

Ist das so richtig?

Naja - "richtig" ist diskussionswürdig Eigentlich ist es nicht richtig, dass hier im Fall, dass es sich weder um den ersten noch den letzten Navlink und auch nicht um die Startseite handelt, trotzdem eine (leere) class eingefügt wird. Da muss ich gelegentlich mal drüber brüten, aber wahrscheinlich ist der Code, den es bräuchte, um die leere Klasse zu unterdrücken, so lang, dass es vertretbarer ist, die leere Klasse drin zu lassen

YL

[frabron]

Böh, ich hab ja jetzt nicht wirklich so die Ahnung vom Templatecode da, aber könnte man nicht einfach

Code: Select all

<li {if $currpage==$navlink.href}class="currentpage"{/if}{if $smarty.foreach.navbar.first}class="navlink_first"{/if}{if $smarty.foreach.navbar.last} class="navlink_last"{/if}">

machen?

[yellowled]

Code: Select all

<li {if $currpage==$navlink.href}class="currentpage"{/if}{if $smarty.foreach.navbar.first}class="navlink_first"{/if}{if $smarty.foreach.navbar.last} class="navlink_last"{/if}">

Nicht wirklich. Wenn der betreffende Navigationspunkt gleichzeitig die aktuelle Seite und der erste oder letze Punkt im Menü ist, gibt's ein doppeltes class="..."

YL