-
Notifications
You must be signed in to change notification settings - Fork 20.5k
Make sure elem.ownerDocument.defaultView is not null #2867
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
Conversation
|
Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA). 📝 Please visit http://contribute.jquery.org/CLA/ to sign. After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know. If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check. |
src/css/var/getStyles.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Space after (.
|
Thanks! If we can figure out a unit test, that would be great. |
|
I like to see a test case added. http://jsbin.com/ruyecuxogu/edit?html,js,console,output would make a fine base ( |
|
@timmywil I don't know what else to do to resolve this CLA error. I did the changes in my pull request directly in Github. Then when I filled in the CLA form and got the error I realized that I didn't have full name in my Github account. Then I filled my full name in Github, but the CLA error still remains. Finally I followed the instructions to update the git config, i.e: and then: still no luck. |
|
@todorpr You have 2 commits in your PR and you amended only one. Can you squash them to one & try again? |
|
@mgol I squashed the two commits into one using this guide: http://stackoverflow.com/a/18690845 but now my CLA page show 4 commits as the first one remains where my name is different from the rest: I'm not sure what should I do? |
|
@todorpr You didn't squash it as the branch still contains a merge commit; this is how a Git tree looks like right now ( in a console to see the tree in text mode. It's best to avoid merges when working on something; you probably merged At this point it's easiest to run: which will remove all your commits but keep the changes. You can then commit the change again, just copy the commit description somewhere so that you can paste it. |
|
@mgol thanks, it's good now. Finally had to |

Since v2.2.0: Cannot read property 'opener' of null when $.parseHTML and then get css style #2866