• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Javascript - biblioteka slider

VPS Starter Arubacloud
0 głosów
303 wizyt
pytanie zadane 16 września 2017 w JavaScript przez zgrybus Pasjonat (24,860 p.)
Stworzyłem bibliotekę do sliderów. Oczywiście mam zamiar ją troszkę rozwinąć i napisać kilka testów w Jasmine. Co myślicie? Jest jakiś duży babol w kodzie?

https://github.com/zgrybus/papa-slider

Główny i najważniejszy kod jest tutaj: https://github.com/zgrybus/papa-slider/blob/master/src/js/papa-slider.js

1 odpowiedź

0 głosów
odpowiedź 17 września 2017 przez ShiroUmizake Nałogowiec (46,300 p.)

Po pierwsze to mi się nie podoba:

   document.addEventListener("DOMContentLoaded", function(event) {
        PapaSlide.setLibrary({
            /*
                * string: elementClass for the container element; default: 'papa-container'
             */ 
            'container': 'papa-container',
            /*
                * string: elementClass for the items element; default: 'papa-item'
             */ 
            'items': 'papa-item',
            /*
                * string: transition-duration css property in miliseconds; default: '300'
             */ 
            'transitionDuration': '300',
            /*
                * string: transition-timing-function css property; default: 'ease-in'
             */ 
            'transitionFunction': 'ease-in',
            /*
                * string: interval between each element, in milliseconds; default: 3000
             */ 
            'timeInterval': '3000',
            /*
                * string: animation type ( more in table ); default: 'fade'
             */ 
            'animationType': 'fade',
            /*
                * string: switching slides ( more in table ); default: 'auto'
             */ 
            'type': 'auto',
            /*
                * number: first index element; default: 0
             */ 
            'startAt': 0,
        });
    })

Niesądzisz, że trochę za dużo argumentów?

Po drugie, co jeżeli chce więcej elementów animowanych? Do każdej mam pisać to?

Lepiej ustawić set do obiektu, najwyżej tam sprawdzać instanceof. Po drugie, czy nie sądzisz, że niektóre z nich mogą być domyślne. 

 .your-container-className {
        position: relative;
    }
    .your-items-className {
        position: absolute;
        top: 0;
        right: 0;
        top: 0;
        bottom: 0;
        opacity: 0;
    }

Pamiętaj, że akrusz CSS też możesz przypiąć do klienta po przez JS. Tylko trzeba założyć jakiś boilerplate. Ale to tak zostawiam jako ciekawostkę. 

Za to w kodzie masz o wiele przyjemniejszą implementację:

fadeAuto.setPapaSlider({container: 'fade-auto', timeInterval: '1000'});

Brak obsługi wyjątków. 

Nie wiem kto ci kazał tak pisać.-> _nazwaMetody ale nieprzyjemnie się to czyta. 

komentarz 17 września 2017 przez zgrybus Pasjonat (24,860 p.)
> Niesądzisz, że trochę za dużo argumentów?

Usunąć przykładowo wartości typu startAt, timingFunction a klasę containerowi i items ustawić z góry i wymagać od użytkownika użycia ich? Btw. są tam domyślne wartośći, użytkownik możę je po prostu nadpisać ( jest na dole tabelka ).

> Po drugie, co jeżeli chce więcej elementów animowanych? Do każdej mam pisać to? Lepiej ustawić set do obiektu, najwyżej tam sprawdzać instanceof.

No właśnie z tym miałem problem. Chodzi Ci o ten Set? https://developer.mozilla.org/pl/docs/Web/JavaScript/Referencje/Obiekty/Set

>Pamiętaj, że akrusz CSS też możesz przypiąć do klienta po przez JS.

Wiem wiem, ale jeśli ktoś ma inną nazwę klasy to co mi z tych styli? Chyba, że rezygnować z nadpisywania klasy container i items.

> Po drugie, czy nie sądzisz, że niektóre z nich mogą być domyślne.

Wszystkie mają swoje wartośći domyślne, po prostu można je nadpisać.

> Brak obsługi wyjątków.

Tzn. jakich dokładnie? Chodzi Ci, gdy ktoś doda jakąś 'dziwną' wartość?

> Nie wiem kto ci kazał tak pisać.-> _nazwaMetody ale nieprzyjemnie się to czyta.

Zmiennym prywatnym dodaje _. W sumie chyba porzucę ten zamysł, ponieważ samemu mi się nie podoba.
komentarz 17 września 2017 przez ShiroUmizake Nałogowiec (46,300 p.)
>Usunąć przykładowo wartości typu startAt, timingFunction a klasę containerowi i items ustawić z góry i wymagać od użytkownika użycia ich? Btw. są tam domyślne wartośći, użytkownik możę je po prostu nadpisać ( jest na dole tabelka ).

Przykładowo, te które są istotne, mogą się róznić zależności od użycia.

 

>No właśnie z tym miałem problem. Chodzi Ci o ten Set? https://developer.mozilla.org/pl/docs/Web/JavaScript/Referencje/Obiekty/Set

Tak. Możesz stworzyć jakby jednego menagera aplikacji która przejmuje pewną listę(kolejkę), a potem na każdym elemencie wykonuje jakieś czyności.

>Wiem wiem, ale jeśli ktoś ma inną nazwę klasy to co mi z tych styli? Chyba, że rezygnować z nadpisywania klasy container i items.

Możesz ustawić important i już nie nadpisze :). A pozwolić edytować plik np: SASS gdzie są wszystkie wartości.

>zn. jakich dokładnie? Chodzi Ci, gdy ktoś doda jakąś 'dziwną' wartość? Exactly.
komentarz 17 września 2017 przez zgrybus Pasjonat (24,860 p.)
Dzięki wielkie. W wolnym czasie postaram się pozmieniać kilka rzeczy. Dzięki za radę :)

Podobne pytania

+1 głos
1 odpowiedź 432 wizyt
0 głosów
1 odpowiedź 664 wizyt
pytanie zadane 3 sierpnia 2019 w JavaScript przez cotyniepowiesz Nowicjusz (120 p.)
0 głosów
2 odpowiedzi 469 wizyt
pytanie zadane 17 stycznia 2019 w JavaScript przez Luckyluck87 Użytkownik (660 p.)

92,452 zapytań

141,262 odpowiedzi

319,085 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...