theman550 / DAT257-K9

Project for course DAT257
1 stars 1 forks source link

Add reusable UI components based on mockup design #24

Closed AndreasWJ closed 4 years ago

AndreasWJ commented 4 years ago

Behöver feedback! @ITJohan Se Typography.jsx. I den filen är idéen att jag ska definiera H3, H4 genom att extenda min H2. Jag vet att man kan använda 'styled(H2)' vilket jag försökt med, men sedan vill jag att själva HTML taggen ska bli h3 respektive h4. Finns en styled-components funktion 'withComponent(h3) eller withComponent(h4)', finns ett kommenterat exempel i koden.

Hursomhelst, skriver 'withComponent()' över den style man får från 'styled(H2)', vilket gör den oanvändbar. Finns det någon sätt att lösa detta?

Har även lagt alla allmänna "reusable" komponenter i en UI mapp om det är ok? Läste en React artikel som rekommenderade det :)

AndreasWJ commented 4 years ago

Fixat testerna, och tog bort gamla kommentarer.

Kollade på 'as' property:n innan, problemet med den är att det inte blir självklart vad taggen åstadkommer. Kommer se ut som:

<H2 as="h2">Header</H2>

Dumt skrivet av styled-components tycker jag.