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

Frozen columns do not fit in viewport #1214

Closed
5 tasks done
Sevdariel opened this issue Aug 1, 2023 · 3 comments
Closed
5 tasks done

Frozen columns do not fit in viewport #1214

Sevdariel opened this issue Aug 1, 2023 · 3 comments

Comments

@Sevdariel
Copy link

Describe the bug

When trying to increase number of frozen columns after expand one of the column width to get max viewPort width with enabled option frozenRightViewportMinWidth new frozen columns are hidden and grid has wrong data.

image

After change one of the frozen column width everything return to normal.

Reproduction

Bug available to repoduce at demo page https://ghiscoding.github.io/Angular-Slickgrid/#/frozen

  1. Expand 'Title' column to max width
  2. Scroll right viewPort to see 'Finish' column
  3. Freeze 'Finish' column

Expectation

Frozen columns are moved to the left and it's available to horizontaly scroll leftViewPort.

Environment Info

Angular-Slickgrid demo

Validations

@ghiscoding
Copy link
Owner

ghiscoding commented Aug 1, 2023

I don't understand what you are trying to do exactly, are you trying to freeze or pin a column that is wider and outside of the viewport? If that is the case you should get an error in the console saying that you cannot and if so that is the expected behavior. So please check your console and see if any errors are displayed, I remember creating this PR in SlickGrid core lib to block user from expanding the column outside of the visible viewport (which is not permitted because then all columns become hidden under an invisible div)

I also cannot provide any help without a way to reproduce the error. I also will not provide support for columns that are outside of the viewport because it's simply too hard to fix

@Sevdariel
Copy link
Author

Yes, that's what i want achieve. And there is error in console. It will be any possibility to change this behavior that left viewport horizontal scroll will be usefull? I mean when freeze column that is outside of the viewport let user scroll right or left. I think it will be much user friendly than console error

@ghiscoding
Copy link
Owner

ghiscoding commented Aug 10, 2023

It will be any possibility to change this behavior that left viewport horizontal scroll will be usefull?

That is not the typical approach, users don't like to see multiple scrollbar that does the same thing. You can add CSS to also show scrollbar of the left viewport yourself if that is what you want. We will not add any grid option for this. Also if you don't want the error, you can always add a try/catch and do nothing when catching the error, if you also add CSS for the scrollbar, then that should go you what you're looking for but that is not something that we will do in this lib. You could maybe try this Stack Overflow answer to show the scrollbar on the left

The error that you see comes from SlickGrid when you indeed try to expand freeze columns that are outside the viewport, this error was brought in this SlickGrid PR and the reason you cannot expand the Title column more than the viewport is because of this SlickGrid PR.

I'm closing this issue since you barely provided any reproduction and you seem to be having problem with freezing columns outside of the viewport which I already said is not, and will not, be supported by SlickGrid... so closing

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