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

Minor css fix #7

Closed
wants to merge 1 commit into from
Closed

Conversation

Mitko-Kerezov
Copy link
Contributor

Some CSS linting tools (like the one used in AppBuilder desktop client) display errors with the given syntax
ping @tjvantoll

@tjvantoll
Copy link
Contributor

@Mitko-Kerezov Unfortunately this updated syntax doesn't work, even though it seems like it should. I'll create an issue for in the modules repo.

@tjvantoll
Copy link
Contributor

Requested: NativeScript/NativeScript#612

@Mitko-Kerezov
Copy link
Contributor Author

@tjvantoll My bad, I didn't realize it doesn't actually work.
I've tweaked the PR so that the syntax works as well as the linting tools don't complain

@tjvantoll tjvantoll closed this in d281c2b Aug 24, 2015
@tjvantoll
Copy link
Contributor

@Mitko-Kerezov No worries, and thanks again. Merged.

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.

2 participants