[Trusiak]

Pierwszy dzień w nowej robocie jako junior js developer.
Senior zrobił mi code review na konwersacji z szefostwem.
Zapowiada się elegancka atmosfera pracy (⌐ ͡■ ͜ʖ ͡■)

#programowanie #javascript #frontend #heheszki

[Gaussik]

@Trusiak: jestem tym seniorem, usuń to ( ͡° ͜ʖ ͡°)

[Phatee]

@Trusiak: jak sie im ify nie podobają to zrób na switch case ( ͡° ͜ʖ ͡°)

[Passer93]

@Phatee: ale wiesz że tu nie chodzi o if-a tylko o iife? xD

[MarcinOrlowski]

@Trusiak: powiedz ze to nie fejk ze w nowej robocie jakis pajac napisał Ci w code review "co to #!$%@? za kulfon"?!?!?!

[Rudy_]

@MarcinOrlowski
@Passer93
@Trusiak

Swoja droga, co złego jest w takim rozwiązaniu? Działa to działa, na cholernie sie dziad pruje? ᕦ(òóˇ)ᕤ

[MarcinOrlowski]

@Rudy_: nic. to błazen z rozdmuchaym ego. Gimbo-odpowiedź "twoja stara" jak najbardziej na miejscu.

[Passer93]

@Rudy_: Nie wiem, moze mają jakaś określoną konwencje. A swoja drogą nie wiedziałem że fb Messenger to platforma do robienia code review xD

[schriker]

Code review kodu do ukrywania menu XD

[tmtmp3]

@Trusiak: ktos seniorowy placi za taki syf, code review funkcji do ukrycia menu? tego sie po prostu nie przekleja? Czemu ktos powazny sobie na to czas traci?

[WaveCreator]

@tmtmp3: W cywilizowanych firmach reviewuje się wszystko, nawet podbicie wersji zależności czy update readme.
Ale - w cywilizowanych firmach nie robi się review w komunikatorze xD

[MarcinOrlowski]

@tmtmp3: code review jest zawsze do PR. Nie pajacuj jak nie masz pojęcia.

[S.....8]

@Trusiak: Ale czemu tak zrobiles?

[Trusiak]

@Sheppard28: żeby mieć pełną kontrole nad niezależnym komponentem, bez wywoływania jego nowej instancji.

[WaveCreator]

@Trusiak: A co powiedział senior jak mu to wytłumaczyłeś? ( ͡° ͜ʖ ͡°)

[doee]

@Trusiak: Dlaczego on Ci robi CR w messengerze? XD Nie macie Githuba/Gitlaba/Bitbucketa? Albo, chociażby Slacka? BTW mam nadzieję, że przeprowadziliście po tym jakąś merytoryczną rozmowę na temat tego rozwiązania, a nie tylko takie kwiatki, bo jak nie to raczej nie masz do czynienia z seniorem XD

[Rebeliant]

@Trusiak: odpisz że kulfon to na jego mordzie ( ͡° ͜ʖ ͡°)

[Deykun]

@Trusiak:
Funkcje do zmieniania opacity przekombinowałeś, bo jakość, a to że ta funkcja się wywołuje bez debounce po 50 razy na scrolla ci umknęło. (✌ ﾟ ∀ ﾟ)☞

[WaveCreator]

raczej nie masz do czynienia z seniorem XD

@doee: Może jakieś Netguru, tam robią z juniora na seniora w 2 lata ( ͡° ͜ʖ ͡°)

debounce

@Deykun: o zoba, człowiek się całe życie uczy.

[mistrzus]

@Trusiak: