-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Open
Description
The 5 issues for us that made updating to the latest not an easy drop in was
- The change to using === when comparing radio values.
https://github.com/knockout/knockout/blob/v3.0.0/src/binding/defaultBindings/checked.js#L64
https://github.com/knockout/knockout/blob/v2.3.0/src/binding/defaultBindings/checked.js#L40 - Having valueAllowUnset default to false
- Throwing errors for already bound; vs ignoring them
- Throwing errors for "Cannot find template with ID"; vs ignoring them (Single Page Application)
- Have value = undefined; if it doesn't match a given value in the select dropdown list.
I made the first 4 issues be options and the 5th one the default behavior as select lists are broken if you populate it with a value that's not in the options.
window.knockout_globals = {
'valueAllowUnset': true,
'useStrictEqualityRadioCheckbox': false,
'alreadyBoundErrorLevel': 0,
'selectFirstWhenUnset': true,
'suppressTemplateError': true,
};
Metadata
Metadata
Assignees
Labels
No labels