script_selector()

Imate gre?ku u ovom delu koda.

U liniji 236 (case: 'list') za oba list items ste stavili %5$s, pa se za obe stavke generi?e klasa .active u slu?aju kada je prikaz na latinici. Osim toga, pretpostavljam da bi za neaktivno pismo trebalo da se dodaje klasa .hidden kako bi ta opcija bila skrivena. Shodno tome bi poslednja dva argumenta funkcije sprintf() trebalo da budu rotirana – kada je na sajtu aktivna latinica da bude prikazan selektor za ?irilicu.

Tako?e, kada se koristi opcija za prikaz selektora kao liste ('list') bilo bi mnogo bolje da se ne generi?e kao cela lista ve? samo kao list items. Ako neko koristi funkciju da bi ru?no dodao selektor na mesto koje mu odgovara lak?e je uokviriti generisane list items sa sopstvenim UL ili OL i odgovaraju?im klasama, nego ih izvu?i iz generisanog teksta da bi se uglavile u postoje?i UL ili OL.

Ili da postoji kao opcija 'list_items'.

Predlog ispravke:

case 'list':
    $return = sprintf(
        '<li class="rstr-script-selector-item%5$s"><a href="%1$s" class="rstr-script-selector-item-link%5$s">%2$s</a></li><li class="rstr-script-selector-item%6$s"><a href="%3$s" class="rstr-script-selector-item-link%6$s">%4$s</a></li>',
        $options->lat,
        $args->lat_caption,
        $options->cyr,
        $args->cyr_caption,
        ($options->active === 'cyr' ? ' active' : ' hidden'),
        ($options->active === 'lat' ? ' active' : ' hidden')
    );
    break;