fix: let commit message wrap instead of truncating with ellipsis#1189
Open
alpurkan17 wants to merge 2 commits into
Open
fix: let commit message wrap instead of truncating with ellipsis#1189alpurkan17 wants to merge 2 commits into
alpurkan17 wants to merge 2 commits into
Conversation
added 2 commits
May 14, 2026 08:19
…1054) ## Summary Add ClearSearchAdornment (clear button) to the Score Breakdown search field on the Miner Overview tab, matching the existing pattern used in all other search inputs across the app. ## Root Cause The DebouncedSearchInput in MinerScoreBreakdown renders a TextField with startAdornment (search icon) but no endAdornment, unlike MinerPRsTable, MinerOpenDiscoveryIssuesByRepo, IssuesList, WatchlistPage, TopRepositoriesTable, and MinerRepositoriesTable which all use ClearSearchAdornment. ## Impact Users on the Score Breakdown view had no one-click way to clear their search query — an inconsistency with all other searchable views in the app. ## Related Issues Fixes entrius#1054 ## Test plan ### How to verify - Open any miner profile with PRs in score breakdown - Type in the search field — a clear (X) button should appear - Click clear — search text should be cleared and results reset - Existing tests pass (ClearSearchAdornment already tested) ### Edge cases - Empty search: clear button hidden (visible={Boolean(draftValue)}) - Mobile: clear button works same as desktop ### Post-merge - All app search fields now have consistent clear behavior
Removes nowrap/overflow/textOverflow/maxWidth constraints so long commit subjects wrap naturally. Adds wordBreak: break-word for long tokens. Keeps layout stable with minWidth: 0 on flex parent.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The commit message bar in
RepositoryCodeBrowserusednowrap+overflow: hidden+textOverflow: ellipsis+maxWidth: 600px, which truncated long commit subjects with no way to read the rest.Changes
TypographywordBreak: 'break-word'for long tokens/URLsoverflow: hiddenwithminWidth: 0on the flex parent for proper shrink behaviorRelated Issues
Closes #1136
Type of Change
Checklist
npm run formatandnpm run lint:fixhave been runnpm run buildpasses