Skip to content

Commit

Permalink
Merge pull request #42742 from nextcloud/fix/41826-avoid_same_h1_and_…
Browse files Browse the repository at this point in the history
…h2_headings

Complete `<h1>` settings heading
  • Loading branch information
blizzz authored Jan 16, 2024
2 parents 58b31a9 + f6ec297 commit 5f0c406
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 5 deletions.
2 changes: 1 addition & 1 deletion apps/theming/src/UserThemes.vue
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@

<template>
<section>
<NcSettingsSection :name="t('theming', 'Appearance and accessibility')"
<NcSettingsSection :name="t('theming', 'Appearance and accessibility settings')"
:limit-width="false"
class="theming">
<!-- eslint-disable-next-line vue/no-v-html -->
Expand Down
5 changes: 4 additions & 1 deletion core/templates/layout.user.php
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,10 @@

<main id="content" class="app-<?php p($_['appid']) ?>">
<h1 class="hidden-visually" id="page-heading-level-1">
<?php p(!empty($_['pageTitle'])?$_['pageTitle']:$theme->getName()); ?>
<?php p((!empty($_['application']) && !empty($_['pageTitle']) && $_['application'] != $_['pageTitle'])
? $_['application'].': '.$_['pageTitle']
: (!empty($_['pageTitle']) ? $_['pageTitle'] : $theme->getName())
); ?>
</h1>
<?php print_unescaped($_['content']); ?>
</main>
Expand Down
4 changes: 2 additions & 2 deletions dist/theming-personal-theming.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/theming-personal-theming.js.map

Large diffs are not rendered by default.

0 comments on commit 5f0c406

Please sign in to comment.