Clean up lint for calendar #349

Merged
merged 2 commits into from Nov 28, 2012

Projects

None yet

3 participants

@apipkin
Contributor
apipkin commented Nov 26, 2012

Clean up lint. Mostly whitespace fixes, some comparison operators, and two function declarations in a for loop.

@allenrabinovich - the two functions defined in the for loop were pulled out as helpers. I wasn't exactly sure the best way to comment these functions, so left them undocumented. Any insight would be helpful.

@davglass

@allenrabinovich
Contributor

@apipkin - thanks for this. Here are the descriptions for the two formerly anonymous functions to add as API docs:

_renderSelectedDatesHelper is a handler for a date selection event (either a click or a keyboard selection) that adds the appropriate CSS class to a specific DOM node corresponding to the date and sets its aria-selected attribute to true.

_renderCustomRulesHelper takes in a date and determines whether that date has any rules matching it in the customRenderer; then calls the specified filterFunction if that's the case and/or disables the date if the rule is specified as a disabledDatesRule.

@davglass
Member

👍

@apipkin apipkin merged commit e71a076 into yui:dev-3.x Nov 28, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment