Skip to content

Commit c55be64

Browse files
committed
fix(frontend): remove fixed height from ManySelect
Frontend development is not my passion, there might be a better fix. I've tested my changes in all places that I found using the chganed components (ManySelect, ScrollablePanel): - Changelog filters - Version filters - Download dialog - Search filters Fixes modrinth#2334
1 parent e81e056 commit c55be64

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

packages/ui/src/components/base/ManySelect.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
"
3232
/>
3333
</div>
34-
<ScrollablePanel v-if="search" class="h-[17rem]">
34+
<ScrollablePanel v-if="search">
3535
<Button
3636
v-for="(option, index) in filteredOptions"
3737
:key="`option-${index}`"

packages/ui/src/components/base/ScrollablePanel.vue

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,7 @@ function onScroll({ target: { scrollTop, offsetHeight, scrollHeight } }) {
113113
overflow-y: auto;
114114
overflow-x: hidden;
115115
position: relative;
116+
max-height: 19rem;
116117
117118
::-webkit-scrollbar {
118119
transition: all;

0 commit comments

Comments
 (0)