Skip to content

Commit bacf539

Browse files
Merge pull request #7893 from nextcloud/fix/talk-modal-scrolling
fix(AddTalkModal): add scrolling for long list
2 parents fba6cf5 + 7bb1fde commit bacf539

1 file changed

Lines changed: 22 additions & 14 deletions

File tree

src/components/Editor/AddTalkModal.vue

Lines changed: 22 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -19,20 +19,22 @@
1919
v-else-if="talkConversations.length === 0"
2020
:description="t('calendar', 'No Talk room available')" />
2121
<NcFormGroup v-else :label="t('calendar', 'Select existing conversation')">
22-
<NcListItem
23-
v-for="conversation in sortedTalkConversations"
24-
:key="conversation.id"
25-
:class="{ selected: selectedRoom && selectedRoom.id === conversation.id }"
26-
:name="conversation.displayName"
27-
class="talk-room-list__item"
28-
@click="selectRoom(conversation)">
29-
<template #icon>
30-
<NcAvatar
31-
:url="avatarUrl(conversation)"
32-
:size="28"
33-
:disable-tooltip="true" />
34-
</template>
35-
</NcListItem>
22+
<div class="talk-room__wrapper">
23+
<NcListItem
24+
v-for="conversation in sortedTalkConversations"
25+
:key="conversation.id"
26+
:class="{ selected: selectedRoom && selectedRoom.id === conversation.id }"
27+
:name="conversation.displayName"
28+
class="talk-room-list__item"
29+
@click="selectRoom(conversation)">
30+
<template #icon>
31+
<NcAvatar
32+
:url="avatarUrl(conversation)"
33+
:size="28"
34+
:disable-tooltip="true" />
35+
</template>
36+
</NcListItem>
37+
</div>
3638
<NcButton
3739
variant="secondary"
3840
class="talk_select-room"
@@ -277,4 +279,10 @@ export default {
277279
.spacer {
278280
margin-top: calc(var(--default-grid-baseline) * 6);
279281
}
282+
283+
.talk-room__wrapper {
284+
max-height: 400px;
285+
overflow-y: auto;
286+
overflow-x: clip;
287+
}
280288
</style>

0 commit comments

Comments
 (0)