Skip to content
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

Truncate long search result lines #1744

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

joelit
Copy link
Contributor

@joelit joelit commented Jan 23, 2025

Link to relevant issue(s), if any

Closes #1528

Description of the changes in this PR

Initial proof-of concept with tests and some functionality missing

Checklist

  • phpUnit tests pass locally with my changes
  • I have added tests that show that the new code works, or tests are not relevant for this PR (e.g. only HTML/CSS changes)
  • The PR doesn't reduce accessibility of the front-end code (e.g. tab focus, scaling to different resolutions, use of .sr-only class, color contrast)
  • The PR doesn't introduce unintended code changes (e.g. empty lines or useless reindentation)

@joelit joelit added this to the 3.0 milestone Jan 23, 2025
@joelit joelit self-assigned this Jan 23, 2025
@joelit joelit mentioned this pull request Jan 23, 2025
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

Long search result lines should be truncated
1 participant