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

Chore: CSS conflicting property in development #2228

Closed
1 of 2 tasks
a0m0rajab opened this issue Nov 29, 2023 · 10 comments
Closed
1 of 2 tasks

Chore: CSS conflicting property in development #2228

a0m0rajab opened this issue Nov 29, 2023 · 10 comments
Labels
🐛 bug Something isn't working good first issue Good for newcomers
Milestone

Comments

@a0m0rajab
Copy link
Contributor

a0m0rajab commented Nov 29, 2023

Type of feature

🐛 Fix

Current behavior

This could be a minor development but thought to address it here, when I worked on #2185 I got this warning in the console:

Warning: Updating a style property during rerender (overflowY) when a conflicting property is set (overflow) can lead to styling bugs. To avoid this, don't mix shorthand and non-shorthand properties for the same value; instead, replace the shorthand with separate values.

I thought it would worth to open an issue for that and check it, this happened at this stream

Suggested solution

No response

Additional context

Discovered in #2185

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
Copy link
Contributor

Thanks for the issue, our team will look into it as soon as possible! If you would like to work on this issue, please wait for us to decide if it's ready. The issue will be ready to work on once we remove the "needs triage" label.

To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please reach out to us on Discord or follow up on the issue itself.

For full info on how to contribute, please check out our contributors guide.

@nickytonline nickytonline added 🐛 bug Something isn't working and removed 👀 needs triage 💡 feature labels Nov 29, 2023
@nickytonline
Copy link
Member

Feel free to work on this if you'd like @a0m0rajab.

@a0m0rajab a0m0rajab added the good first issue Good for newcomers label Nov 29, 2023
@a0m0rajab
Copy link
Contributor Author

@nickytonline thanks for the suggestion, I think I will keep it open and work on something else these days, might pick it up in the next few weeks if no one picked it up.

@sital002
Copy link
Contributor

.take

@nickytonline nickytonline added this to the Papercuts milestone Nov 29, 2023
@sital002
Copy link
Contributor

sital002 commented Dec 1, 2023

I am not able to reproduce the bug can you provide the URL for it.

@nickytonline
Copy link
Member

@a0m0rajab, do you remember what page this warning was happening on?

@a0m0rajab
Copy link
Contributor Author

It was in this URL: app.opensauced.pizza/ai/dashboard/filter/top-100-repos?range=30

Here is the video of the error: https://www.youtube.com/live/pGGV8cVsaj0?si=m_PlWAzwxV22lJ06&t=4563

I think it appeared when I run the project in debugger in VsCode.

@nickytonline
Copy link
Member

This might only be a development mode issue as it does not occur on that page in production.

@a0m0rajab
Copy link
Contributor Author

I think so since the warning does not appear on production, do you think we should work on this or just ignore that?

@sital002 sital002 removed their assignment Dec 15, 2023
@brandonroberts
Copy link
Contributor

Let's close this one for now as its a dev only warning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants