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

Kod do oceny - Menu

Object Storage Arubacloud
0 głosów
222 wizyt
pytanie zadane 23 lipca 2017 w Rozwój zawodowy, nauka, praca przez rafal1997 Gaduła (3,180 p.)
Witam, chciałbym aby ktoś kto zna się na tworzeniu stron internetowych ocenił mi menu pod względem poprawności kodu itp

https://github.com/Ersum/Menu

3 odpowiedzi

+1 głos
odpowiedź 23 lipca 2017 przez Tomek Sochacki Ekspert (227,510 p.)
  1. W instrukcji IF stosuj klamerki {} co nie tylko zwiększy czytelność ale i zabezpieczy Cię przed możliwymi błędami, jeśli za chwilę z jakiś powodów zechcesz dać w IF drugą instrukcję. To samo w else.
  2. Proponuję używać jakiegoś narzędzia do analizy kodu, np. ESLint ponieważ raz stosujesz konwencję spacji w nawiasach, a raz nie (lepiej to ujednolicić).
0 głosów
odpowiedź 23 lipca 2017 przez ProgramistaStepek Nałogowiec (27,020 p.)

Menu jak menu. Wygląda nieźle, zastosowałeś responsywność. Osobiście poprawił bym dwie rzeczy.

Po pierwsze wstawianie ikonek. Nie powinno się do tego używać znacznika <i> Link do artykułu na ten temat.

Po drugie nie wiem czemu to zapisałeś w js zamiast użyć media queries w css.

$( window ).resize(function() {
  if ($(window).width() > 768) $(".menu__nav").css("display","flex");
    else $(".menu__nav").css("display","none");
});

 

komentarz 23 lipca 2017 przez rafal1997 Gaduła (3,180 p.)
Dzięki za odpowiedź, a z tym js to sam nwm czemu tak zrobiłem :)
komentarz 23 lipca 2017 przez rafal1997 Gaduła (3,180 p.)
a no już wiem bo jeśli zrobiłbym to za pomocą css to przy sprawdzaniu w przeglądarce czy menu jest responsywne nie działało by tak ja trzeba. Jak się zmniejszy okno i się kliknie w ikonę otwierającą menu i się powiększy okno to zostaje w menu__nav to flex-direction:column; i trzeba odświeżyć żeby to działało
0 głosów
odpowiedź 23 lipca 2017 przez xmentor Nałogowiec (49,520 p.)

 

Podobne pytania

0 głosów
1 odpowiedź 214 wizyt
pytanie zadane 4 sierpnia 2017 w JavaScript przez turtelian Obywatel (1,760 p.)
0 głosów
0 odpowiedzi 354 wizyt
pytanie zadane 23 lutego 2017 w C# przez marcin99b Szeryf (82,080 p.)
0 głosów
1 odpowiedź 226 wizyt

92,555 zapytań

141,402 odpowiedzi

319,552 komentarzy

61,939 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

Kolejna edycja największej imprezy hakerskiej w Polsce, czyli Mega Sekurak Hacking Party odbędzie się już 20 maja 2024r. Z tej okazji mamy dla Was kod: pasjamshp - jeżeli wpiszecie go w koszyku, to wówczas otrzymacie 40% zniżki na bilet w wersji standard!

Więcej informacji na temat imprezy 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!

...