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

fix height IDE preview bug #146

Merged
merged 6 commits into from
Mar 9, 2015
Merged

fix height IDE preview bug #146

merged 6 commits into from
Mar 9, 2015

Conversation

manuio
Copy link
Contributor

@manuio manuio commented Mar 5, 2015

No description provided.

@drasko
Copy link
Contributor

drasko commented Mar 5, 2015

@nmarcetic can you please review this before merge, please.

@manuio
Copy link
Contributor Author

manuio commented Mar 5, 2015

@nmarcetic I've only changed this line: manuio@3398750#diff-b62ecd8d7fee6978acb0ddda272544f9R436

The others modification are onely some clean-up.

I've tried some methods to set "height" of "iframeContainerIndex" at 100% like in "iframeContainer" but it doesn't work... Maybe you know better solution ?

@nmarcetic
Copy link
Contributor

@manuio With current structure this is less painful approach.
Thanks!

@manuio
Copy link
Contributor Author

manuio commented Mar 5, 2015

@nmarcetic Cool!
@drasko If is good for everyone this issue can be closed: #43

drasko added a commit that referenced this pull request Mar 9, 2015
@drasko drasko merged commit bce5b78 into nodesign:next Mar 9, 2015
@manuio manuio deleted the branch_issue43 branch May 14, 2015 13:19
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