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

Min property and scroll bug #64

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Devin-Harris
Copy link
Contributor

Changes for issues described in #63

Change list includes doing width and height calculations off the max between the w and minW properties and the h and minH properties. Also triggering the calculateRenderData method after the content fully inits because the width of the parent was being consumed incorrectly in the ngOnInit method causing items to overflow outside parents bounds causing scroll.

@shahnawaz09
Copy link

@Devin-Harris Hi Devin, When can these fixes will be available to us. We are using Katoid in our application. Could you please tell me. So after merging do they need to release this in new version or how it will be.

@Devin-Harris
Copy link
Contributor Author

@Devin-Harris Hi Devin, When can these fixes will be available to us. We are using Katoid in our application. Could you please tell me. So after merging do they need to release this in new version or how it will be.

I would imagine it would it would be available upon a new release, assuming its good to merge. As far as when that would be you would probably need to ask a maintainer of the repo as I'm not sure what the plan is for future releases.

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