Skip to content

Commit

Permalink
Merge pull request #10369 from nextcloud/backport/10356/stable27
Browse files Browse the repository at this point in the history
[stable27] fix(LeftSidebar): wrong user status after scrolling
  • Loading branch information
Antreesy authored Aug 25, 2023
2 parents dbba07f + ea84245 commit 0e0b98e
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions src/components/ConversationIcon.vue
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,11 @@
:height="size"
:alt="item.displayName"
class="avatar icon">
<!-- NcAvatar doesn't fully support props update and works only for 1 user -->
<!-- Using key on NcAvatar forces NcAvatar re-mount and solve the problem, could not really optimal -->
<!-- TODO: Check if props update support in NcAvatar is more performant -->
<NcAvatar v-else
:key="item.token"
:size="size"
:user="item.name"
:disable-menu="disableMenu"
Expand All @@ -48,14 +52,12 @@
class="conversation-icon__avatar" />
<div v-if="showCall"
class="overlap-icon">
<VideoIcon :size="20"
:fill-color="'#E9322D'" />
<VideoIcon :size="20" :fill-color="'#E9322D'" />
<span class="hidden-visually">{{ t('spreed', 'Call in progress') }}</span>
</div>
<div v-else-if="showFavorite"
class="overlap-icon">
<Star :size="20"
:fill-color="'#FFCC00'" />
<Star :size="20" :fill-color="'#FFCC00'" />
<span class="hidden-visually">{{ t('spreed', 'Favorite') }}</span>
</div>
</div>
Expand Down

0 comments on commit 0e0b98e

Please sign in to comment.