search

HenrikJoreteg / human-view

A smart base view for Backbone apps, to make it easy to bind collections and properties to the DOM.
33 stars 6 forks source link

IE10 fix: when reRender calls addView via each, the view is rendered everytime #19

Closed CNovakCAA closed 10 years ago

CNovakCAA commented 10 years ago

For some reason IE10 loses view.el.innerHTML when reRender() is run and calls addView() via each().

Newly added views are rendered correctly, but existing views all have an empty view.el.

With this change, each view calls render() whether or not it previously existed. This prevents IE10 from wiping the entire collection when create() is called on a child collection of a model.

Not sure if this is related to human-view and human-model being tied to Backbone 1.0.0 when Backbone 1.1.2 is available... I tried upgrading Backbone, but that broke everything. :-/

CNovakCAA commented 10 years ago

This is no longer the correct solution to this undesired behavior. The issue and problem are described in detail in issue #21.