Skip to content

Conversation

erikruth
Copy link
Contributor

I am confused about when you use single click and when you use SingleClick. In particular, single click functionality vs. SingleClick functionality. Please review with your team and make changes to be consistent.
Please also check the inline comment I added for technical accuracy.

erikruth and others added 2 commits December 12, 2015 14:03
I am confused about when you use single click and when you use SingleClick. In particular, single click functionality vs. SingleClick functionality.  Please review with your team and make changes to be consistent.
Please also check the inline comment I added for technical accuracy.
Erik, technically, the sentence is OK.
@Iankodj
Copy link
Contributor

Iankodj commented Dec 14, 2015

Erik, I checked the inline comment. It is OK, so I removed it.

As for single click, using SingleClick functionality results to a single click button. I do not find references for "single click functionality" in this article. Only "single click button" reference in the introduction. Let me know if I am missing something that needs improvement.

@erikruth
Copy link
Contributor Author

You can find the text single click functionality in the caption for figure 1. Please review whether to use single click (two words) or SingleClick (one word, proper case) throughout the article and then change the figure caption so it is consistent with the rest of the article. Thanks!

From: Ianko Djemerenov [mailto:notifications@github.com]
Sent: Monday, December 14, 2015 12:53 AM
To: telerik/ajax-docs ajax-docs@noreply.github.com
Cc: Erik Ruthruff erik@a23consulting.com
Subject: Re: [ajax-docs] Update single-click.md (#68)

Erik, I checked the inline comment. It is OK, so I removed it.

As for single click, using SingleClick functionality results to a single click button. I do not find references for "single click functionality" in this article. Only "single click button" reference in the introduction. Let me know if I am missing something that needs improvement.


Reply to this email directly or view it on GitHubhttps://github.com//pull/68#issuecomment-164360493.

@Iankodj
Copy link
Contributor

Iankodj commented Dec 15, 2015

Thanks for pointing it out.

Iankodj added a commit that referenced this pull request Dec 15, 2015
@Iankodj Iankodj merged commit d89c71d into ImageButton Dec 15, 2015
@Iankodj Iankodj deleted the erikruth-patch-4 branch December 15, 2015 13:57
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