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

fix(files): Move loading icon before grid toggle #47370

Merged
merged 2 commits into from
Aug 21, 2024
Merged

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Aug 21, 2024

Summary

This prevents jumping of that toggle button when the view is loading. Also adjust the design for Nextcloud 30 (prevent jumping of breadcrumbs when loading due to increased height).

Before

Bildschirmaufnahme_20240821_022625.webm

After

Bildschirmaufnahme_20240821_022550.webm

Checklist

@susnux susnux added bug design Design, UI, UX, etc. 3. to review Waiting for reviews feature: files labels Aug 21, 2024
@susnux susnux added this to the Nextcloud 31 milestone Aug 21, 2024
@susnux susnux requested a review from a team August 21, 2024 00:30
@susnux susnux requested a review from skjnldsv as a code owner August 21, 2024 00:30
@susnux susnux requested review from artonge and Fenn-CS and removed request for a team August 21, 2024 00:30
@susnux
Copy link
Contributor Author

susnux commented Aug 21, 2024

/backport to stable30

@susnux
Copy link
Contributor Author

susnux commented Aug 21, 2024

/compile

This prevents jumping of that toggle button when the view is loading.
Also adjust the design for Nextcloud 30 (prevent jumping of breadcrumbs when loading due to increased height).

Signed-off-by: Ferdinand Thiessen <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux merged commit 974cda6 into master Aug 21, 2024
113 checks passed
@susnux susnux deleted the fix/loading-icon branch August 21, 2024 12:37
@susnux susnux mentioned this pull request Aug 31, 2024
28 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug design Design, UI, UX, etc. feature: files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: List/grid toggle jumps while loading
3 participants