Tags: tpolyak/SlickGrid
Tags
Revert 06-single-stylesheet.patch due to various issues - example 'example-multi-grid-basic.html' was added to show two grids on a page - test '40 grids on a page test.html' was added to allow the problem in IE to be demonstrated consistently - we would like to use a single stylesheet for all grids on page rather than adding a new stylesheet per grid - this would get around the 31 stylesheet limitation in IE7-9 PROBLEMS: - modifying an existing stylesheet in chrome clears and regenerates all the DOM nodes for that stylesheet from scratch - this clears the cached references to the column styles for any previously initialised grids - since the CSS styles are added and cached in the final stage of initialisation, we can't fix this by delaying final init - this would take major rearchitecting of the init process to fix - note that IE does not reset all the existing nodes so doesn't have this problem SO - the first attempted fix was to have one stylesheet in IE and go back to multiple stylesheets for other browsers BUT: - the single-stylesheet approach in IE renders fine, but column resizing fails in IE8 with 'permission denied' on all but the first grid - it appears this is a CORS-relates issue. I note the href is null for inline stylesheets. Not sure if this has an impact. So in the final analysis, the best option is to revert back to the way it was originally and live with the IE7-9 31 stylesheet limitation - IE 10+ does not have this limitation anyway
Revert sparkline to 1.6 - according to mleibman/SlickGrid#855 sparkline 2+ uses a different allocation model that requires jQuery cleanup - this will have the effect of leaking memory - revert sparkline to 1.6, the last pre-2.0 version - add jquery-migrate to make it work - TODO: create an async post-remove-from-cache cleanup
Fix 6pac#302 and 6pac#319 (though the user seems confused about his o… …wn bug report): - Use a fixed position (i.e. always on screen) hidden DIV as a focusable keyboard event sink, and focus on that instead of the grid canvas. Since it is always in the viewport, no scrolling occurs. We still listen to events from editors bubbling up to the canvas and set focus on canvas clicks. - Exposed a "focus()" method on the grid. - Got rid of IE's .setActive() call since we have a better mechanism now. - Fixed "fast" arrow key navigation when asyncEditorLoading is on.
PreviousNext