Skip to content

Conversation

@disberd
Copy link
Contributor

@disberd disberd commented Sep 25, 2025

Currently within the text backend, providing a value of 0 to some of the entries in fixed_data_column_widths does not make the column width size based on the largest cell contradicting the documentation.

This simple PR just update the code to reflects the behavior in the documentation (and modifies a single test to also test providing a 0 instead of a negative number).

I considered updating the documentation, especially because all tests explicitly put -1 for autosizing, but this change would seem more consistent with the behavior when a single Int is provided which actually autosizes for 0

@ronisbr
Copy link
Owner

ronisbr commented Sep 25, 2025

Thanks @disberd ! Nice catch.

@ronisbr ronisbr merged commit b701697 into ronisbr:master Sep 25, 2025
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants