-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
[v4.0.0-beta.8] ion-col size not supported #15495
Comments
I can confirm that and just posted the same issue moments after yours. |
@comino I presented the fix above and you could be right: They made changes to that function on 31st August ;) |
Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out. |
Bug Report
If you use
<ion-col>
with the size property (size-md etc), it's not working. I already did some research and found the issue:In col.tsx (https://github.com/ionic-team/ionic/blob/master/core/src/components/col/col.tsx) the following
const columns = this.getColumns('size');
always returns "undefined" (in methodcalculateSize()
). So it fails and he falls back to auto.The root cause problem is the
matchBreakpoint();
function. That should be:I will make a PR, will see if they merge this :D
The text was updated successfully, but these errors were encountered: