Thanks, Kangax. My primary goal was to have a working model, but
there's no doubt that there is room for improvement.
I've changed the parameter handling per your suggestion and will look
for other ways to reduce the code size. Thanks again!
- J
On Aug 8, 10:24 pm, kangax <[EMAIL PROTECTED]> wrote:
> 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 || '&laquo;'
>
> 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-an...
> 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 [email protected]
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
-~----------~----~----~----~------~----~------~--~---
