Skip to content

Commit

Permalink
Merge pull request #40768 from nextcloud/fix/37098/dashboard-duplicat…
Browse files Browse the repository at this point in the history
…ed-ids

fix(dashboard): remove duplicated IDs on the dashboard page
  • Loading branch information
susnux authored Oct 4, 2023
2 parents 7641af5 + 9ffd5a3 commit a3657ed
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
8 changes: 4 additions & 4 deletions apps/dashboard/src/DashboardApp.vue
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,13 @@
class="panel">
<div class="panel--header">
<h2>
<div aria-labelledby="panel--header--icon--description"
<div :aria-labelledby="`panel-${panels[panelId].id}--header--icon--description`"
aria-hidden="true"
:class="apiWidgets[panels[panelId].id].icon_class"
role="img" />
{{ apiWidgets[panels[panelId].id].title }}
</h2>
<span id="panel--header--icon--description" class="hidden-visually">
<span :id="`panel-${panels[panelId].id}--header--icon--description`" class="hidden-visually">
{{ t('dashboard', '"{title} icon"', { title: apiWidgets[panels[panelId].id].title }) }}
</span>
</div>
Expand All @@ -39,13 +39,13 @@
<div v-else :key="panels[panelId].id" class="panel">
<div class="panel--header">
<h2>
<div aria-labelledby="panel--header--icon--description"
<div :aria-labelledby="`panel-${panels[panelId].id}--header--icon--description`"
aria-hidden="true"
:class="panels[panelId].iconClass"
role="img" />
{{ panels[panelId].title }}
</h2>
<span id="panel--header--icon--description" class="hidden-visually"> {{ t('dashboard', '"{title} icon"', { title: panels[panelId].title }) }} </span>
<span :id="`panel-${panels[panelId].id}--header--icon--description`" class="hidden-visually"> {{ t('dashboard', '"{title} icon"', { title: panels[panelId].title }) }} </span>
</div>
<div class="panel--content" :class="{ loading: !panels[panelId].mounted }">
<div :ref="panels[panelId].id" :data-id="panels[panelId].id" />
Expand Down
4 changes: 2 additions & 2 deletions dist/dashboard-main.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/dashboard-main.js.map

Large diffs are not rendered by default.

0 comments on commit a3657ed

Please sign in to comment.