From 54cc0d7130a21a63600578099f6da8d35d0f7a74 Mon Sep 17 00:00:00 2001 From: "@suet-kei.chan" Date: Wed, 28 Jan 2026 09:52:34 +0100 Subject: [PATCH 1/7] style: show storage quota and settings, hide WebDAV/warnings/shortcuts sections --- css/apps/files.scss | 9 +++++++++ css/components/ncappnavigation.scss | 18 ++++++------------ 2 files changed, 15 insertions(+), 12 deletions(-) diff --git a/css/apps/files.scss b/css/apps/files.scss index e5b80c6..2d0d001 100644 --- a/css/apps/files.scss +++ b/css/apps/files.scss @@ -1225,6 +1225,15 @@ table.files-filestable { } } +// Hide WebDAV, Warnings, and Keyboard shortcuts sections in Files settings +.app-settings__content { + #settings-section_webdav, + #settings-section_warning, + #settings-section_shortcuts { + display: none; + } +} + // Deleted Files list table and header #app-content-vue { .files-list__header { diff --git a/css/components/ncappnavigation.scss b/css/components/ncappnavigation.scss index 3228b5e..5a722f5 100644 --- a/css/components/ncappnavigation.scss +++ b/css/components/ncappnavigation.scss @@ -86,21 +86,15 @@ } .app-navigation-entry__settings { - padding: 0 0 1rem; - - > li { - display: none; - } } - &__content &-entry__settings { - - // hide nextcloud quota and files settings links - .app-navigation-entry-wrapper { - display: none; - } - } + // Show nextcloud quota and files settings links + // &__content &-entry__settings { + // .app-navigation-entry-wrapper { + // display: none; + // } + // } } // default navbar link styling From 2eb3e04f6b6ca2037953b3bb9c2cbb3428c29fb8 Mon Sep 17 00:00:00 2001 From: "@suet-kei.chan" Date: Thu, 29 Jan 2026 01:19:49 +0100 Subject: [PATCH 2/7] fix: add webpack DefinePlugin to resolve process is not defined error --- webpack.config.cjs | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/webpack.config.cjs b/webpack.config.cjs index 3659850..480d04e 100644 --- a/webpack.config.cjs +++ b/webpack.config.cjs @@ -1,5 +1,6 @@ // webpack with standard nextcloud config const path = require('path') +const webpack = require('webpack') const webpackConfig = require('@nextcloud/webpack-vue-config') const webpackRules = require('@nextcloud/webpack-vue-config/rules') @@ -29,4 +30,14 @@ webpackConfig.module.rules = Object.values(webpackRules) // Workaround for https://github.com/nextcloud/webpack-vue-config/pull/432 causing problems with nextcloud-vue-collections webpackConfig.resolve.alias = {} +// Fix process is not defined error +webpackConfig.plugins.push( + new webpack.DefinePlugin({ + 'process.env': '{}', + 'process.version': '""', + 'process.platform': '""', + 'process.browser': 'true', + }) +) + module.exports = webpackConfig From fa9a682468241726c8e03178305d662fb8c7235f Mon Sep 17 00:00:00 2001 From: "@suet-kei.chan" Date: Thu, 29 Jan 2026 02:03:24 +0100 Subject: [PATCH 3/7] style: hide default storage quota in Files navigation settings --- css/components/ncappnavigation.scss | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/css/components/ncappnavigation.scss b/css/components/ncappnavigation.scss index 5a722f5..64ff37d 100644 --- a/css/components/ncappnavigation.scss +++ b/css/components/ncappnavigation.scss @@ -87,14 +87,17 @@ .app-navigation-entry__settings { padding: 0 0 1rem; + + > li { + display: none; + } } - // Show nextcloud quota and files settings links - // &__content &-entry__settings { - // .app-navigation-entry-wrapper { - // display: none; - // } - // } + &__content &-entry__settings { + .app-navigation-entry-wrapper { + display: none; + } + } } // default navbar link styling From 36392e4355435e1e63949b9a12ea23436c5b655b Mon Sep 17 00:00:00 2001 From: "@suet-kei.chan" Date: Thu, 29 Jan 2026 10:26:55 +0100 Subject: [PATCH 4/7] style: add comment for hidden navigation settings --- css/components/ncappnavigation.scss | 3 +++ 1 file changed, 3 insertions(+) diff --git a/css/components/ncappnavigation.scss b/css/components/ncappnavigation.scss index 64ff37d..e414394 100644 --- a/css/components/ncappnavigation.scss +++ b/css/components/ncappnavigation.scss @@ -86,6 +86,7 @@ } .app-navigation-entry__settings { + padding: 0 0 1rem; > li { @@ -94,6 +95,8 @@ } &__content &-entry__settings { + + // hide nextcloud quota and files settings links .app-navigation-entry-wrapper { display: none; } From d375f4f4b4d45878535898816b192954fde98d1f Mon Sep 17 00:00:00 2001 From: "@suet-kei.chan" Date: Thu, 29 Jan 2026 19:23:04 +0100 Subject: [PATCH 5/7] style: improve modal responsiveness and checkbox hover states - Add responsive breakpoints for modals on mobile and tablet devices - Update checkbox/radio hover and focus background color to be lighter - Remove bottom margins from settings sections for better spacing - Reduce modal padding on small screens for better mobile UX --- css/apps/photos.scss | 12 +++++++++++- css/components/ncappnavigation.scss | 1 - css/components/nccheckbox.scss | 6 ++++-- css/layouts/modal.scss | 30 ++++++++++++++++++++++++++++- 4 files changed, 44 insertions(+), 5 deletions(-) diff --git a/css/apps/photos.scss b/css/apps/photos.scss index 64b0997..9a8ee7b 100644 --- a/css/apps/photos.scss +++ b/css/apps/photos.scss @@ -1273,7 +1273,7 @@ } .app-settings-section { - margin-bottom: 2rem; + margin-bottom: 0; } #settings-section_layout-settings, @@ -1285,6 +1285,16 @@ // Device Sessions modal window #body-settings .modal-wrapper--small > .modal-container { width: 550px; + + @media screen and (max-width: $breakpoint-mobile-medium) { + width: 90vw; + max-width: 500px; + } + + @media screen and (max-width: $breakpoint-mobile-small) { + width: 95vw; + max-width: none; + } } // Share Album modal window diff --git a/css/components/ncappnavigation.scss b/css/components/ncappnavigation.scss index e414394..ae4c4b2 100644 --- a/css/components/ncappnavigation.scss +++ b/css/components/ncappnavigation.scss @@ -86,7 +86,6 @@ } .app-navigation-entry__settings { - padding: 0 0 1rem; > li { diff --git a/css/components/nccheckbox.scss b/css/components/nccheckbox.scss index 6e04882..7b7406f 100644 --- a/css/components/nccheckbox.scss +++ b/css/components/nccheckbox.scss @@ -102,8 +102,10 @@ span.checkbox-radio-switch-radio.checkbox-radio-switch, .checkbox-radio-switch-c } &--checked:not(&--disabled):focus-within .checkbox-radio-switch__label, - &--checked:not(&--disabled) .checkbox-radio-switch__label:hover { - background-color: initial; + &--checked:not(&--disabled) .checkbox-radio-switch__label:hover, + &--checked:not(&--disabled):focus-within .checkbox-radio-switch__content, + &--checked:not(&--disabled) .checkbox-radio-switch__content:hover { + background-color: var(--color-background-hover); } // override min-height of checkbox diff --git a/css/layouts/modal.scss b/css/layouts/modal.scss index 2420c06..0c6689a 100644 --- a/css/layouts/modal.scss +++ b/css/layouts/modal.scss @@ -8,6 +8,7 @@ */ @import '../_mixins.scss'; +@import '../_variables.scss'; #body-user .dialog__modal { @@ -19,6 +20,10 @@ &__content { padding-inline: 1.5rem; + @media screen and (max-width: $breakpoint-mobile-small) { + padding-inline: 1rem; + } + .button-vue--vue-secondary { background: var(--nmc-ods-blue-primary); border-radius: var(--border-radius-pill); @@ -60,7 +65,7 @@ } .app-settings-section__name, .app-settings-section__content, .setting-section-subline, .photos-locations { - margin: 0 0 0.75rem; + margin: 0; } } } @@ -68,6 +73,29 @@ .modal-wrapper--large > .modal-container { width: 650px; + + @media screen and (max-width: $breakpoint-mobile-medium) { + width: 90vw; + max-width: 600px; + } + + @media screen and (max-width: $breakpoint-mobile-small) { + width: 95vw; + max-width: none; + } + } + + .modal-container { + @media screen and (max-width: $breakpoint-mobile-small) { + margin: 1rem; + } + + .dialog__name { + @media screen and (max-width: $breakpoint-mobile-small) { + font-size: 1.25rem; + padding: 1rem 1rem 0.5rem; + } + } } &#unified-search .modal-container .dialog__name { From 3d23ca942f931a4ad46f2f169d54769cd5233178 Mon Sep 17 00:00:00 2001 From: "@suet-kei.chan" Date: Thu, 29 Jan 2026 19:25:21 +0100 Subject: [PATCH 6/7] style: add spacing in app navigation settings selector --- css/components/ncappnavigation.scss | 1 + 1 file changed, 1 insertion(+) diff --git a/css/components/ncappnavigation.scss b/css/components/ncappnavigation.scss index ae4c4b2..3228b5e 100644 --- a/css/components/ncappnavigation.scss +++ b/css/components/ncappnavigation.scss @@ -86,6 +86,7 @@ } .app-navigation-entry__settings { + padding: 0 0 1rem; > li { From 07333a1a063f44adad990d0c37ab2f1c262a7dd3 Mon Sep 17 00:00:00 2001 From: "@suet-kei.chan" Date: Thu, 29 Jan 2026 20:32:01 +0100 Subject: [PATCH 7/7] fix: enable scrolling in Files settings modal Simplify flexbox layout to allow content scrolling while preserving close button functionality. --- css/layouts/modal.scss | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/css/layouts/modal.scss b/css/layouts/modal.scss index 280c183..5fad7a3 100644 --- a/css/layouts/modal.scss +++ b/css/layouts/modal.scss @@ -16,15 +16,21 @@ padding: 0; .dialog { - overflow: visible; + max-height: 80vh; + display: flex; + flex-direction: column; &__wrapper { - overflow: visible; + max-height: 80vh; + display: flex; + flex-direction: column; + overflow: hidden; } &__content { - overflow: visible; + overflow-y: auto; padding-inline: 1.5rem; + padding-bottom: 1.5rem; @media screen and (max-width: $breakpoint-mobile-small) { padding-inline: 1rem;