θΏ™ζ˜―indexlocζδΎ›ηš„ζœεŠ‘οΌŒδΈθ¦θΎ“ε…₯任何密码
Skip to content

Removed redundant scrollbar in search containers #3865

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

17ColinMiPerry
Copy link
Contributor

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

What is in this change?

Search containers had a redundant and unused scrollbar. This scrollbar was made functional, and the outer scroll bar was removed for improved UI presentation.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

@timothycarambat
Copy link
Member

timothycarambat commented May 26, 2025

Can you provider screenshots pre and post fix to showcase what the issue is since there is no associated GH issue with this PR?

@timothycarambat timothycarambat added blocked needs info / can't replicate Issues that require additional information and/or cannot currently be replicated, but possible bug labels May 26, 2025
@17ColinMiPerry
Copy link
Contributor Author

Before

before

After

after

@timothycarambat timothycarambat merged commit 3c44d6a into Mintplex-Labs:master May 26, 2025
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
* removed extra overflow scrollbar

* improved scrollbar presentation

* ran yarn lint

---------

Co-authored-by: timothycarambat <rambat1010@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked needs info / can't replicate Issues that require additional information and/or cannot currently be replicated, but possible bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants