Commit 676bd228 authored by Mark Florian's avatar Mark Florian

Avoid scrollbar in more maintainable way

What this (and the previous) approach aims to achieve is *not* to show
a scrollbar in the severity filter dropdown. It's a fixed list of
severities, and inside a `dropdown-content`, the dropdown items are only
*just* tall enough to be taller than the max-height of
`dropdown-content`, making it render with a scrollbar, looking terrible.

The report type filter dropdown is also a fixed list of only four items,
so also doesn't need an internal scrollbar.

The projects filter dropdown, however, is not a fixed list, and requires
the internal scrollbar provided by `dropdown-content`.
parent 8575ee72
...@@ -91,7 +91,7 @@ export default { ...@@ -91,7 +91,7 @@ export default {
:placeholder="__('Filter...')" :placeholder="__('Filter...')"
/> />
<div class="dropdown-content" style="max-height: 280px"> <div :class="{ 'dropdown-content': filterId === 'project_id' }">
<button <button
v-for="option in filteredOptions" v-for="option in filteredOptions"
:key="option.id" :key="option.id"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment