[Rails-spinoffs] Re: Scal Update - Calendar Control for Prototype and Scriptaculous

[kangax]

Jamie,


It's all very nice and we DO need prototype based date picker, but...
I think your code can be easily reduced twice (at least)

i.e.

if (typeof this.options.prevbutton != 'undefined'){
   this.prevbutton = this.options.prevbutton;
}else{
   this.prevbutton = '<';
}


can be written like this:


this.prevButton = this.options.prevButton || '«'


I suggest you take a look at prototype/scriptaculous source to get familiar
with common techniques

-- 
View this message in context: 
http://www.nabble.com/Scal-Update---Calendar-Control-for-Prototype-and-Scriptaculous-tf4236031.html#a12065369
Sent from the RubyOnRails Spinoffs mailing list archive at Nabble.com.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Spinoffs" group.
To post to this group, send email to rubyonrails-spinoffs@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-spinoffs?hl=en
-~----------~----~----~----~------~----~------~--~---