[FIX] Text overflow on timeline #8325
Conversation
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR Summary
This PR fixes text overflow issues in the timeline summary component by implementing proper width constraints and text truncation with ellipsis for long content.
- Added
width: 100%toStyledSummaryin/packages/twenty-front/src/modules/activities/timeline-activities/components/EventRow.tsxto prevent container expansion - Implemented text truncation in
/packages/twenty-front/src/modules/activities/timeline-activities/rows/activity/components/EventRowActivity.tsxwithtext-overflow: ellipsisandwhite-space: nowrap - Added
overflow: hiddento prevent horizontal scrolling on long task titles and bodies
2 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile
| width: 100%; | ||
| overflow: hidden; | ||
| text-overflow: ellipsis; | ||
| white-space: nowrap; |
There was a problem hiding this comment.
style: Consider adding max-width property to ensure the text doesn't push other elements out of position
|
Looking good! Can you check why some values are not taking the full width?
|
|
Is there a bug with the timeline? @Bonapara https://www.loom.com/share/de72e092dc024af89e27f0ff2d334ae3?sid=2b14e0f7-4731-4e33-a584-b180942388f0 The timeline doesn't show changes in local setup too... |
|
I had the same problem @Hitarthsheth07 but running the worker solved my issue: |
Gotcha! |
Did it work? |
Havent tried it yet! I'm travelling right now. Will get back to you in about an hour But why is it not working on the demo site then? |
FIX: #6977
Implementation:
text-overflow: ellipseswhen overflown.