search

ddomingues / X-SlickGrid

An extension of SlickGrid providing Frozen Columns and Rows, Tree Columns, etc.
http://ddomingues.com/X-SlickGrid/
MIT License
48 stars 19 forks source link

updateCell funciton uses DOM API instead of jQuery API #10

Open mohoch1 opened 9 years ago

mohoch1 commented 9 years ago

The cellNode should not be treated as a DOM element as it is a jQuery object. The current code, is actually the code from the original SlickGrid, trying to set the HTML of the cell using innerHTML, while in the X-SlickGrid this element is actually a jQuery object.

The actual way to set the HTML in this element is using the jQuery.html() funciton

NOTE: There might be a need to review the entire flow and handle this in a lower level (maybe the getCellNode funciton), but this solves the critical issue.

ddomingues commented 9 years ago

Hi @mohoch1.

Could you add an example that demonstrates this bug?