jQuery UI Widgets Forums Grid Certain text breaks column resizing

This topic contains 3 replies, has 2 voices, and was last updated by  Peter Stoev 8 years, 10 months ago.

Viewing 4 posts - 1 through 4 (of 4 total)
  • Author
  • Certain text breaks column resizing #85069

    StanLogan
    Participant

    Hi,

    We have some text in one of our fields that is displaying in the grid. It is a string field. When this text appears, the column will not resize correctly when grid.jqxGrid(‘autoresizecolumns’) is called. The column is sized much smaller than necessary, but I can’t see any logic behind the size it ends up with. Here are the contents of the field:

    
    				document.write("<");<xml><<<> </xml>
    				

    This text was used specifically to stress test our old home-grown grid. Hopefully, there is a way to allow this to work correctly.

    Thanks,
    Stan

    Certain text breaks column resizing #85071

    Peter Stoev
    Keymaster

    Hi Stan,

    The auto-size feature calculates the column’s size based on what the browser returns as Width for the specific text. It’s not perfect and that is the reason you can apply manually a width to a column.

    Best Regards,
    Peter Stoev

    jQWidgets Team
    http://www.jqwidgets.com

    Certain text breaks column resizing #85072

    StanLogan
    Participant

    Hi Peter,

    Thanks for the reply. This text appears only in one row for that column. The other rows have text that is also larger than the calculated column width. I understand that things are not always perfect, but even if the cell with the text returns a smaller width value, why would that affect the final column width when the other rows have values that should make the column wider?

    I can live with this issue, since our users are unlikely to have text like this, and they can still resize the column manually. I just thought it indicated a problem that you might like to address.

    Stan

    Certain text breaks column resizing #85073

    Peter Stoev
    Keymaster

    We’ll test such scenario for a future update. Thanks for feedback.

Viewing 4 posts - 1 through 4 (of 4 total)

You must be logged in to reply to this topic.