TarikHuber / material-ui-responsive-drawer

Material-UI responsive Drawer is a React-Redux component that uses Material-UI to create a responsive Drawer.
MIT License
43 stars 18 forks source link

Suddenly sandwich icon appears on right #8

Closed sandorvasas closed 7 years ago

sandorvasas commented 7 years ago

Hi,

An issue of the sandwich icon appeared for me. I checked applied css styles and tried to enable-disable each of them without luck. I'm not sure why this is happening. See the screenshots below.

sandwich-1 sandwich-2

Is this kind of a known issue or do you have any clues how to fix it besides applying additional css -- would be too hackish?

Thanks, Sandor

TarikHuber commented 7 years ago

Hi @sandorvasas ,

Are you using the stable or next version? Do you have some of you'r code I could check?

You can find in this repo react-most-wanted a complete example how to use this project.

sandorvasas commented 7 years ago

I'm using the stable version. Don't have code, which is the origin of the problem since I don't know where any related changes were introduced that could affect the sandwich icon. Before that specific unknown change it had been working fine.

I keep investigating further and share any results.

sandorvasas commented 7 years ago

So far I believe there is an incompatibility with material-ui-icons package. Whenever I install it, the gray div appears on the right.

TarikHuber commented 7 years ago

Do you have a small code snipped on how you use this module?

sandorvasas commented 7 years ago
material-ui-responsive-drawer-codesnippet

There is nothing special. I had used this module before in another project and it worked fantastic. I feel this happens because of upgrading probably the drawer package itself and installing material-ui-icons, but not sure about it more precisely. It's not my css surely, since I tried also without any css.

TarikHuber commented 7 years ago

Have you tried to remove all those child components from ResponsiveDrawer?

sandorvasas commented 7 years ago

It seems this issue is present in version 1.2.3. Installing 1.1.31 resolved it.

Thanks

TarikHuber commented 7 years ago

Yes that could cause the problem because the version 1.2.x is prepared for the material-ui@next