Fix SenseCard horizontal overflow in MergeDupsCompleted WordCard#4206
Draft
Fix SenseCard horizontal overflow in MergeDupsCompleted WordCard#4206
Conversation
- Remove minWidth: 150 from SenseCard to prevent overflow when container is narrow - Add overflowX: hidden to WordCard's content div as a safety measure Co-authored-by: imnasnainaec <6411521+imnasnainaec@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix horizontal overflow in MergeChange component
Fix SenseCard horizontal overflow in MergeDupsCompleted WordCard
Mar 16, 2026
|
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4206 +/- ##
==========================================
- Coverage 75.15% 66.34% -8.81%
==========================================
Files 302 246 -56
Lines 11099 6254 -4845
Branches 1394 791 -603
==========================================
- Hits 8341 4149 -4192
+ Misses 2357 1846 -511
+ Partials 401 259 -142
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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
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.
In Data Cleanup's completed Merge Duplicates view,
SenseCardcould overflow itsWordCardhorizontally when the word card was narrow — producing an unwanted horizontal scrollbar.Root cause:
SenseCardhadminWidth: 150. With MUICardContent's 16px left/right padding, aWordCard~190px wide leaves only ~158px of usable space, but the sense card withm: 1margins needed ≥166px minimum.Changes (
MergeDupsCompleted.tsx)minWidth: 150fromSenseCard— sense card should size to available width;maxWidth: 300is retained to cap width in wider layoutsoverflowX: "hidden"toWordCardcontent div — guards against any residual horizontal overflowOriginal prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.
This change is