Roger, > Looking at the final code, I have just one comment: > > Should CalendarLangFileLinkDesc still exist ? After all, now it seems > to be very specific do CalendarWidgets... > > BUT, if it makes sense to preserve this WidgetDescription, its name > should be changed from "Localizable JS Link" to something like > "CalendarLangFileLink" or "CalendarLangFile Link"... (if you look at > the Widget Browser you will see 2 "Localizable JS Link")
Good point. I decided to simply remove it for now for the sake of YAGNI. Max. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "TurboGears Trunk" 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/turbogears-trunk -~----------~----~----~----~------~----~------~--~---
