Skip to content

Commit

Permalink
Merge pull request #23396 from ahmedGaber93/issue-22951
Browse files Browse the repository at this point in the history
fix MultipleAvatars.js remainding users tooltip text
  • Loading branch information
madmax330 authored Jul 25, 2023
2 parents 6e6d2bd + c252504 commit a426cca
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/components/MultipleAvatars.js
Original file line number Diff line number Diff line change
Expand Up @@ -209,8 +209,8 @@ function MultipleAvatars(props) {
))}
{avatars.length > props.maxAvatarsInRow && (
<Tooltip
// We only want to cap tooltips to only the first 10 users or so since some reports have hundreds of users, causing performance to degrade.
text={tooltipTexts.slice(3, 10).join(', ')}
// We only want to cap tooltips to only 10 users or so since some reports have hundreds of users, causing performance to degrade.
text={tooltipTexts.slice(avatarRows.length * props.maxAvatarsInRow - 1, avatarRows.length * props.maxAvatarsInRow + 9).join(', ')}
>
<View
style={[
Expand Down

0 comments on commit a426cca

Please sign in to comment.