Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reducing column size too much crush the program #266

Closed
Kerruba opened this issue Jun 6, 2018 · 8 comments
Closed

Reducing column size too much crush the program #266

Kerruba opened this issue Jun 6, 2018 · 8 comments

Comments

@Kerruba
Copy link

Kerruba commented Jun 6, 2018

To replicate:

I've a coulmn whit very long text values on a row and very short text values on other rows.
When I'm trying to reduce the size of the column to make it almost collapse, scim crash

The only thing I can see at the top is an A column and some potentially error code: "36388 segmentation fault scim"

@andmarti1424
Copy link
Owner

Hello .Whats the size of your terminal and what is the approx. size you are trying to increase the screen to? Thanks.

@Kerruba
Copy link
Author

Kerruba commented Jun 11, 2018

Here is a gif showing you the issue, hopefully is big enought
scim_col_collpse_crash

@Kerruba
Copy link
Author

Kerruba commented Jun 28, 2018

I think this is also screwing up the terminal itself, in the sense that when it crashes an ls command is spreading the results around

@andmarti1424
Copy link
Owner

"I think this is also screwing up the terminal itself, in the sense that when it crashes an ls command is spreading the results around" > this always happen whan an ncurses app crashes.

@andmarti1424
Copy link
Owner

Hello. What version of sc-im are you using? freeze branch? Also, take note that you can use 'aa' command in normal mode to autofit content to column.

@andmarti1424
Copy link
Owner

well, just in case, I did change a validation so this behaviour can be caught. Please, feel free to update to latest commit in freeze branch and try again. Thanks.

@andmarti1424
Copy link
Owner

@Kerruba, could you test this?

@Kerruba
Copy link
Author

Kerruba commented Oct 23, 2018

Hi @andmarti1424 , sorry for not replying.

I'll give it a go and if there are further issues I'll let you know by reopening this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants