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

Prepare 0.15.2 release #4636

Merged
merged 57 commits into from
Jul 9, 2016
Merged

Prepare 0.15.2 release #4636

merged 57 commits into from
Jul 9, 2016

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Jul 6, 2016

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.

(react-v15.2.0-master) $ git rebase 0.15-stable -i

@alitaheri Can you have a look at the different 😁 ?

Closes #4572.

@alitaheri
Copy link
Member

I think this is a lot better and safer 👍 😅 But the changelog will need updating 😁

@oliviertassinari
Copy link
Member Author

@alitaheri Awesome, let's use this branch in that case 😁 . Do you want to update the changelog? I can do it otherwise.

@alitaheri
Copy link
Member

@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 )

@oliviertassinari
Copy link
Member Author

@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!

mbrookes and others added 25 commits July 7, 2016 23:19
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
oliviertassinari and others added 22 commits July 7, 2016 23:20
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 #4282.

Closes #4282
Closes #4575.
Closes #4258.
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.
Continuation of #4620

Closes #4620
Closes #4276
Closes #4331
…ext when rendering AutoComplete menu item

We were using itemValue instead of itemText which made the AutoComplete menu render the wrong property as primaryText.
@oliviertassinari oliviertassinari added this to the 0.15.2 milestone Jul 7, 2016
@oliviertassinari
Copy link
Member Author

@callemall/material-ui It should be ready to go. Please have a look. I have updated the CHANGELOG as suggested by @alitaheri.

@nathanmarks
Copy link
Member

@oliviertassinari Button text alignment is broken by 1px, but this is also in 0.15.1 and 0.15.0 as well as far as I can see... is this new in chrome? How did we not notice this?

image

@oliviertassinari
Copy link
Member Author

@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.

@nathanmarks
Copy link
Member

@oliviertassinari LGTM outside of that

@oliviertassinari oliviertassinari merged commit 341a886 into mui:0.15-stable Jul 9, 2016
@oliviertassinari oliviertassinari deleted the react-v15.2.0 branch July 9, 2016 09:08
@zannager zannager added the docs Improvements or additions to the documentation label Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.