Skip to content

Conversation

@Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Nov 4, 2025

☑️ Resolves

Note

vue-virtual-scroller uses CSS transform: translate() for positioning. It was known as performance optimisation via moving the rendering to a new layer with GPU acceleration and being composite-only, without repaint/layout recalculation.
But in fact, doing it for many items makes it slower with the layout overhead, at least in Chromium.
So while it makes sense for some animation optimisations, it doesn't for position/list items rendering.
Also, it sometimes has known rasterisation issues. For example, this is why NcActions were blurry on scale.
Nowadays virtual scrollers more often use just a simple padding-top.

🖌️ UI Checklist

🚧 Tasks

  • Reactivity issues
  • Smooth scrolling

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

@ShGKme

This comment was marked as resolved.

@Antreesy Antreesy force-pushed the fix/noid/use-virtual-list branch 3 times, most recently from 9b23016 to 64f3b31 Compare November 4, 2025 13:15
@Antreesy Antreesy marked this pull request as ready for review November 4, 2025 13:25
@Antreesy Antreesy force-pushed the fix/noid/use-virtual-list branch from 8a2b94a to 80a405a Compare November 14, 2025 17:07
@Antreesy Antreesy requested a review from ShGKme November 14, 2025 17:08
@Antreesy Antreesy force-pushed the fix/noid/use-virtual-list branch 3 times, most recently from 9e08381 to 2e73de8 Compare November 15, 2025 11:37
@Antreesy Antreesy force-pushed the fix/noid/use-virtual-list branch from 2e73de8 to a9f8342 Compare November 17, 2025 13:59
@Antreesy Antreesy merged commit 7f25eba into main Nov 17, 2025
56 checks passed
@Antreesy Antreesy deleted the fix/noid/use-virtual-list branch November 17, 2025 14:03
@Antreesy
Copy link
Contributor Author

/backport to stable32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants