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.

Sign up for GitHub

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

Jump to bottom

[🐞] CLS warning is incorrectly styled. #4840

Closed
mhevery opened this issue Jul 21, 2023 · 2 comments
Closed

[🐞] CLS warning is incorrectly styled. #4840

mhevery opened this issue Jul 21, 2023 · 2 comments
Labels
STATUS-1: needs triage New issue which needs to be triaged TYPE: bug Something isn't working

Comments

@mhevery
Copy link
Contributor

mhevery commented Jul 21, 2023

Which component is affected?

Qwik Runtime

Describe the bug

https://www.loom.com/share/0f9759427cb04528ac5a917781e224e0

image

The text CLS 0.008 is incorrectly styled to be position: absolute, resulting it being fixed on the page, when the page is scrolled.

Reproduction

https://www.loom.com/share/0f9759427cb04528ac5a917781e224e0

Steps to reproduce

No response

System Info

na

Additional Information

No response
@mhevery mhevery added TYPE: bug Something isn't working STATUS-1: needs triage New issue which needs to be triaged labels Jul 21, 2023
@manucorporat
Copy link
Contributor

Please provide a repo case, position: absolute is the correct option, in other pages it scroll as expected

@gioboa
Copy link
Member

gioboa commented Jun 16, 2024

I recreated a CLS scenario and absolute is working good.
Probably in your test you had a clash with the rest of the CSS.
I close it because it's working fine.

@gioboa gioboa closed this as completed Jun 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
STATUS-1: needs triage New issue which needs to be triaged TYPE: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mhevery @manucorporat @gioboa