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.

Sign up for GitHub

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

Jump to bottom

101 xxlovett0706 #24

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

101 xxlovett0706 #24

wants to merge 2 commits into from

Conversation

xxlovett0706
Copy link

No description provided.

@xxlovett0706 xxlovett0706 changed the title Xxlovett0706 Xxlovett0706 101 Nov 19, 2015
@xxlovett0706 xxlovett0706 changed the title Xxlovett0706 101 Xxlovett0706 Nov 19, 2015
@xxlovett0706 xxlovett0706 changed the title Xxlovett0706 101 xxlovett0706 Nov 19, 2015
@shaohua shaohua added the 101 label Nov 19, 2015
shaohua
shaohua reviewed Nov 19, 2015
View reviewed changes
101-clickable-grid/101.html

CreateCell.prototype.init = function(){
//创建DOM元素
var box = $('<div class="box"></div>');
Copy link
Contributor

Choose a reason for hiding this comment

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

考虑使用$box

Copy link
Author

Choose a reason for hiding this comment

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

这个$box指的是什么意思？

Copy link
Author

Choose a reason for hiding this comment

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

想起来了，我说之前为什么不对，我创建变量是var num = ....，应该创建jQuery变量 var $num = ... 这样

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
101
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xxlovett0706 @shaohua