Skip to content

Commit 4b20fa3

Browse files
authored
Merge pull request #3191 from nextcloud/fix/popover
Fix the NcPopoverMenu component
2 parents 93acbc6 + 6064555 commit 4b20fa3

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

src/components/NcPopoverMenu/NcPopoverMenu.vue

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,5 +58,6 @@ export default {
5858
ul {
5959
display: flex;
6060
flex-direction: column;
61+
gap: 4px;
6162
}
6263
</style>

src/components/NcPopoverMenu/NcPopoverMenuItem.vue

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -179,6 +179,7 @@ export default {
179179

180180
<style scoped>
181181
button.menuitem {
182+
border-radius: var(--border-radius-large) !important;
182183
text-align: left;
183184
}
184185
@@ -196,8 +197,8 @@ export default {
196197
}
197198
198199
.menuitem.active {
199-
box-shadow: inset 2px 0 var(--color-primary);
200-
border-radius: 0;
200+
border-left: 4px solid var(--color-primary);
201+
border-radius: 0 var(--border-radius-large) var(--border-radius-large) 0 !important;
201202
}
202203
</style>
203204

@@ -216,7 +217,6 @@ li {
216217
cursor: pointer;
217218
line-height: $clickable-area;
218219
border: 0;
219-
border-radius: 0; // otherwise Safari will cut the border-radius area
220220
background-color: transparent;
221221
display: flex;
222222
align-items: flex-start;

0 commit comments

Comments
 (0)