Skip to content

Conversation

@nnaydenow
Copy link
Contributor

@nnaydenow nnaydenow commented Oct 14, 2025

Simplify the selector by moving the .ui5-content-native-scrollbars CSS class from the body element to the html element.

The previous approach introduced performance overhead because it required checking whether the class was applied to the body, while the relevant styles actually needed to target the html element and all of its children. Using the :has() selector for this purpose proved expensive on large DOMs.

@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Oct 14, 2025

🧹 Preview deployment cleaned up: https://pr-12485--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview October 14, 2025 08:59 Inactive
@nnaydenow nnaydenow changed the title chore: scrollbar test chore: native scrollbar styles Oct 14, 2025
@nnaydenow nnaydenow changed the title chore: native scrollbar styles fix: native scrollbar styles Oct 17, 2025
@nnaydenow nnaydenow merged commit 92a954f into main Oct 17, 2025
21 of 22 checks passed
@nnaydenow nnaydenow deleted the sc-test2 branch October 17, 2025 08:18
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview October 17, 2025 08:18 Inactive
@ui5-webcomponents-bot
Copy link
Collaborator

🎉 This PR is included in version v2.16.0-rc.2 🎉

The release is available on v2.16.0-rc.2

Your semantic-release bot 📦🚀

openui5bot pushed a commit to UI5/openui5 that referenced this pull request Nov 5, 2025
…llbars

UI5/webcomponents#12485

SNOW: DINC0692223
Change-Id: I18d2fe512b4b9037198445a1b34b3425cfaf3453
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants