Skip to content

fix(tickets): apply review feedback — semantic HTML, grammar/typo fixes, use title partial#10

Merged
tuturd merged 2 commits into
modification-page-ticketfrom
copilot/sub-pr-8
Mar 27, 2026
Merged

fix(tickets): apply review feedback — semantic HTML, grammar/typo fixes, use title partial#10
tuturd merged 2 commits into
modification-page-ticketfrom
copilot/sub-pr-8

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 9, 2026

Addresses review comments on the tickets page update: grammar errors, typos, non-semantic markup, and inconsistent use of shared partials.

Changes

  • Grammar/typos: "des Association" → "des Associations", "Delais" → "Délais", "crèpieres" → "crêpières", "pour toutes demande" → "pour toute demande"
  • Partial consistency: Replaced raw <h2> headings with {% include "partials/title.html" %} for both "C'est quoi ?" and "Quels types de tickets ?", matching the rest of the site
  • Semantic HTML: Converted ticket type list from a flat <p> block with <br> separators to a <dl>/<dt>/<dd> structure for accessibility and readability
<!-- Before -->
<p class="fs-5 text-justify">
    Tickets Notification d'évènements : <b>Obligatoire</b> ...
    Delais : Déposer le ticket avant le mercredi ...
    <br>
    Tickets Prêt de matériel : pour emprunter ...
    ...
</p>

<!-- After -->
<dl>
    <dt>Tickets Notification d'évènements</dt>
    <dd>
        <b>Obligatoire</b> pour tous les évènements ...<br>
        Délais : Déposer le ticket avant le mercredi ...
    </dd>
    <dt>Tickets Prêt de matériel</dt>
    <dd>
        Pour emprunter du matériel ..., comme des crêpières ...<br>
        Délais : Déposer le ticket au moins 48h avant.
    </dd>
    ...
</dl>

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…to dl structure

Co-authored-by: TomLrs <200967687+TomLrs@users.noreply.github.com>
Copilot AI changed the title [WIP] Update informational content on tickets page fix(tickets): apply review feedback — semantic HTML, grammar/typo fixes, use title partial Mar 9, 2026
Copy link
Copy Markdown
Member

@tuturd tuturd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'aime pas trop relire le code d'un bot mais bon.

@tuturd tuturd marked this pull request as ready for review March 27, 2026 21:43
@tuturd tuturd merged commit c7b2858 into modification-page-ticket Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants