Merge pull request #44031 from nextcloud/fix/app-settings-markdown

fix(settings): Allow to limit markdown heading levels to ensure correct hierarchy

apps/settings/src/components/AppDetails.vue

@@ -142,7 +142,7 @@
 				target="_blank"
 				rel="noreferrer noopener">{{ t('settings', 'Developer documentation') }} ↗</a>
 		</p>
-		<Markdown class="app-details__description" :text="app.description" />
+		<Markdown class="app-details__description" :min-heading="2" :text="app.description" />
 	</div>
 </template>
 

apps/settings/src/components/Markdown.vue

@@ -35,6 +35,10 @@ export default {
 			type: String,
 			default: '',
 		},
+		minHeading: {
+			type: Number,
+			default: 1,
+		},
 	},
 	computed: {
 		renderMarkdown() {
@@ -60,6 +64,10 @@ export default {
 				out += '>' + text + '</a>'
 				return out
 			}
+			renderer.heading = (text, level) => {
+				level = Math.min(6, level + (this.minHeading - 1))
+				return `<h${level}>${text}</h${level}>`
+			}
 			renderer.image = function(href, title, text) {
 				if (text) {
 					return text

apps/settings/src/views/Apps.vue

@@ -136,7 +136,7 @@
 				:order="1">
 				<div v-for="release in app.releases" :key="release.version" class="app-sidebar-tabs__release">
 					<h2>{{ release.version }}</h2>
-					<Markdown v-if="changelog(release)" :text="changelog(release)" />
+					<Markdown v-if="changelog(release)" :min-heading="3" :text="changelog(release)" />
 				</div>
 			</NcAppSidebarTab>
 		</NcAppSidebar>