Skip to content

Conversation

@artonge
Copy link
Collaborator

@artonge artonge commented Jan 16, 2026

  1. Properly access blurhash value
  2. Do not request small preview if blurhash exists
  3. Be more honest about the actual preview size. The server only servers sizes that are multiple of 4, so 64, 256, 1024, 4096.
  4. Use smaller previews on mobile.

@artonge artonge requested a review from susnux January 16, 2026 08:42
@codecov
Copy link

codecov bot commented Jan 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@artonge artonge force-pushed the artonge/feat/better_preview_management branch from 7941daa to 5558ecd Compare January 16, 2026 09:03
@artonge artonge self-assigned this Jan 16, 2026
@artonge artonge added 3. to review Waiting for reviews javascript Javascript related ticket performance 🚀 Performance issues and optimisations feature: previews and thumbnails bug Something isn't working enhancement New feature or request labels Jan 16, 2026
@artonge artonge added this to the Nextcloud 33 milestone Jan 16, 2026
@artonge
Copy link
Collaborator Author

artonge commented Jan 16, 2026

/backport to stable32

@artonge
Copy link
Collaborator Author

artonge commented Jan 16, 2026

/backport to stable31

@backportbot backportbot bot added the backport-request Pending backport by the backport-bot label Jan 16, 2026
@artonge
Copy link
Collaborator Author

artonge commented Jan 16, 2026

/backport to stable30

@artonge artonge force-pushed the artonge/feat/better_preview_management branch 2 times, most recently from cd409b6 to c086985 Compare January 16, 2026 13:28
Signed-off-by: Louis Chmn <louis@chmn.me>
Signed-off-by: Louis Chmn <louis@chmn.me>
The server only servers sizes that are multiple of 4, so 64, 256, 1024, 4096.

Signed-off-by: Louis Chmn <louis@chmn.me>
Signed-off-by: Louis Chmn <louis@chmn.me>
The virtual list do not render anything further than 0.5 anyway.

Signed-off-by: Louis Chmn <louis@chmn.me>
Signed-off-by: Louis Chmn <louis@chmn.me>
@artonge artonge force-pushed the artonge/feat/better_preview_management branch from c086985 to cfc7b8c Compare January 16, 2026 14:21
@artonge
Copy link
Collaborator Author

artonge commented Jan 16, 2026

Cypress failure related to the sidebar API change

@artonge artonge merged commit 0951c9f into master Jan 16, 2026
44 of 50 checks passed
@artonge artonge deleted the artonge/feat/better_preview_management branch January 16, 2026 14:46
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 Something isn't working enhancement New feature or request feature: previews and thumbnails javascript Javascript related ticket performance 🚀 Performance issues and optimisations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants