fix #5525: add independent scroll to filters bar

[guillermo-escire]

References

Fixes #5525

Description

The filters bar was causing the whole page to scroll when interacting
with it. Added independent scroll behavior to the filters sidebar so
it scrolls on its own without moving the entire page.

Instructions for Reviewers

List of changes in this PR:

• Added overflow-y: auto to the :host selector in search-filters.component.scss to enable independent scrolling
• Added overscroll-behavior: contain to prevent scroll events from propagating to the parent page

To test:

1. Go to the Search page (/search)
2. Hover over the filters sidebar (Autor/a, Materias, Fecha, etc.)
3. Scroll with the mouse wheel
4. Expected: only the filters area scrolls, the results list stays still

Checklist

This checklist provides a reminder of what we are going to look for when reviewing your PR. You do not need to complete this checklist prior creating your PR (draft PRs are always welcome).
However, reviewers may request that you complete any actions in this list if you have not done so. If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

• My PR is created against the main branch of code (unless it is a backport or is fixing an issue specific to an older branch).
• My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
• My PR passes ESLint validation using npm run lint
• My PR doesn't introduce circular dependencies (verified via npm run check-circ-deps)
• My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
• My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
• My PR aligns with Accessibility guidelines if it makes changes to the user interface.
• My PR uses i18n (internationalization) keys instead of hardcoded English text, to allow for translations.
• My PR includes details on how to test it. I've provided clear instructions to reviewers on how to successfully test this fix or feature.
• If my PR includes new libraries/dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
• If my PR includes new features or configurations, I've provided basic technical documentation in the PR itself.
• If my PR fixes an issue ticket, I've linked them together.

[kanasznagyzoltan]

I have tested, the independent scrollbar appeared which scrolls only the sidebar facets. In my test I could only get to the bottom facets if I am using just a little bit the search result scrolling pane.
I made a video comparing the current sandbox vs this PR branch.

pr_5556.mp4

[nathanmlf]

Unlike the issue mentioned by @kanasznagyzoltan, I am able to scroll all the way down to the bottom facets within the sidebar without needing to scroll the main page first. For context, I tested this on Windows 10 using Google Chrome at 1920x1080.

Approving the PR as it works fine in my environment, but it might be worth double-checking how it behaves on others viewports. Great addition, @guillermo-escire!

[kanasznagyzoltan]

@nathanmlf
Thanks I had 125% upscale in my Windows Display Settings, maybe this had the effect.
I tried again, this time only the "Reset filter" is problematic for me. What am I doing wrong?

pr_5556_2.mp4

[nathanmlf]

@kanasznagyzoltan
Thanks for the feedback. I revisited the issue and tested it again and I’ve reproduced the same problem you're having. I'am also using a 1920x1080 monitor with a 125% upscale. I don't think we're necessarily supposed to see the settings options, but I do think the "Reset Filters" button should be visible. Here's a video of my test:

pr_5556_test2.mp4

Also, I'am using the sandbox backend and someone happened to be testing a banner right above the header. This reinforces what you said in your first comment, the sidebar scrollbar only works perfectly when we scroll down the main page.

pr_5556_test3.mp4

[nathanmlf]

Hello @guillermo-escire!
I’ve been testing the new implementation localy and I can confirm the behavior mentioned by @kanasznagyzoltan . On my end, I noticed that the "Reset Filters" button is currently hidden and only becomes accessible after scrolling down the main page. I am using a 1920x1080 monitor with an upscale of 125%.

I’m marking this as "Request Changes" just so we can discuss if this is the intended behavior. Also, while testing using the sandbox backend there was a banner above the header, and it seems to interfere with how the filter height is calculated. It looks like the max-height might need a more dynamic approach to account for different header sizes or page offsets.

I’ve attached two videos in my last comment to show exactly what’s happening on my end. Thanks for the contribution, I will be happy to re-test it if needed!