[c.....2]

#programowanie #javascript

Czy to duży wstyd?
Czy może nie głupie bo działa?
Wiem, że da się prościej innymi sposobami.

[Dezanhil]

@crassman22: oj byczku, użyj property .style zamiast zmieniać innerHTML

[taukuZielonka]

@crassman22 jeśli cos jest głupie i działa to nie jest głupie... Dopóki nie musisz wrócić do kodu po x lat i sam siebie nienawidzisz ;) Jeśli to jakiś g---o projekt który zginie za miesiąc to luz - ale jakbys mi PR z czymś takim wysłał, to byśmy mieli poważną rozmowę ;)

[kao3991]

@crassman22: nie tyle głupie, co pokazuje brak wiedzy autora, albo (co częste w wiekszych projektach) bardzo dziwna historię powstawania. Należałoby uprościć.

[Marmite]

@crassman22: słabe, znacznie lepszym byłoby albo użyć do tego różnych klas, albo użyć zmiennych css (a następnie modyfikować wartości zmiennych jsem)

[Marmite]

jeśli cos jest głupie i działa to nie jest głupie

@taukuZielonka: jeśli coś jest głupie i działa, to jest głupie i działa

[taukuZielonka]

@Marmite Czy przeczytałeś resztę mojego komentarza, czy zafiksowałeś się na jednym z najstarszych powiedzonek z branży? XD

[Marmite]

@taukuZielonka: zafiksowałem się na nim ponieważ strasznie irytuje mnie jego użycie w tej formie. IMO poprawna forma to "jeśli coś WYGLĄDA GŁUPIO, ale działa, to nie jest głupie". Jeśli coś jest głupie, ale działa... no to jest głupie ale działa ¯\_(ツ)_/¯

[C.....y]

@crassman22 o chuju złoty

[P.....y]

@crassman22: dios mio

[Chino]

@crassman22: (╯°□°）╯︵ ┻━┻

[LubieKiedy]

Prawda jest taka że każdy będzie narzekał, każdy kto by napisał inaczej to by się znalazła osoba która bedzir na to narzekać i że do poprawki i tak się żyje w tym IT ( ͡° ͜ʖ ͡°)

[a.....2]

@crassman22: a po co tak, skoro można ładniej, walnij style.backgroundColor

[milo77]

zostawiajac juz dyskusje z "jesli cos jest glupie, ale dziala", to czy jest jakis w miare cywilizowany sposob na takie podmianki cssa w locie? bardziej chodzi mi o podmianki w kontekscie stylowania calego dokumentu, nie pojedynczego obiektu

[c.....2]

@angie12: no to wiadomo, mi po prostu chodziło o reakcje ludzi na ten kod xD

[psposki]

@crassman22: changeColorCSS() - proszem
Czemu id nie przekazujesz jako parametr do funkcji? Byłoby uniwersalne

[Marmite]

@milo77: można podmieniać całe elementy style, albo podmieniać np. klasę na jakimś root elemencie, albo podmieniać wartości zmiennych cssowych. To wszystko zdaje się być całkiem cywilizowane

[milo77]

@Marmite: podmiana klasy root elementu brzmi sensownie

[Marmite]

@milo77: akurat ten sposób jest chyba najmniej elegancki z tych które mi przyszły do głowy, bowiem musisz potem pisać wszystkie klasy poprzedzając je tymi zmienianymi klasami... no chyba że jakoś to ogarniesz preprocesorem, idealnie tak żeby te standardowe klasy nie wiedziały w ogóle o istnieniu tych zmienialnych

[milo77]

@Marmite: niby tak, ale wydaje mi sie to lepsze niz szukanie w kodzie, gdzie sa podmieniane zmienne w cssie

[Marmite]

@milo77: jeśli to będzie zakopane gdzieś głęboko to tak, ale jeśli będzie do tego dedykowany service ThemeSwitcher czy też przekazywać je przez np. Context w React, no to chyba nie będzie tragedii