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

Time Update Logic #338

Open
the-indian-hustler opened this issue Jun 2, 2024 · 1 comment
Open

Time Update Logic #338

the-indian-hustler opened this issue Jun 2, 2024 · 1 comment

Comments

@the-indian-hustler
Copy link

Describe the bug
There's a logic issue in your time update script

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior
The setInterval function that clears the time every minute (time.innerText = "") seems unnecessary and might cause the time display to disappear. Consider removing this interval

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@the-indian-hustler
Copy link
Author

please assign this issue to me under GSSOC, I will make sure to improve the quality of the code.

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

No branches or pull requests

1 participant