[Melcma]

#webdev @rbielawski

Sprawa jest taka:
Dostałem zadanie rekrutacyjne PSD TO HTML samo zakodowanie layoutu i pierwszej strony głównej. Skończyłem i ogólnie jestem zadowolony z efektu.
Ładnie wyświetla się na 5 różnych (updateowanych) przeglądarkach.
RWD jest
Na githubie co skończylem to commitowałem

Teraz chciałbym żebyście ocenili, wskazali jak moge poprawić kod który napisałem (html/css/js/jquery), gdzie użyłem złych praktych, jak to zrobić dobrze, czy kod jest czytelny, dobrze skomentowany itd, nie ukrywam, że mi mocno na tym zależy.

http://rugbywebdev.co.uk/propeller/
https://github.com/melcma/Propeller-Front-end-Test

[npsr]

@Melcma: nie podpisuj z nimi umowy

p.s. robota dobra

[A.....W]

@Melcma: birthday - u mnie w input cyfr nie widać - są białe

[Melcma]

@AVAW: jaka przeglądarka?

[A.....W]

@Melcma: chrome ofc v 47.0.2526.106 m

[pitu-pr]

@Melcma: Chrome rwd u mnie https://prtscn.in/xweLa

[epi]

@Melcma:

co to za fix? :)

[Melcma]

@epi: sprawa jest taka, że safari dziwnie czyta spacje/taby dla inline-block i bez tych strzałek robi odstepy pomiedzy obrazkami, co przy width 25% oczywiscie rozwala mi koncepcje i przenosi ostatnie zdjecie do nastepnej linijki. Te strzałki to taki trick który usuwa te puste spacje przy okazji nie psujac innych przegladarek. Probowalem innych trickow ale ten wydaje mi sie najlepszy.

[llorT]

@Melcma: Ile łącznie czasu poświęciłeś na to zadanie?

[Eggy]

@Melcma: commitujesz przetranspilowany css? serio?

[Melcma]

@llorT: mniej niż 25 godzin, ciężko mi ocenić

[incognito_man]

@Melcma: nie wiem czy wiesz ale to wygląda jakby szukali frajera, 25h pracy ? zadanie rekrutacyjne to max 2h a nie 3 dni robocze

[Buda]

@Melcma: PRzy zmniejszeniu rozdzielczość, menu zmienia się w przycisk, natomiast po jej zwiększeniu menu calkowicie znika.

[Melcma]

@incognito_man: w sumie moglo tak byc, praca przez rekrutera, widzialem ze kilka osob wykonywalo to zadanie i oni mają ogólnie taki test, dokladnie ta sama strona, grafika ta sama, ale wg mnie lepiej wykonalem to od innych, wydaje sie legitne

[xcccx]

safari dziwnie czyta spacje/taby dla inline-block i bez tych strzałek robi odstepy

@Melcma: font-size: 0 na rodzicu nie załatwia sprawy?

[Melcma]

@xcccx: probowalem, font-size: 0, content: " ", margin ujemny dzialal ale robil inne problemy, jakies inne tricki

[isamaul]

@xcccx: mi zawsze działa z font-size 0 :P

[vibro_vagins]

@Melcma:
@epi:
@xcccx:
Nie tylko safari to boli. To wyjście jest okej, można również font-size 0 dać na parent albo kolejne wyjście - trzymać te elementy w linii, tzn elementelementelement zamiast
Element
Element
Element

[Melcma]

@vibro_vagins: tak to w linii dziala tez, ale chyba strzalki lepiej wygladają, poprobuje jeszcze z font-size: 0

[xcccx]

@Melcma: na wszystkie przeglądarki poza safari fs:0 daje radę, ale nie mam safari żeby sprawdzić z ciekawości dokładnie.

[Melcma]

@xcccx: na wszystkie przegladarki to float: left rozwiazuje sprawe, tu sie wlasnie o safari rozchodzi. Daje font-size: 0 na wszystko co sie da, 0 efektu,