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.

[Popper] Extra render found

See original GitHub issue

Exactly, we don’t need this line code https://github.com/mui-org/material-ui/blob/a1018937b3df5aaa1c6f11bfcafc662a8249f7e2/packages/material-ui/src/Popper/Popper.js#L174-L177

Because we have handleRef which call handleOpen https://github.com/mui-org/material-ui/blob/a1018937b3df5aaa1c6f11bfcafc662a8249f7e2/packages/material-ui/src/Popper/Popper.js#L148-L154

Issue Analytics

State:
Created 3 years ago
Reactions:1
Comments:5 (5 by maintainers)

Top GitHub Comments

1reaction

inomdzhoncommented, May 18, 2020

@oliviertassinari sure 😄I will do it this week 👌

0reactions

oliviertassinaricommented, May 18, 2020

@inomdzhon Cool, this sounds fair. Do you want to open a pull request? 😄

Top Results From Across the Web

[Popper] Extra render found #21102 - mui/material-ui - GitHub

I create demo in codesandbox. Look to console – when handleOpen will updated, handleRef will triggered again because handleRef will be new ...

Render Props - Popper

A function (render prop) that takes as argument an object containing the following properties: ref : used to retrieve the React refs of...

Popper.js React Wrapper - React.createElement: type is invalid

It does render the component. However, when I click everything breaks. I am using it in Gatsby.js - if that should make a...

Popper Z-Depth Pass export in Octane and C4D (Quick Tip)

In this video, I show how to properly export out a Z-Depth Pass from Cinema 4D and Octane Render. I also show how...

The Original Popco Silicone Microwave Popcorn Popper with ...

Amazon.com: The Original Popco Silicone Microwave Popcorn Popper with Handles, Silicone Popcorn Maker, Collapsible Bowl Bpa Free and Dishwasher Safe - 10 ...