#24408 Issue: Update _AutoHideScrollbar.pcss

[akshattchhabra]

Checklist

• Tests written for new code (and old code if feasible)
• Linter and other CI checks pass
• Sign-off given on the changes (see CONTRIBUTING.md)

Signed-off-by: Akshatt Chhabra [email protected]

---

This PR currently has none of the required changelog labels.

A reviewer can add one of: T-Deprecation, T-Enhancement, T-Defect, T-Task to indicate what type of change this is, or add Type: [enhancement/defect/task] to the description and I'll add them for you.

[t3chguy]

Sign-off given on the changes (see CONTRIBUTING.md)

Missing sign off. Please read the document linked.

[t3chguy]

Won't this make it very different between webkit and non-webkit browsers?

[weeman1337]

+ @akshattchhabra can you send screenshots before and after the changes? We should verify, that the scrollbars don't overlap with other content or break the layout. Someone should test the final solution with a non-webkit browser.

[akshattchhabra]

Before:

After:

Here, I have attached the screenshots.

So, I checked the solution on a non-webkit browser. Yeah, you are right there would be differences between them and I checked about the possible solutions to make this compatible.

Solution:
To resolve this issue, we need to change the "scrollbar-width: thin" to "scrollbar-width: auto". This will work for the non-webkit browsers. Also, we need to completely remove the webkit-scrollbar css. This will ensure that the scrollbars are wide enough and identical for both kinds of browsers. Please find the attached screenshots for solution.

Webkit browser

non-webkit browser

This is what I think should be the solution. However, I am not sure if this is correct. Please suggest if you have any other solution.

[daniellekirkwood]

👋 Is there a corresponding issue associated with this so that I might better understand the problem you're solving and the approach you're taking?

[robintown]

@daniellekirkwood My guess is element-hq/element-web#22084 and element-hq/element-web#24408

[MidhunSureshR]

I'm closing this PR since there's no associated issue and because we don't want the scrollbars to be this wide