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

useParentHeight option #74

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

djgilcrease
Copy link

Add an option to useParentHeight as the target height of the container.
If you have a fixed height parent with more then 1 column the math (targetHeight = maxHeight / numCols) breaks down

Add an option to useParentHeight as the target height of the container.
If you have a fixed height parent with more then 1 column the math (targetHeight = maxHeight / numCols) breaks down
@adamwulf
Copy link
Owner

adamwulf commented Dec 6, 2012

the goal here would be to fill the parent's height entirely, but not necessarily its width? for instance, if you had enough text to fill 50% of the parent container, then it would fill the left side with columnized content instead of the top half as it currently does, yes?

just making sure i'm understanding your use case. any reason you can't just pass in the height option explicitly?

@djgilcrease
Copy link
Author

yes exactly.

I could pass the height explicitly, but that also limited the number of loops which caused some odd display issues in my case.

@strarsis
Copy link

@djgilcrease: Is it possible to do this with current columnizer code?
I want to fill the height of an element by 100% and distribute the content over its width by 50% width.

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.

3 participants