If attribute value is 'min-height', set that CSS property instead.#2
If attribute value is 'min-height', set that CSS property instead.#2morganiq wants to merge 4 commits intom43nu:masterfrom morganiq:feature/optional-min-height
Conversation
|
Thanks for your addition! You need to change the code to: If you can make this change by yourself, I will test and merge the PR afterwards. |
|
Fixed! Sorry, forgot about this PR for a while. |
|
Thank you for fixing it. I will check it out later today. |
|
hi morganiq i just talked to m43nu about this stalled request - the cause we didn't merge it yet is the remaining merge conflicts. would you mind fixing them? alternatively we'll do - sometime ;) |
Conflicts: dist/auto-height.js src/auto-height.coffee
|
Sorry again for the delay; I've merged the changes. Again, I manually edited and tested the JavaScript source only. CoffeeScript changes were eyeballed and are untested. |
# Conflicts: # dist/auto-height.js
|
Updated with properly re-generated and disted |
To allow vertical expansion of the container below the fold for content tall enough to do so, look for an optional value of
min-heightand, if present, set that property instead ofheight. Includes README update.Note: CoffeeScript edits untested. Manually edited and tested JavaScript source only.