-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
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
Prepare 0.15.2 release #4636
Prepare 0.15.2 release #4636
Conversation
I think this is a lot better and safer 👍 😅 But the changelog will need updating 😁 |
@alitaheri Awesome, let's use this branch in that case 😁 . Do you want to update the changelog? I can do it otherwise. |
@oliviertassinari Please update this branch ( add the recent merged commits from master ) and then merge this. I'll update the changelog tomorrow ( if you need it fast, you can use the one from my PR 😁 and go on with the release ) |
@alitaheri Sure, I'm gonna redo it with the current master branch. Would be great to release it as soon as possible. I will take care of the CHANGELOG starting from your PR. Thanks! |
It is no longer necessary to set the `linkButton` prop with the href prop. Add missing `href` prop to IconButton
`href` was added to the props destructure when linkButton was deprecated, so is no longer in `...other`. The style attrubute wasn't being applied as a result. Also fixed the `href` PropType.
alphabetize styles Update CalendarYear.js add prepareStyles
Currently it is only possible to style the icon if the entire MenuItem is in desktop mode. Be consistent with the right icon and allow styling.
Affects the following components: Checkbox, DatePicker, EnhancedTextarea, IconButton, ListItem, MenuItem, RadioButton, SelectField, Stepper, Table, TextField, Toggle
some values of a style can be nulls and doesnt need a replacement. we need to skeep them. Plese not that in the cases like 'direction' (#L44) we want replacment for null anyway. So only when we invoke function we check for null. Fix for this issue: #4364
…tial tab selection.
Continuation of #4601. As raised by @nathanmarks, stubing the console is often not a good idea. I have removed them to a more *unit test* approach. Closes #4601. Closes #4592.
Continuation of #4037. Thanks @marcel-ernst for the original PR! Closes #4037 Closes #4037
Continuation of #4531. There are several cases where `onMouseLeave` (which normally unsets the `hovered` state won't be called). One of those is if you disable a button within the `onTouchTap` method. This check ensures that a button will never be disabled but have an internal hovered state. Closes #4531.
…ext when rendering AutoComplete menu item We were using itemValue instead of itemText which made the AutoComplete menu render the wrong property as primaryText.
@callemall/material-ui It should be ready to go. Please have a look. I have updated the CHANGELOG as suggested by @alitaheri. |
@oliviertassinari Button text alignment is broken by 1px, but this is also in |
@nathanmarks I haven't noticed it. Good catch. The root of this issue seems old and unclear. I don't think that it should block the release. But it would be good to address it in the next one. |
@oliviertassinari LGTM outside of that |
Redo #4572 to double check the different, just to make sure it's working fine this time 😄 .
I have simply run the following command and dropped the breaking changes. Hopefully, we have only one: #4449.
@alitaheri Can you have a look at the different 😁 ?
Closes #4572.