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(Select): issues after upgrade to MUI v4 #590

Merged
merged 1 commit into from
Aug 15, 2019

Conversation

havenchyk
Copy link
Contributor

FX-373

Description

The structure of the select has been changed mui/material-ui#16137, so styles should be fixed

How to test

  • check demo

Review

  • Annotate all props in component with documentation
  • Create examples for component
  • Ensure that deployed demo has expected results and good examples
  • Ensure that visuals specs are green See the documentation

@havenchyk havenchyk requested a review from a team as a code owner August 14, 2019 18:51
@havenchyk havenchyk self-assigned this Aug 14, 2019
@havenchyk havenchyk added the WIP Work in Progress (Don't review) label Aug 14, 2019
@havenchyk
Copy link
Contributor Author

15/172 - compare with #577 (currently 16/172 there)

@havenchyk havenchyk removed the WIP Work in Progress (Don't review) label Aug 14, 2019
@@ -20,7 +20,6 @@ declare module '@material-ui/core/styles/createTypography' {
}

const typography = {
useNextVariants: true,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice 👍 thanks!

@havenchyk havenchyk changed the title fix: try to fix select after change of the structure fix(Select): issues after upgrade to MUI v4 Aug 15, 2019
@havenchyk havenchyk merged commit 98fa61e into fx-move-to-v4 Aug 15, 2019
@havenchyk havenchyk deleted the FX-373-fix-native-select branch August 15, 2019 06:08
@toptal-devbot
Copy link
Collaborator

🎉 Last commit is successfully deployed 🎉

Demo is available on:

Your davinci-bot 🚀

conf pushed a commit that referenced this pull request Aug 23, 2019
# [3.0.0](v2.27.1...v3.0.0) (2019-08-23)

### Bug Fixes

* **Accordion:** fix Accordion styles ([#592](#592)) ([c2636b9](c2636b9))
* **Accordion:** fix an issue with no children rendered ([b9d198e](b9d198e))
* **CssBaseline:** [FX-368] Use our own css baseline ([#595](#595)) ([bf9052c](bf9052c))
* **Grid:** divide Picasso grid spacing according to MUI ([#586](#586)) ([41b9783](41b9783))
* **paper:** fix default text color ([#594](#594)) ([5522976](5522976))
* **Select:** issues after upgrade to MUI v4 ([#590](#590)) ([7c34698](7c34698))
* **Slider:** [FX-378] Move Slider from lab to core ([#596](#596)) ([58e33b9](58e33b9))
* **Table:** adjust cell paddings ([#591](#591)) ([550d901](550d901))
* **Tabs:** after upgrade to v4 ([#593](#593)) ([07dcef8](07dcef8))
* fix disabled state for Checkbox and Radio ([#587](#587)) ([bc73cea](bc73cea))
* **Tooltip:** issues after upgrade to v4 ([#581](#581)) ([65676e0](65676e0))
* **Typography:** fix typography, baseline visual issues ([#585](#585)) ([68e8aec](68e8aec))

### chore

* **TextField:** remove deprecated props and rename to Input ([#606](#606)) ([1d63602](1d63602))
* rename `justifyContent` prop in Grid to match Container ([#600](#600)) ([8da9a02](8da9a02))
* **Icon:** delete deprecated icon components ([#598](#598)) ([b37be08](b37be08))
* **Icon:** remove deprecated prop `size` from icons ([#599](#599)) ([033a5ac](033a5ac))

### Features

* [FX-377] Forward refs for all components ([#612](#612)) ([677af2e](677af2e))
* [FX-377] Proxy ref from forwardRef to FileInput ([#621](#621)) ([54668a3](54668a3))
* upgrade MUI to v4 ([29d14bd](29d14bd))

### BREAKING CHANGES

* **TextField:** TextField component to Input
* **Slider:** Slider component has been moved from `@toptal/picasso/lab` to `@toptal/picasso`
* rename `justify` prop in `Grid`  to `justifyContent`
* **Icon:** remove deprecated prop `size` from icons
* **Icon:** icon components deprecated in v2 have been fully removed in v3. To obtain support migrating these components, check: https://toptal-core.atlassian.net/wiki/x/boACG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants