kernoeb / PlanningSup

Planning universitaire réalisé en Nuxt.js
https://planningsup.app
Apache License 2.0
30 stars 29 forks source link

add word boundary check to highlighting regex #143

Closed leana8959 closed 4 days ago

leana8959 commented 4 days ago

Cela corrige l'erreur ou "CMPL-TD" peut se trouver en couleur de CM.

N'hésite pas à me dire s'il y a des choses à améliorer, et merci pour le projet ^^

Avant: image

Après: image

kernoeb commented 4 days ago

Eh mais c'est vraiment pas bête ça, je ne savais même pas qu'on pouvait faire ça via une expression régulière, sympa

Je vois juste un cas qui peut être problématique, il peut y avoir TPi par exemple

image
leana8959 commented 4 days ago

J'ai vu qu'on teste déjà "TDi" dans le prédicat des TP, est-ce que c'est normal ?

Et sinon je propose de faire une condition supplémentaire "TPi" (comme pour "TDi" déjà présent), sinon le regex risque d'être trop complexe et ça nuit à la lisibilité.

kernoeb commented 4 days ago

On m'avait indiqué à l'époque que TDi devrait être classifié comme un TP (aucune idée de pourquoi, ça date) Mais sinon, oui, pourquoi pas

Edit : Quelqu'un vient de m'indiquer que c'est le cas à l'IUT

kernoeb commented 4 days ago

Nickel, merci Léana !