-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Does anyone miss descriptive class names? (applies not just to emotion) #452
Comments
@spherop Try adding |
Thanks for reply. This is great that this capability is there -- But I must be doing something wrong.
.babelrc
|
I don't believe so. I think it was a very recent addition though, so might need to upgrade your version of the babel plugin. #442 https://github.com/emotion-js/emotion/blob/master/CHANGELOG.md#rocket-enhancement |
@evan-scott-zocdoc thanks so much - you were right - had to update. Now is working ...ahh such a relief to look in console at readable html hierarchy. Icing on the cake for me with react-emotion. Thanks! |
Glad you got it worked out @spherop. |
So, I am loving react-emotion! We're going down the path of switching to it on my team after reviewing the options.
But.. the only downside compared to vanilla css for me is that the resultant css class attrs no longer map easily to what they are in code.
Eq:
...
ends up as...
I understand the need for the unique ID. Wondering if anyone else would derive value from retaining original component name in the ID, eg:
or just
This would save me a lot of mental effort looking through and understanding code in chrome console. Perhaps this is just me... curious anyone else's thoughts.
[Of course this adds some extra data in the file - could be optional.]
The text was updated successfully, but these errors were encountered: