Skip to content

Commit

Permalink
Merge pull request #35501 from nextcloud/enh/enh-33741-Focused_item_o…
Browse files Browse the repository at this point in the history
…verlaps_neighbor_elements_with_text_and_border

Change app menu focused items behavior that neighbor elements (text and border) have no overlapping
  • Loading branch information
szaimen authored Nov 30, 2022
2 parents 2adb1f8 + db67670 commit d8d15c3
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 12 deletions.
18 changes: 9 additions & 9 deletions core/src/components/AppMenu.vue
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@
<a :href="app.href"
:class="{ 'has-unread': app.unread > 0 }"
:aria-label="appLabel(app)"
:title="app.name"
:aria-current="app.active ? 'page' : false">
<img :src="app.icon" alt="">
<div class="app-menu-entry--label">
Expand Down Expand Up @@ -195,25 +196,27 @@ $header-icon-size: 20px;
text-align: center;
bottom: -5px;
left: 50%;
top: 45%;
display: block;
min-width: 100%;
transform: translateX(-50%);
transition: all 0.1s ease-in-out;
width: 100%;
text-overflow: ellipsis;
overflow: hidden;
letter-spacing: -0.5px;
}
&:hover,
&:focus-within {
opacity: 1;
.app-menu-entry--label {
opacity: 1;
font-weight: bold;
font-size: 14px;
font-weight: bolder;
bottom: 0;
width: auto;
overflow: visible;
width: 100%;
text-overflow: ellipsis;
overflow: hidden;
}
}
Expand All @@ -227,7 +230,7 @@ $header-icon-size: 20px;
opacity: 1;
img {
margin-top: -6px;
margin-top: -8px;
}
.app-menu-entry--label {
Expand All @@ -254,10 +257,7 @@ $header-icon-size: 20px;
&:focus-visible {
opacity: 1;
background-color: transparent !important;
border-radius: var(--border-radius);
outline: none;
box-shadow: 0 0 0 2px var(--color-primary-text);
outline: none !important;
}
}
Expand Down
4 changes: 2 additions & 2 deletions dist/core-main.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/core-main.js.map

Large diffs are not rendered by default.

0 comments on commit d8d15c3

Please sign in to comment.