Skip to content

[dashboard] try using skeletons for loading repo suggestions #20766

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
Apr 25, 2025

Conversation

filiptronicek
Copy link
Member

@filiptronicek filiptronicek commented Apr 23, 2025

Description

We've received feedback that the suggested repos component can trigger layout shifts. This PR attempts to fix those.

Related Issue(s)

Fixes CLC-1307

How to test

TBD

Preview

https://ft-improve13010e87f3.preview.gitpod-dev.com/workspaces

@filiptronicek filiptronicek requested review from a team as code owners April 23, 2025 08:24
@filiptronicek filiptronicek marked this pull request as draft April 23, 2025 08:25
@roboquat roboquat added size/XL and removed size/S labels Apr 24, 2025
@filiptronicek filiptronicek self-assigned this Apr 24, 2025
@filiptronicek filiptronicek marked this pull request as ready for review April 24, 2025 11:45
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The diff shows many changes in this file, you'll be happier with hiding whitespace changes.

@geropl
Copy link
Member

geropl commented Apr 24, 2025

@filiptronicek Can you add a git integration or make me admin?

Copy link
Member Author

@gero just did both

Copy link
Member Author

filiptronicek commented Apr 24, 2025

Please keep in mind it can take up to 10m to get from the empty state to content, because of how our caching for the queries is set up (we cache the suggested repository query quite heavily to make it snappy most of the time)

Copy link
Member

@geropl geropl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noice! ✔️

@roboquat roboquat merged commit 51254eb into main Apr 25, 2025
19 checks passed
@roboquat roboquat deleted the ft/improve-suggested-repos-layout-shift branch April 25, 2025 13:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants