question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

ExpandableTile tileMaxHeight property is overwritten.

See original GitHub issue

Detailed description

Describe in detail the issue you’re having.

tileMaxHeigh property that is passed to ExpandableTile components gets overwritten by other values after clicking on the Tile.

Is this issue related to a specific component?

ExpandableTile

What did you expect to happen? What happened instead? What would you like to see changed?

I’m not entirely sure as the purpose of tileMaxHeight prop is not very clear but I would expect somewhat respecting this passed height by either TileAboveTheFoldContent or TileBelowTheFoldContent or both.

What browser are you working in?

Chrome

What version of the Carbon Design System are you using?

6.33.4

Steps to reproduce the issue

  1. Use official storybook
  2. Go to the Tile/Expendable
  3. Change the tileMaxHeight knob
  4. After clicking on the tile twice maxHeight gets reset.

Please create a reduced test case in code sandbox

https://codesandbox.io/s/n7891xom4 < = What’s interesting here is that I was not able to reproduce It anywhere else than storybook, using codesandbox tileMaxHeight property has no effect on the component at all (which is also not that good).

Additional information

I would love to make a PR for this as a Tile component seems very useful but I need more clarification of what is the exact purpose of the tileMaxHeight prop.

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Comments:8 (6 by maintainers)

github_iconTop GitHub Comments

1reaction
asudohcommented, May 8, 2019

Not sure if what @lekterable said matches my proposal, but will keep it in mind. May need stateful/stateless separation tacked in #2207.

0reactions
carbon-botcommented, May 9, 2019

Hi there! 👋 If you’re wondering why this issue was moved, we’re currently updating our repo structure so that every package is found in the same project.

This should not have any impact for you, but we wanted to give you a heads up in case you were wondering what is going on. If you have any questions, feel free to reach out to us on Slack or contact us at: carbon@us.ibm.com. Thanks!

Read more comments on GitHub >

github_iconTop Results From Across the Web

ExpandableTile tileMaxHeight property is overwritten. #2594
tileMaxHeigh property that is passed to ExpandableTile components gets overwritten by other values after clicking on the Tile. Is this issue ...
Read more >
@carbon/layout: Versions | Openbase
Emit custom properties for @carbon/themes color tokens. Icons Bug fixes. The following icons have been tweaked. battery--low.svg; ibm-security.svg; logo ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found