Title: script_selector()
Last modified: December 22, 2020

---

# script_selector()

 *  Resolved [cunnin6](https://wordpress.org/support/users/cunnin6/)
 * (@cunnin6)
 * [5 years, 5 months ago](https://wordpress.org/support/topic/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;
       ```
   

Viewing 1 replies (of 1 total)

 *  Plugin Author [Ivijan-Stefan Stipic](https://wordpress.org/support/users/ivijanstefan/)
 * (@ivijanstefan)
 * [5 years, 5 months ago](https://wordpress.org/support/topic/script_selector/#post-13822468)
 * Hvala na informaciji i kôdu.
 * Ispraviću ovaj bug odmah u sledećoj verziji.
 * Mnoge od ovih stvari se lepo mogu definisati kroz CSS pa se nisu dodavale dodatne
   CSS klase.
 * Ipak, dodaću CSS klasu `inactive` i `list_items` opciju da se ljudi mogu igrati
   sa time a ujedno ću dodati i filter pa svako može za sebe da menja kako želi.
 * Uskoro možete očekivati update.
 * Hvala!

Viewing 1 replies (of 1 total)

The topic ‘script_selector()’ is closed to new replies.

 * ![](https://ps.w.org/serbian-transliteration/assets/icon-256x256.png?rev=2374607)
 * [Transliterator – Multilingual and Multi-script Text Conversion](https://wordpress.org/plugins/serbian-transliteration/)
 * [Frequently Asked Questions](https://wordpress.org/plugins/serbian-transliteration/#faq)
 * [Support Threads](https://wordpress.org/support/plugin/serbian-transliteration/)
 * [Active Topics](https://wordpress.org/support/plugin/serbian-transliteration/active/)
 * [Unresolved Topics](https://wordpress.org/support/plugin/serbian-transliteration/unresolved/)
 * [Reviews](https://wordpress.org/support/plugin/serbian-transliteration/reviews/)

 * 1 reply
 * 2 participants
 * Last reply from: [Ivijan-Stefan Stipic](https://wordpress.org/support/users/ivijanstefan/)
 * Last activity: [5 years, 5 months ago](https://wordpress.org/support/topic/script_selector/#post-13822468)
 * Status: resolved