line breaks in tooltips & text #8783
Conversation
2 - line breaks for tooltips and text wrapped
guillim
changed the title
![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 enhances text display functionality by implementing proper line breaks in tooltips and text wrapping, while cleaning up unused variables across components.
- Added
white-space: pre-wrapstyling inpackages/twenty-ui/src/display/tooltip/OverflowingTextWithTooltip.tsxto preserve line breaks - Added
<Styledpre>component inOverflowingTextWithTooltipfor proper multiline tooltip formatting - Removed unused
isLabelandallowDisplayWrapprops fromOverflowingTextWithTooltipcomponent - Added
isTooltipMultiline={true}inpackages/twenty-front/src/modules/ui/field/display/components/TextDisplay.tsx - Modified text wrapping behavior with
-webkit-line-clampand-webkit-box-orientfor better overflow handling
3 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile
| }: { | ||
| size?: 'large' | 'small'; | ||
| text: string | null | undefined; | ||
| isTooltipMultiline?: boolean; | ||
| displayedMaxRows?: number; | ||
| isLabel?: boolean; | ||
| allowDisplayWrap?: boolean; | ||
| }) => { | ||
| const textElementId = `title-id-${+new Date()}`; |
There was a problem hiding this comment.
logic: Using timestamp for ID generation can cause duplicate IDs if multiple tooltips are created in the same millisecond. Consider using a more reliable unique ID generation method.
|
fixes #8770 |
|
Also added the request from @Bonapara to align the top the copy/paste button (before was middle aligned |
|
Thanks @guillim for your contribution! |
This feature goal is to :
1 - clean variables not used any longer
2 - add line breaks for tooltips and text wrapped
Request From @Bonapara