search

cfjedimaster / ColdFusion-UI-the-Right-Way

A list of articles demonstrating how to do UI with ColdFusion on the back end.
100 stars 31 forks source link

document ready #47

Closed PhillipSenn closed 10 years ago

PhillipSenn commented 10 years ago

Since you've put jquery.js at the bottom in your cfajaxproxy example, then there's no need for document ready.

This would simplify the example just a tad.

cfjquery commented 10 years ago

I didn't write the cfajaxproxy example.

On Tue, May 20, 2014 at 7:51 PM, Phillip Senn notifications@github.comwrote:

Since you've put jquery.js at the bottom in your cfajaxproxy example, then there's no need for document ready.

This would simplify the example just a tad.

— Reply to this email directly or view it on GitHubhttps://github.com/cfjedimaster/ColdFusion-UI-the-Right-Way/issues/47 .

Regards,

Dave White

cfjedimaster commented 10 years ago

You may have gotten this email because you are signed up for Notifications, Dave.

On Tue, May 20, 2014 at 2:35 PM, Dave White notifications@github.comwrote:

I didn't write the cfajaxproxy example.

On Tue, May 20, 2014 at 7:51 PM, Phillip Senn notifications@github.comwrote:

Since you've put jquery.js at the bottom in your cfajaxproxy example, then there's no need for document ready.

This would simplify the example just a tad.

— Reply to this email directly or view it on GitHub< https://github.com/cfjedimaster/ColdFusion-UI-the-Right-Way/issues/47> .

Regards,

Dave White

— Reply to this email directly or view it on GitHubhttps://github.com/cfjedimaster/ColdFusion-UI-the-Right-Way/issues/47#issuecomment-43672593 .

Raymond Camden, Web Developer for Adobe

Email : raymondcamden@gmail.com Blog : www.raymondcamden.com Twitter: raymondcamden

cfjquery commented 10 years ago

Indeed. Sorry about that Ray and Phil. I was a bit distracted by trying to zero my inbox. I apologise, need to slow down ;-)

On Tue, May 20, 2014 at 8:41 PM, Raymond Camden notifications@github.comwrote:

You may have gotten this email because you are signed up for Notifications, Dave.

On Tue, May 20, 2014 at 2:35 PM, Dave White notifications@github.comwrote:

I didn't write the cfajaxproxy example.

On Tue, May 20, 2014 at 7:51 PM, Phillip Senn notifications@github.comwrote:

Since you've put jquery.js at the bottom in your cfajaxproxy example, then there's no need for document ready.

This would simplify the example just a tad.

— Reply to this email directly or view it on GitHub< https://github.com/cfjedimaster/ColdFusion-UI-the-Right-Way/issues/47> .

Regards,

Dave White

— Reply to this email directly or view it on GitHub< https://github.com/cfjedimaster/ColdFusion-UI-the-Right-Way/issues/47#issuecomment-43672593>

.

Raymond Camden, Web Developer for Adobe

Email : raymondcamden@gmail.com Blog : www.raymondcamden.com Twitter: raymondcamden

— Reply to this email directly or view it on GitHubhttps://github.com/cfjedimaster/ColdFusion-UI-the-Right-Way/issues/47#issuecomment-43673280 .

Regards,

Dave White

adamcameron commented 10 years ago

Again, it would be more useful to "fix" the issue than just comment on it, Phillip. That said, I'm not entirely sure I agree as it's fairly (?) ubiquitous boilerplate to have the inline JS to wait for the document to be ready before executing. You're kind of advocating not putting curly braces on and if() statement that has only one statement for the true case. Mileage varies as to whether that's a practice with merit.

I'm going with "this is a nuisance issue". Closing