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;