Merge pull request #4825 from nextcloud-libraries/fix/41504-Create_sc…

…rollable_content_over_sidebar_and_settings_dialog_on_small_screens_ Create scrollable content via `min-height` over sidebar and settings …
nextcloud-libraries · Nov 16, 2023 · 5db6b10 · 5db6b10
2 parents 1c43bdc + 73fa6b1
commit 5db6b10
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/components/NcAppSettingsDialog/NcAppSettingsDialog.vue b/src/components/NcAppSettingsDialog/NcAppSettingsDialog.vue
@@ -395,6 +395,9 @@ export default {
 
 <style lang="scss" scoped>
 .app-settings {
+	&:deep(.dialog) {
+		min-height: 256px;
+	}
 	&__navigation {
 		min-width: 200px;
 		margin-right: 20px;
@@ -409,6 +412,7 @@ export default {
 		overflow-x: hidden;
 		padding: 24px;
 		width: 100%;
+		min-height: 256px;
 	}
 }
 

diff --git a/src/components/NcAppSidebar/NcAppSidebarTabs.vue b/src/components/NcAppSidebar/NcAppSidebarTabs.vue
@@ -323,8 +323,7 @@ export default {
 
 	&__content {
 		position: relative;
-		// take full available height
-		min-height: 0;
+		min-height: 256px;
 		height: 100%;
 		// force the use of the tab component if more than one tab
 		// you can just put raw content if you don't use tabs