Skip to content

Commit

Permalink
Merge pull request #1897 from nextcloud/fix-appnavigationitem-layout
Browse files Browse the repository at this point in the history
Fix appnavigationitem layout
  • Loading branch information
marcoambrosini authored Apr 30, 2021
2 parents 89d9e33 + a08dcf9 commit f9bb5d8
Showing 1 changed file with 11 additions and 6 deletions.
17 changes: 11 additions & 6 deletions src/components/AppNavigationItem/AppNavigationItem.vue
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,10 @@ Just set the `pinned` prop.

<!-- Counter and Actions -->
<div v-if="hasUtils" class="app-navigation-entry__utils">
<slot name="counter" />
<div v-if="$slots.counter"
class="app-navigation-entry__counter-wrapper">
<slot name="counter" />
</div>
<Actions menu-align="right"
:placement="menuPlacement"
:open="menuOpen"
Expand Down Expand Up @@ -448,6 +451,7 @@ export default {
box-sizing: border-box;
width: 100%;
min-height: $clickable-area;
padding: 4px;
// When .active class is applied, change color background of link and utils. The
// !important prevents the focus state to override the active state.
Expand Down Expand Up @@ -582,11 +586,12 @@ export default {
display: flex;
align-items: center;
flex: 0 1 auto;
// visually balance the menu so it's not
// stuck to the scrollbar
.action-item {
margin-right: 2px;
}
}
/* counter */
.app-navigation-entry__counter-wrapper {
// Add slightly more space to the right of the counter
margin-right: 2px;
}
// STATES
Expand Down

0 comments on commit f9bb5d8

Please sign in to comment.