SlickGrid icon indicating copy to clipboard operation
SlickGrid copied to clipboard

Not so much an issue. More a question: slick.grid.js updateColumnProps - if (!m.resizable)..

Open JGFMK opened this issue 5 years ago • 2 comments

if (!m.resizable) { // there is difference between user resizable and autoWidth resizable //m.autoSize.autosizeMode = Slick.ColAutosizeMode.Locked; }

Is there some unfinished business here? If with commented out code...

JGFMK avatar Nov 30 '20 19:11 JGFMK

seems like @6pac when he introduced the column auto-width resized by text content

image

ghiscoding avatar Nov 30 '20 20:11 ghiscoding

I think this can be removed, it's a feature that is still needing a bit of tweaking, so I think I left it in as a memory jogger. Looks like I had it initially assuming that !m.resizable meant the user wanted a fixed width column, but then realised that it concerns whether the column is user-resizible. So they may want it to be non-user-resizable, but still to have an initial autosize based on the text width.

The autosizing framework does need a bit of developer tweaking of the column autosize options in certain column cases to get good results, so it doesn't always work well in the default 'ContentIntelligent' mode. However, if you don't like it, you can always go back to the old way by using Legacy mode (hint - that does an even worse job!).

It's not till you try to build it that you realise that autosizing well is pretty hard to automate.

6pac avatar Nov 30 '20 23:11 6pac

that code was removed in the upcoming v4.0 - Roadmap

ghiscoding avatar May 10 '23 04:05 ghiscoding