Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to styled-components #5

Merged
merged 1 commit into from Nov 23, 2018
Merged

Conversation

mxstbr
Copy link
Contributor

@mxstbr mxstbr commented Nov 22, 2018

:trollface: ?

(yes it works)

<S.Link target="_blank" rel="noopener" href="https://github.com/kitze/twizzy-landing">View source</S.Link>
<S.Link target="_blank" rel="noopener" href="https://github.com/kitze/twizzy-landing">
View source
</S.Link>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prettier auto-formatted

disableEsLint,
addBabelPlugin
} = require('customize-cra');
const { override, disableEsLint, addBabelPlugin } = require('customize-cra');
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prettier auto-formatted

@kitze
Copy link
Owner

kitze commented Nov 22, 2018

@mxstbr hmm I thought styled-components will fail because things like {height: 50} don't work in objects, you have to use {height: '50px'} Was that recently fixed? 🤔️

@mxstbr
Copy link
Contributor Author

mxstbr commented Nov 22, 2018

Yep! We added support for that recently. Object styles are now a first-class citizen!

@kitze kitze merged commit 168631b into kitze:master Nov 23, 2018
@kitze
Copy link
Owner

kitze commented Nov 23, 2018

Merged. Dammit @mxstbr your marketing game is 💯️

I went from aphrodite -> styled-components -> glamorous -> emotion -> styled-components.

@mxstbr mxstbr deleted the styled-components branch November 23, 2018 07:17
@Andarist
Copy link

Andarist commented Nov 23, 2018

This is kinda a weird PR to me - maybe because I'm more involved in developing emotion but I love both projects.

It's weird to me because nothing was really added, no argument was given in favour of the change. Seems like a marketing/power grab move to me, which doesn't sound nice. I don't want to accuse anyone of anything, but this is really strange to me.

That being said ofc anyone is free to use any solution they want 😉

If this was intended more as a joke (not necessarily a bad one, it only shows how both projects are interoperable ❤️ ) then it's fine. Maybe Im just weird that it has surprised me 😅

@mxstbr
Copy link
Contributor Author

mxstbr commented Nov 23, 2018

Definitely a joke :trollface: (I didn't think @kitze would actually merge this PR)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants