HTML/PHP - kolor przycisku menu na danej stronie

@edit
boże, to miało iść do programowania, przenieście jak coś xd sry ale 5:20

A więc mam sobie menu na stronie
https://i.imgur.com/cRMy8iF.png
Jak widać mam zrobione żeby w zależności na której jestem stronie pokazywało że dany "przycisk" jest aktywny, obecnie mam to zrobione tak :

Jest jakiś sposób żeby to ogarnąć w bardziej czytelny, łatwiejszy do późniejszych edycji sposób? Dokładniej żeby w zależności od ?pid= pojawiało się w odpowiednim miejscu class=active...

obecnie mam to tak zrobione, strasznie to wygląda XD

Kod:

---

<?if (!(isset($_GET['pid']))){
        echo"
        <li class='active'><a href='http://tibia24.com'>Kup Konto</a></li>
                <li><a href='?pid=2'>SMS</a></li>
                <li><a href='?pid=3'>PRZELEW</a></li>
                <li ><a href='?pid=6'>Paczki Kont</a></li>
                <li><a href='?pid=4'>Regulamin</a></li>
                <li><a href='mailto:[email protected]'>Kontakt</a></li>";
        }else{
        if ($_GET['pid']=='2'){
                echo"
        <li ><a href='http://tibia24.com'>Kup Konto</a></li>
                <li class='active'><a href='?pid=2'>SMS</a></li>
                <li><a href='?pid=3'> PRZELEW</a></li>
                <li ><a href='?pid=6'>Paczki Kont</a></li>
                <li><a href='?pid=4'>Regulamin</a></li>
                <li><a href='mailto:[email protected]'>Kontakt</a></li>";
        }elseif ($_GET['pid']=='3'){
                echo"
        <li ><a href='http://tibia24.com'>Kup Konto</a></li>
                <li><a href='?pid=2'>SMS</a></li>
                <li class='active'><a href='?pid=3'>PRZELEW</a></li>
                <li ><a href='?pid=6'>Paczki Kont</a></li>
                <li><a href='?pid=4'>Regulamin</a></li>
                <li><a href='mailto:[email protected]'>Kontakt</a></li>";
        }elseif ($_GET['pid']=='4'){
                echo"
        <li ><a href='http://tibia24.com'>Kup Konto</a></li>
                <li><a href='?pid=2'>SMS</a></li>
                <li ><a href='?pid=3'>PRZELEW</a></li>
                <li ><a href='?pid=6'>Paczki Kont</a></li>
                <li class='active'><a href='?pid=4'>Regulamin</a></li>
                <li><a href='mailto:[email protected]'>Kontakt</a></li>";
        }elseif ($_GET['pid']=='5'){
                echo"
        <li ><a href='http://tibia24.com'>Kup Konto</a></li>
                <li><a href='?pid=2'>SMS</a></li>
                <li ><a href='?pid=3'>PRZELEW</a></li>
                <li ><a href='?pid=6'>Paczki Kont</a></li>
                <li ><a href='?pid=4'>Regulamin</a></li>
                <li class='active'><a href='mailto:[email protected]'>Kontakt</a></li>";
        }elseif ($_GET['pid']=='6'){
                echo"
        <li ><a href='http://tibia24.com'>Kup Konto</a></li>
                <li><a href='?pid=2'>SMS</a></li>
                <li ><a href='?pid=3'>PRZELEW</a></li>
                <li class='active' ><a href='?pid=6'>Paczki Kont</a></li>
                <li ><a href='?pid=4'>Regulamin</a></li>
                <li><a href='mailto:[email protected]'>Kontakt</a></li>";
        }else{
                echo"
        <li ><a href='http://tibia24.com'>Kup Konto</a></li>
                <li><a href='?pid=2'>SMS</a></li>
                <li ><a href='?pid=3'>PRZELEW</a></li>
                <li ><a href='?pid=6'>Paczki Kont</a></li>
                <li ><a href='?pid=4'>Regulamin</a></li>
                <li ><a href='mailto:[email protected]'>Kontakt</a></li>";
       
       
        };
       
        };?>

---

chyba najlepiej będzie ci jsem zmienić, byle wysłać jakoś odpowiednie id do skryptu (jak masz w URI to po prostu wywlecz z niego)

Spróbuj jquery, coś w tym stylu

Kod:

---

<script src="http://code.jquery.com/jquery-latest.js"></script>
<ul class="navigation">
<li class="active"><a href="index.html">Index</a></li>
<li><a href="#">Link</a></li>
<li><a href="#">Link</a>
<ul>
<li><a href="#">Sublink</a></li>
<li><a href="#">Sublink</a></li></ul>
</li>
<li><a href="#">Link</a></li>
</ul>

---

Kod:

---

$(document).ready(function(){
var str=location.href.toLowerCase();
$(".navigation li a").each(function() {
if (str.indexOf(this.href.toLowerCase()) > -1) {
 $("li.active").removeClass("active");
$(this).parent().addClass("active");
}
 });
$("li.active"').parents().each(function(){
if ($(this).is("li")){
$(this).addClass("active");
}
});
 })

---

Kod PHP:

---

<li class="<?php echo (!isset($_GET['pid'])) ? 'active' : ''; ?>"><a href='http://tibia24.com'>Kup Konto</a></li> <li class="<?php echo ((int)$_GET['pid'] == 2) ? 'active' : ''; ?>"><a href='?pid=2'>SMS</a></li> <li class="<?php echo ((int)$_GET['pid'] == 3) ? 'active' : ''; ?>"><a href='?pid=3'>PRZELEW</a></li> <li class="<?php echo ((int)$_GET['pid'] == 6) ? 'active' : ''; ?>"><a href='?pid=6'>Paczki Kont</a></li> <li class="<?php echo ((int)$_GET['pid'] == 4) ? 'active' : ''; ?>"><a href='?pid=4'>Regulamin</a></li> <li class=""><a href='mailto:[email protected]'>Kontakt</a></li>

---

Hardcore pomysł żeby to robić JSem ;o

@edit
W sumie jak przeczytałem teraz kod JS Decretisa, to faktycznie ma to sens, nie musi wpisywać warunków nigdzie. W wiekszej stronie, oczywiscie menu byloby generowane a nie wpisane z palca, wiec wtedy podczas petli sprawdzenie czy id aktualnie wyswietlanej strony == id linku, i wtedy jest automatycznie i bez JS.

Nie wierze, ze ktos jeszcze w taki sposob pisze strony.

@topic
Uzyj jquery, jak Pan up zaproponowal.

@Bazan ; jasne, że można, ale to wygląda dość paskudnie i kodu jest 5x więcej niż w js, a z każdym dodatkowym elementem tylko rośnie

if ($(this).attr("href") == window.location.pathname) {
$(this).addClass("active");
}

i zamykasz sie w jednej prostym bloku

query string tez mozna, tylko potrzebuejsz zrobic jakis getter, np.

Kod:

---

class Functions {

    static getQueryString(variable) {
        var query = window.location.search.substring(1);
        var vars = query.split("&");
        for (var i = 0; i < vars.length; i++) {
            var pair = vars[i].split("=");
            if (pair[0] == variable) { return pair[1]; }
        }
        return (false);
    }
}

---

Dzięki panowie =D

Cytuj:

---

szulak napisał

Nie wierze, ze ktos jeszcze w taki sposob pisze strony.

---

ojtam ojtam, ja prosty chłopak jestem XD