Skip to content

react-datetime renders too much html elements without a reason #648

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

Open
jocoone opened this issue Aug 19, 2019 · 2 comments
Open

react-datetime renders too much html elements without a reason #648

jocoone opened this issue Aug 19, 2019 · 2 comments

Comments

@jocoone
Copy link

jocoone commented Aug 19, 2019

I'm Submitting a ...

[ ] Bug report
[ ] Feature request
[x] Support request

Steps to Reproduce

If the datepicker is loaded into the DOM, it already exists out of more than 70 DOM-nodes. Therefor if a lot of datepickers are rendered on screen, the UI becomes very laggy.

Expected Results

These DOM-nodes should only be rendered when the datepicker is open.

Actual Results

Only render the datepicker when it is open.

Other Information (e.g. stacktraces, related issues, suggestions how to fix)

/

@jocoone jocoone changed the title react-datetime renders way to much html elements without a reason react-datetime renders to much html elements without a reason Aug 19, 2019
@jocoone jocoone changed the title react-datetime renders to much html elements without a reason react-datetime renders too much html elements without a reason Aug 19, 2019
@arqex
Copy link
Owner

arqex commented Sep 20, 2020

Thanks for raising the issue. The rendered calendar is needed in case we wanted to animate when the calendar opens or closes. But it's true that we don't need the whole picker with its content to start the animation, so probably we can render only the container. That would be an enhancement.

@arqex
Copy link
Owner

arqex commented Sep 23, 2020

We are tackling the issue at #732

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants