mathias-rosa / MyDadJoke

0 stars 0 forks source link

inline styles #2

Open everythinginjs opened 3 weeks ago

everythinginjs commented 3 weeks ago

Are you aware of the inline styles in you code ? like this

mathias-rosa commented 3 weeks ago

Yes, actually because I didn't kwew if it was worth it creating a new styled component for just one div. I think the best options is by using material ui layouts but I'm not sure. Are inlines styles worse that an external css file? For img elements I don't know what is the better option.

everythinginjs commented 3 weeks ago

Good, the point is being consistent with efficient solution. it's not about just one div in large scale projects. but the point is Inline styles increase the HTML file size, which can affect the page load time you can go with emotion solution in MUI and make a styled component for it EZ.