Skip to content

Fix search box overflow by limiting input to 200 chars and increase result cap (#1459) #1462

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.

Sign up for GitHub

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

Jump to bottom
Merged
davsclaus merged 1 commit into from
Jan 5, 2026
Merged

Fix search box overflow by limiting input to 200 chars and increase result cap (#1459) #1462

davsclaus merged 1 commit into from
Jan 5, 2026

Conversation

@GaneshPatil7517
Copy link
Contributor

@GaneshPatil7517 GaneshPatil7517 commented Jan 5, 2026 •
edited
Loading

This PR resolves issue #1459 by limiting search result snippet text.

Changes:

  • Each search result description truncated to 200 characters max
  • CSS line-clamp limits visible lines to 3 for cleaner display
  • Increased search result cap from 5 → 10 for better visibility

Verification:
✔ Result snippets no longer overflow
✔ Dropdown shows more compact results
✔ 10 results visible
✔ No console or UI errors

image

Closes: #1459

@GaneshPatil7517 GaneshPatil7517 force-pushed the fix-searchbox-char-limit branch from 75c4a4b to d7f5915 Compare January 5, 2026 10:05
@GaneshPatil7517 GaneshPatil7517 mentioned this pull request Jan 5, 2026
Closed
@davsclaus
Copy link
Contributor

davsclaus commented Jan 5, 2026

Its not really the text field that should be limited to 200, but each search result entry that is shown. Each result has a title and a description (smaller text). The description is too large and should be limited to 200 chars (etc) so the drop down can show more results and the description is not overly verbose and take up too much space.

@GaneshPatil7517
Copy link
Contributor Author

GaneshPatil7517 commented Jan 5, 2026

Its not really the text field that should be limited to 200, but each search result entry that is shown. Each result has a title and a description (smaller text). The description is too large and should be limited to 200 chars (etc) so the drop down can show more results and the description is not overly verbose and take up too much space.

ok ill work on it...

@GaneshPatil7517
fix(search): limit search result snippets to 200 chars and increase r…
f9ed661 
…esult cap

Truncates each search result description to 200 characters

Prevents result list overflow that hides other hits

Updated search result return limit from 5 to 10

Added CSS line-clamp for cleaner result display
@GaneshPatil7517 GaneshPatil7517 force-pushed the fix-searchbox-char-limit branch from d7f5915 to f9ed661 Compare January 5, 2026 11:13
@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2026

🚀 Preview is available at https://pr-1462--camel.netlify.app

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2026

🚀 Preview is available at https://pr-1462--camel.netlify.app

@GaneshPatil7517
Copy link
Contributor Author

GaneshPatil7517 commented Jan 5, 2026

Hey @davsclaus & @oscerd all checks are passed successfully very excited to see this PR merged.....

@davsclaus davsclaus merged commit 63a5c45 into apache:main Jan 5, 2026
1 check passed
@GaneshPatil7517
Copy link
Contributor Author

GaneshPatil7517 commented Jan 5, 2026

Thank you so much @davsclaus ....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@davsclaus davsclaus davsclaus approved these changes

@oscerd oscerd oscerd approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Search box - Cut text entry to a max number of chars so the search box does not get big and you cannot see other results

3 participants

@GaneshPatil7517 @davsclaus @oscerd