Skip to content

Commit 3a8e999

Browse files
authored
Déplace l'URL de la page sous son nom dans le rapport (#257)
* fix: move page url below its name in report errors * docs: update changelog
1 parent 2389c43 commit 3a8e999

2 files changed

Lines changed: 25 additions & 20 deletions

File tree

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,12 @@
22

33
Tous les changements notables de Ara sont documentés ici avec leur date, leur catégorie (nouvelle fonctionnalité, correction de bug ou autre changement) et leur pull request (PR) associée.
44

5+
## 9/12/2022
6+
7+
### Autres changements ⚙️
8+
9+
- Dans le rapport, déplace l'URL de la page sous son nom ([#257](https://github.com/DISIC/Ara/pull/257))
10+
511
## 15/12/2022
612

713
### Corrections 🐛

confiture-web-app/src/components/ReportErrors.vue

Lines changed: 19 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -351,18 +351,21 @@ function updateActiveAnchorLink(id: string, event: MouseEvent) {
351351
</button>
352352
</div>
353353
<section v-for="page in errors" :key="page.pageId" class="fr-mb-8w">
354-
<div class="fr-mb-4w page-title-container">
355-
<h2
356-
:id="`${getPageSlug(page.pageId)}`"
357-
class="fr-h3 fr-mb-0 page-title"
358-
>
359-
{{ page.pageName }}
360-
</h2>
361-
<a :href="page.pageUrl" class="" target="_blank" rel="noopener">
362-
{{ page.pageUrl }}
363-
<span class="sr-only"> (ouvre dans un nouvel onglet) </span>
364-
</a>
365-
</div>
354+
<h2
355+
:id="`${getPageSlug(page.pageId)}`"
356+
class="fr-h3 fr-mb-2w page-title"
357+
>
358+
{{ page.pageName }}
359+
</h2>
360+
<a
361+
:href="page.pageUrl"
362+
class="fr-mb-4w page-url"
363+
target="_blank"
364+
rel="noopener"
365+
>
366+
{{ page.pageUrl }}
367+
<span class="sr-only"> (ouvre dans un nouvel onglet) </span>
368+
</a>
366369

367370
<div
368371
v-for="(topic, i) in page.topics"
@@ -489,18 +492,14 @@ function updateActiveAnchorLink(id: string, event: MouseEvent) {
489492
box-shadow: inset -1px 0 0 0 var(--border-default-grey);
490493
}
491494
492-
.page-title-container {
493-
display: flex;
494-
justify-content: space-between;
495-
align-items: center;
496-
gap: 1rem;
497-
flex-wrap: wrap;
498-
}
499-
500495
.page-title {
501496
color: var(--text-active-blue-france);
502497
}
503498
499+
.page-url {
500+
display: inline-block;
501+
}
502+
504503
.criterium-title {
505504
color: var(--text-title-grey);
506505
}

0 commit comments

Comments
 (0)