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

Kod do oceny - Menu

VPS Starter Arubacloud
0 głosów
214 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ź 211 wizyt
pytanie zadane 4 sierpnia 2017 w JavaScript przez turtelian Obywatel (1,760 p.)
0 głosów
0 odpowiedzi 345 wizyt
pytanie zadane 23 lutego 2017 w C# przez marcin99b Szeryf (81,480 p.)
0 głosów
1 odpowiedź 220 wizyt

92,453 zapytań

141,262 odpowiedzi

319,087 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!

...