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

[CardMedia] Guard against null children and overlays #8055

Merged
merged 1 commit into from Sep 5, 2017

Conversation

AlexMayants
Copy link

CardMedia with actAsExpander prop set to true throws an exception here https://github.com/callemall/material-ui/blob/master/src/Card/CardMedia.js#L110 because Card adds an undefined child to it if there's no showExpandableButton.

This PR fixes the issue by checking whether a child is empty.

@oliviertassinari oliviertassinari merged commit f96acf1 into mui:master Sep 5, 2017
@oliviertassinari
Copy link
Member

@AlexMayants Thanks.

djbuckley added a commit to manchesergit/material-ui that referenced this pull request Sep 14, 2017
* call_em_all_-_master/master:
  [EnhancedButton] -- Prevent href being overwritten by undefined. (mui#8103)
  [CardMedia] Guard against null children and overlays (mui#8055)
  [Docs] Add v0.19.1 to versions.json
  v0.19.1
  [CHANGELOG] Prepare v0.19.1
  Have the InkBar in the Tabs component honor the value prop (mui#8010)
  [AutoComplete] Fix bug where controlled component behaves uncontrolled. (mui#7967)
  stop preventing event default on iconmenu open (mui#7933)
  Fix PopOver animation origin when horizontal=middle (mui#7930)
  [Slider] Resolve slider value to respect max and min (mui#7863)
  Update onNewRequest documentation to explain chosenRequest returns dataSource value. (mui#7835)
  [Docs] Add v0.19.0 to versions.json
  v0.19.0
  [CHANGELOG] Prepare v0.19.0
  [core] Add react^16.0.0-beta.5 in peer dependencies
  [DatePicker] Fixed bug where 'isSelectedDateDisabled' was returning undefined (mui#7735)
  Create CODE_OF_CONDUCT.md (mui#7704)
  Update related projects. (mui#7706)

# Conflicts:
#	src/internal/EnhancedButton.js
@AlexMayants
Copy link
Author

Hi @oliviertassinari, if you don't mind asking, when do you plan to publish a new release including this PR?

@oliviertassinari
Copy link
Member

Today.

@AlexMayants
Copy link
Author

Great to know, thanks!

@AlexMayants AlexMayants deleted the card_media_expansion branch September 18, 2017 19:44
@zannager zannager added the component: card This is the name of the generic UI component, not the React module! label Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: card This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants