Układy artykułu - prośba o ocenę.

pytanie zadane 14 kwietnia 2018 w Nasze projekty przez Crocoderka Początkujący (270 p.)

To mój pierwszy post tutaj, więc witam Was wszystkich. :)
Uczę się kodowania na własną rękę od około roku i myślę, że to już jest czas na dzielenie się swoimi projektami, żeby bardziej doświadczeni pokazali moje błędy w kodzie.

Tak więc wczoraj zakodowałam projekt przykładowych układów artykułu na stronie i proszę o ocenę - szczególnie zależy mi na ocenie kodu js, bo mam wrażenie, że chyba powinien być inaczej napisany (głównie fragment, który przełącza artykuły).
Starałam się pisać w BEM na tyle na ile to rozumiem.
Design jest autorski, więc również można go oceniać. :)

1	komentarz 14 kwietnia 2018 przez Mariusz08 Maniak (62,300 p.) Arrow Function

komentarz 14 kwietnia 2018 przez k.wichura Pasjonat (19,870 p.)
edycja 14 kwietnia 2018 przez k.wichura

const jest zawsze lepsze od let, != to nie to samo co !==. Staraj sie uzywac tego drugiego i doczytaj dlaczego. Używasz let, a potem var - czemu ?

'.article--' + type_attr; => `.article--${type_attr}` - es6

Nie da rady tego kodu podzielić na funkcje ?

komentarz 14 kwietnia 2018 przez Crocoderka Początkujący (270 p.)

Ok, dzięki, zmieniłam zmienne na const tam, gdzie była taka możliwość, ścisłe porównanie, template literals i funkcje strzałkowe też dodane. Co do var to zwykłe przeoczenie, zrobiłam sobie dłuższą przerwę od js i jak widać zapomniałam o es6.. ;)

Pewnie się da ten kod podzielić, liczyłam na jakąś podpowiedź, bo mam mało praktyki w js, ale tak czy inaczej będę nad tym myśleć.

komentarz 14 kwietnia 2018 przez k.wichura Pasjonat (19,870 p.)
edycja 14 kwietnia 2018 przez k.wichura

item_active i inne chyba nie musza byc zmiennymi globalnymi

Co do funkcji ja nie jestem przekonany tutaj do funkcji strzalkowych, bo po co mamy przypisywac ja do zmiennej, ale np w eventlistener wydzielibym funkcje i przekazywal ja do eventlistenera.

ps. kod jest spoko, ale chciales review to robie :)