Skip to content

Conversation

@dougfabris
Copy link
Member

@dougfabris dougfabris commented Aug 4, 2025

Proposed changes (including videos or screenshots)

Our current "on scroll" only auto hide strategy for scrollbars, have been tricky for users, specially for those without touchpads or those that rely on mouse. After a discussion with the design team, we decided to change the approach to make the scroll's affordance more visible when moving the pointer in a scrollable area.

So, this PR aims to adopt the approach of hide scrollbars unless the pointer moves in the element or the user scrolls.

Kapture 2025-08-04 at 20 21 48

Issue(s)

Closes #31717
Closes #22331

Steps to test or reproduce

Further comments

CORE-1151

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Aug 4, 2025

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

@changeset-bot
Copy link

changeset-bot bot commented Aug 4, 2025

🦋 Changeset detected

Latest commit: bcc264f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 39 packages
Name Type
@rocket.chat/meteor Minor
@rocket.chat/core-typings Minor
@rocket.chat/rest-typings Minor
@rocket.chat/uikit-playground Patch
@rocket.chat/api-client Patch
@rocket.chat/apps Patch
@rocket.chat/core-services Patch
@rocket.chat/cron Patch
@rocket.chat/ddp-client Patch
@rocket.chat/freeswitch Patch
@rocket.chat/fuselage-ui-kit Major
@rocket.chat/gazzodown Major
@rocket.chat/http-router Patch
@rocket.chat/livechat Patch
@rocket.chat/model-typings Patch
@rocket.chat/ui-avatar Major
@rocket.chat/ui-client Major
@rocket.chat/ui-contexts Major
@rocket.chat/web-ui-registration Major
@rocket.chat/account-service Patch
@rocket.chat/authorization-service Patch
@rocket.chat/ddp-streamer Patch
@rocket.chat/omnichannel-transcript Patch
@rocket.chat/presence-service Patch
@rocket.chat/queue-worker Patch
@rocket.chat/stream-hub-service Patch
@rocket.chat/license Patch
@rocket.chat/omnichannel-services Patch
@rocket.chat/pdf-worker Patch
@rocket.chat/presence Patch
rocketchat-services Patch
@rocket.chat/models Patch
@rocket.chat/network-broker Patch
@rocket.chat/mock-providers Patch
@rocket.chat/ui-video-conf Major
@rocket.chat/ui-voip Major
@rocket.chat/instance-status Patch
@rocket.chat/omni-core Patch
@rocket.chat/omni-core-ee Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dougfabris dougfabris added this to the 7.10.0 milestone Aug 4, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 4, 2025

PR Preview Action v1.6.2

🚀 View preview at
https://RocketChat.github.io/Rocket.Chat/pr-preview/pr-36623/

Built to branch gh-pages at 2025-08-05 15:16 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@codecov
Copy link

codecov bot commented Aug 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.66%. Comparing base (a54f883) to head (bcc264f).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #36623      +/-   ##
===========================================
- Coverage    65.68%   65.66%   -0.02%     
===========================================
  Files         3197     3197              
  Lines       106812   106799      -13     
  Branches     20337    20328       -9     
===========================================
- Hits         70155    70127      -28     
- Misses       34014    34027      +13     
- Partials      2643     2645       +2     
Flag Coverage Δ
e2e 56.94% <ø> (-0.01%) ⬇️
unit 71.22% <100.00%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dougfabris dougfabris changed the title feat: Display scrollbars while pointer moves or the user scrolls feat: Display scrollbars while pointer moves or user scrolls Aug 4, 2025
@dougfabris dougfabris marked this pull request as ready for review August 5, 2025 13:05
@dougfabris dougfabris requested a review from a team as a code owner August 5, 2025 13:05
@MartinSchoeler MartinSchoeler added the stat: QA assured Means it has been tested and approved by a company insider label Aug 5, 2025
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Aug 5, 2025
@kodiakhq kodiakhq bot merged commit 40bc0d7 into develop Aug 5, 2025
47 of 49 checks passed
@kodiakhq kodiakhq bot deleted the feat/scrollbar-onMove branch August 5, 2025 16:00
nmagedman added a commit to seekingalpha/Rocket.Chat that referenced this pull request Sep 4, 2025
https://github.com/RocketChat/Rocket.Chat/pull/36623/files

In 7.5.0, they re-implemented how they manage scrollbars.
A negative side-effect was that scrollbars auto-hide even
on platforms that have no elevator-less scrolling ability.
This PR causes the scrollbar to reappear on any mouse movement.

Issue-Url: https://seekingalpha.atlassian.net/browse/AR-3667
nmagedman added a commit to seekingalpha/Rocket.Chat that referenced this pull request Nov 18, 2025
https://github.com/RocketChat/Rocket.Chat/pull/36623/files

In 7.5.0, they re-implemented how they manage scrollbars.
A negative side-effect was that scrollbars auto-hide even
on platforms that have no elevator-less scrolling ability.
This PR causes the scrollbar to reappear on any mouse movement.

Issue-Url: https://seekingalpha.atlassian.net/browse/AR-3667
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: QA assured Means it has been tested and approved by a company insider stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Always show scroll bars No scroll bars in Channel user list, user profile panel, or left grey panel.

3 participants