Merge pull request 'fix #301 #301 #311' (#329) from bugfix/issue326-menudark2 into develop
Some checks are pending
ci/woodpecker/push/woodpecker Pipeline is pending
Some checks are pending
ci/woodpecker/push/woodpecker Pipeline is pending
Reviewed-on: #329
This commit is contained in:
commit
01c782d844
|
@ -83,6 +83,12 @@
|
|||
padding-left: 12px !important;
|
||||
margin-left: 5px !important;
|
||||
margin-left: 3px !important;
|
||||
overflow: hidden;
|
||||
}
|
||||
|
||||
.side-menu-opener span {
|
||||
position: relative;
|
||||
left: 50px;
|
||||
}
|
||||
|
||||
.side-menu-opener:active, .side-menu-opener:focus {
|
||||
|
|
|
@ -11,7 +11,7 @@
|
|||
"stylelint:fix": "./node_modules/.bin/stylelint src --fix"
|
||||
},
|
||||
"dependencies": {
|
||||
"axios": "^0.24.0",
|
||||
"axios": "^1.6.7",
|
||||
"trim": "^1.0.1",
|
||||
"vue": "^2.6.11"
|
||||
},
|
||||
|
|
|
@ -168,6 +168,7 @@ $header-icon-size: 20px;
|
|||
flex-shrink: 1;
|
||||
flex-wrap: wrap;
|
||||
}
|
||||
|
||||
.app-menu-main {
|
||||
display: flex;
|
||||
flex-wrap: nowrap;
|
||||
|
@ -177,7 +178,6 @@ $header-icon-size: 20px;
|
|||
height: 50px;
|
||||
position: relative;
|
||||
display: flex;
|
||||
opacity: .7;
|
||||
|
||||
&.app-menu-entry__active {
|
||||
opacity: 1;
|
||||
|
@ -217,6 +217,7 @@ $header-icon-size: 20px;
|
|||
width: $header-icon-size;
|
||||
height: $header-icon-size;
|
||||
padding: calc((100% - $header-icon-size) / 2);
|
||||
box-sizing: content-box;
|
||||
filter: var(--background-image-invert-if-bright, var(--primary-invert-if-bright));
|
||||
}
|
||||
|
||||
|
@ -226,8 +227,8 @@ $header-icon-size: 20px;
|
|||
font-size: 12px;
|
||||
color: var(--color-primary-text);
|
||||
text-align: center;
|
||||
bottom: -5px;
|
||||
left: 50%;
|
||||
top: 45%;
|
||||
display: block;
|
||||
min-width: 100%;
|
||||
transform: translateX(-50%);
|
||||
|
@ -235,6 +236,7 @@ $header-icon-size: 20px;
|
|||
width: 100%;
|
||||
text-overflow: ellipsis;
|
||||
overflow: hidden;
|
||||
letter-spacing: -0.5px;
|
||||
}
|
||||
|
||||
&:not(.app-menu-entry__hidden-label):not(.app-menu-entry__show-hovered):hover,
|
||||
|
@ -242,11 +244,11 @@ $header-icon-size: 20px;
|
|||
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;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
@ -263,8 +265,10 @@ $header-icon-size: 20px;
|
|||
&:not(.app-menu-main__hidden-label):not(.app-menu-main__show-hovered):focus-within,
|
||||
.app-menu-entry:not(.app-menu-entry__hidden-label):hover,
|
||||
.app-menu-entry:not(.app-menu-entry__hidden-label):focus {
|
||||
opacity: 1;
|
||||
|
||||
img {
|
||||
margin-top: -6px;
|
||||
margin-top: -8px;
|
||||
}
|
||||
|
||||
.app-menu-entry--label {
|
||||
|
@ -280,7 +284,7 @@ $header-icon-size: 20px;
|
|||
&.app-menu-main__show-hovered .app-menu-entry:hover,
|
||||
&.app-menu-main__show-hovered .app-menu-entry:focus {
|
||||
img {
|
||||
margin-top: -6px;
|
||||
margin-top: -8px;
|
||||
}
|
||||
|
||||
.app-menu-entry--label {
|
||||
|
@ -295,38 +299,42 @@ $header-icon-size: 20px;
|
|||
}
|
||||
|
||||
::v-deep .app-menu-more .button-vue--vue-tertiary {
|
||||
color: var(--color-primary-text);
|
||||
opacity: .7;
|
||||
margin: 3px;
|
||||
filter: var(--background-image-invert-if-bright, var(--primary-invert-if-bright));
|
||||
|
||||
&:hover {
|
||||
opacity: 1;
|
||||
background-color: transparent !important;
|
||||
&:not([aria-expanded="true"]) {
|
||||
color: var(--color-primary-element-text);
|
||||
|
||||
&:hover {
|
||||
opacity: 1;
|
||||
background-color: transparent !important;
|
||||
}
|
||||
}
|
||||
|
||||
&: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);
|
||||
}
|
||||
&:focus-visible {
|
||||
opacity: 1;
|
||||
outline: none !important;
|
||||
}
|
||||
}
|
||||
|
||||
.app-menu-popover-entry {
|
||||
.app-icon {
|
||||
position: relative;
|
||||
height: 44px;
|
||||
width: 48px;
|
||||
display: flex;
|
||||
align-items: center;
|
||||
justify-content: center;
|
||||
filter: var(--background-invert-if-bright, var(--primary-invert-if-bright));
|
||||
|
||||
&.has-unread::after {
|
||||
background-color: var(--color-main-text);
|
||||
}
|
||||
|
||||
img {
|
||||
filter: var(--background-invert-if-bright, var(--primary-invert-if-bright));
|
||||
width: $header-icon-size;
|
||||
height: $header-icon-size;
|
||||
padding: calc((50px - $header-icon-size) / 2);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
@ -335,7 +343,7 @@ $header-icon-size: 20px;
|
|||
content: "";
|
||||
width: 8px;
|
||||
height: 8px;
|
||||
background-color: var(--color-primary-text);
|
||||
background-color: var(--color-primary-element-text);
|
||||
border-radius: 50%;
|
||||
position: absolute;
|
||||
display: block;
|
||||
|
|
|
@ -15,11 +15,18 @@ You should have received a copy of the GNU Affero General Public License
|
|||
along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||
-->
|
||||
<template>
|
||||
<button class="side-menu-opener side-menu-closer"></button>
|
||||
<button class="side-menu-opener side-menu-closer" :arial-label="label">
|
||||
<span v-text="label"></span>
|
||||
</button>
|
||||
</template>
|
||||
|
||||
<script>
|
||||
export default {
|
||||
name: 'CloserButton',
|
||||
data() {
|
||||
return {
|
||||
label: t('side_menu', 'Close the menu'),
|
||||
}
|
||||
}
|
||||
}
|
||||
</script>
|
||||
|
|
|
@ -15,7 +15,9 @@ You should have received a copy of the GNU Affero General Public License
|
|||
along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||
-->
|
||||
<template>
|
||||
<button class="side-menu-opener" :arial-label="label"></button>
|
||||
<button class="side-menu-opener" :arial-label="label">
|
||||
<span v-text="label"></span>
|
||||
</button>
|
||||
</template>
|
||||
|
||||
<script>
|
||||
|
@ -23,7 +25,7 @@ export default {
|
|||
name: 'OpenerButton',
|
||||
data() {
|
||||
return {
|
||||
label: t('side_menu', 'Toggle menu'),
|
||||
label: t('side_menu', 'Toggle the menu'),
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -95,4 +95,4 @@
|
|||
"Hide labels on mouse over": ""
|
||||
"Except the hovered app": ""
|
||||
"Search": ""
|
||||
"Toggle menu": ""
|
||||
"Toggle the menu": ""
|
||||
|
|
|
@ -3,7 +3,13 @@ module.exports = (tagName, attributes) => {
|
|||
|
||||
if (typeof attributes === 'object') {
|
||||
for (let i in attributes) {
|
||||
element.setAttribute(i, attributes[i])
|
||||
if (i === 'text') {
|
||||
element.textContent = attributes[i]
|
||||
} else if (i === 'html') {
|
||||
element.innerHTML = attributes[i]
|
||||
} else {
|
||||
element.setAttribute(i, attributes[i])
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -16,7 +16,8 @@ if ($_['always-displayed']) {
|
|||
const sideMenuContainer = SMcreateElement('div', {id: 'side-menu-container'})
|
||||
const sideMenuOpener = SMcreateElement('button', {
|
||||
'class': 'side-menu-opener',
|
||||
'arial-label': t('side_menu', 'Toggle menu')
|
||||
'arial-label': t('side_menu', 'Toggle the menu'),
|
||||
'html': `<span>${t('side_menu', 'Toggle the menu')}</span>`
|
||||
})
|
||||
const sideMenu = SMcreateElement('div', {id: 'side-menu'})
|
||||
|
||||
|
@ -38,6 +39,15 @@ if ($_['always-displayed']) {
|
|||
sideMenu.setAttribute('data-sidewithcategories', '1')
|
||||
<?php endif; ?>
|
||||
|
||||
const sideMenuFocus = () => {
|
||||
let a = document.querySelector('#side-menu .side-menu-app.active a')
|
||||
|| document.querySelector('#side-menu .side-menu-app a')
|
||||
|
||||
if (a) {
|
||||
a.focus()
|
||||
}
|
||||
}
|
||||
|
||||
document.querySelector('body').addEventListener('side-menu.apps', (e) => {
|
||||
const apps = e.detail.apps;
|
||||
|
||||
|
@ -80,22 +90,6 @@ if ($_['always-displayed']) {
|
|||
return
|
||||
}
|
||||
|
||||
sideMenuFocus = () => {
|
||||
let a = document.querySelector('.side-menu-app.active a', sideMenu)
|
||||
|
||||
if (!a) {
|
||||
return
|
||||
}
|
||||
|
||||
if (a.length === 0) {
|
||||
a = sideMenu.querySelector('.side-menu-app:first-child a')
|
||||
}
|
||||
|
||||
if (a.length > 0) {
|
||||
a.focus()
|
||||
}
|
||||
}
|
||||
|
||||
<?php if ($_['opener-hover']): ?>
|
||||
const sideMenuMouseLeave = () => {
|
||||
sideMenu.classList.remove('open')
|
||||
|
@ -127,14 +121,8 @@ if ($_['always-displayed']) {
|
|||
|
||||
headerMenuOpener.addEventListener('click', () => {
|
||||
sideMenu.classList.add('open')
|
||||
|
||||
const a = sideMenu.querySelector('.side-menu-app.active a')
|
||||
|
||||
if (a !== null) {
|
||||
a.focus()
|
||||
}
|
||||
|
||||
headerMenuOpener.blur()
|
||||
sideMenuFocus()
|
||||
})
|
||||
|
||||
for (let opener of sideMenuOpener) {
|
||||
|
|
|
@ -241,6 +241,7 @@ $labelReset = 'Reset to default';
|
|||
<div class="section">
|
||||
<?php if (!$_['force']): ?>
|
||||
<button id="side-menu-save" class="btn btn-info" arial-label="<?php p($l->t('Save')); ?>">
|
||||
<?php p($l->t('Save')); ?>
|
||||
<progress max="100" value="0" id="side-menu-save-progress"></progress>
|
||||
</button>
|
||||
|
||||
|
|
Loading…
Reference in a new issue