From: Leon A. <leo...@cl...> - 2005-09-16 00:02:47
|
David Fallon wrote: > one (debatable) issue with the patch - the debatable part being why > I'm posting rather than just fixing. The selection box lists "all > combined", which is meaningless in that context. I can see an argument > for it being there, but since the pulldown to go to a specific > calendar or all is still there, it seems better to not have it. thoughts? > > I agree that it's meaningless there. If I remember correctly, it wasn't immediately obvious how to exclude it, so I left it for someone else to figure out. My goal was to put the framework in there and allow someone more intimate with the code to smooth out the rough edges. It might make sense to get rid of the single selector altogether and replace it with the multi-select. And then add a javascript button to toggle on all the calendars. Thanks, Leon |