Skip to content
This repository has been archived by the owner on Oct 21, 2022. It is now read-only.

Force include selector with camelCase #27

Closed
saunders1989 opened this issue Jun 11, 2015 · 10 comments
Closed

Force include selector with camelCase #27

saunders1989 opened this issue Jun 11, 2015 · 10 comments

Comments

@saunders1989
Copy link

Hi,

I found an issue when you have a selector like #pbOverlay which is being used by a plugin (I dont agree with the naming convention) that if you try and forceInclude this selector it doesn't include it in the generated css

is there a work around other than changing my css selector / adding these selectors in manually?

@jefflembeck
Copy link
Collaborator

@saunders1989 in the output, is #pboverlay in there?

@saunders1989
Copy link
Author

@jefflembeck its not in the output CSS file but is in the source CSS file I pass in

@jefflembeck
Copy link
Collaborator

@saunders1989 so, no version of it shows up in the output css at all?

@saunders1989
Copy link
Author

@jefflembeck unfortunately not.

@jefflembeck
Copy link
Collaborator

@saunders1989 any chance you can make a small test case and put it up somewhere? it'd really help me fix it.

@saunders1989
Copy link
Author

@jefflembeck sure I will do that tomorrow for you.

@jefflembeck
Copy link
Collaborator

@saunders1989 awesome, thank you!

@saunders1989
Copy link
Author

@jefflembeck Hi,

I have created a test case on my github - https://github.com/saunders1989/camelcase-test

When i did the test case the camel case selector did get added to the critical css file that is generated but it the selector is now all lowercase

@jefflembeck
Copy link
Collaborator

@saunders1989 ah, this is what I expected :(

This is a known issue, but we're trying to figure out a solution (it has to do with html not being case sensitive)

@saunders1989
Copy link
Author

@jefflembeck ahh okay. sorry for creating a duplicate issue. I will have to debug my more complicated grunt file / css and find out why it doesnt get included even all lowercase

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

No branches or pull requests

3 participants