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

Different behaviour in FF and Chromium with flexbox #3231

Closed
vmx opened this issue May 2, 2015 · 4 comments
Closed

Different behaviour in FF and Chromium with flexbox #3231

vmx opened this issue May 2, 2015 · 4 comments

Comments

@vmx
Copy link
Contributor

vmx commented May 2, 2015

I'm using CodeMirror with a flexbox layout. I see differences between Firefox (39.0a2, 2015-04-29) and Chromium (41.0.2272.118), both on Debian. I can get it work properly with either of them, but not both at the same time.

Please see this minimal example and the comment in the CSS for more:
http://output.jsbin.com/vapudetemi/1/edit?css,output

@marijnh
Copy link
Member

marijnh commented May 5, 2015

There is nothing in the CSS of the jsbin you linked.

@vmx
Copy link
Contributor Author

vmx commented May 5, 2015

I've updated the original comment to contain the correct link.

@marijnh
Copy link
Member

marijnh commented May 6, 2015

That is interesting. This looks like a browser bug -- Chrome interpreting the height: 100% of the CodeMirror instance in a weird way. Even if I make .item have position: relative, the editor ends up too high. I don't have much experience with flexbox, but I don't think it is supposed to violate height calculation expectations like that. Even if I replace the editor instances with simply a div with class CodeMirror, the problem persists (the divs are the height of the outer element, rather than the inner ones).

@marijnh
Copy link
Member

marijnh commented Sep 2, 2015

I am now getting even stranger behavior in Firefox. Closing this on the assumption that the browser are buggy and this isn't something CodeMirror itself can help (the issues show up even if CodeMirror is removed from the equation)

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

No branches or pull requests

2 participants