Skip to content

Conversation

@jbrieuclp
Copy link
Contributor

Permet d'ouvrir la fiche espèce et la fiche JDD dans un nouvel onglet et non à la place de la synthèse pour éviter de perdre la recherche réalisée.

@codecov
Copy link

codecov bot commented Aug 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.54%. Comparing base (bdec0c9) to head (2bfceaf).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #3656   +/-   ##
========================================
  Coverage    84.54%   84.54%           
========================================
  Files          133      133           
  Lines        10320    10320           
========================================
  Hits          8725     8725           
  Misses        1595     1595           
Flag Coverage Δ
pytest 84.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@camillemonchicourt
Copy link
Member

J'avais justement demandé qu l'on retire les "target _blank" sur les liens vers les fiches taxon, car il est conseillé de les utiliser le moins possible, et encore moins pour des liens internes à un site, éventuellement pour des liens externes.

Après c'est vrai que si on a fait une recherche dans la Synthèse, alors on perd celle-ci.
Le problème principal est peut-être plutôt que si on revient en arrière on ne retrouve pas sa recherche ?

@jbrieuclp
Copy link
Contributor Author

Oui, le retour précédent fait recharger la synthèse avec les N premières obs.
Perso quand je veux cliquer sur une ligne pour zoomer sur la loc, mon clic souris tombe facilement sur le nom du taxon. Et c'est normal, cette colonne représente bien un petit tiers de la largeur du tableau, donc en cliquant pour zoomer on se retrouve avec une nouvelle page et une recherche perdue. Relou.

Sinon je me posais la question de faire un bouton de menu contextuel par ligne (comme le menu icon de material https://material.angular.dev/components/menu/examples#menu-icons), après le i d'info. On pourrait avoir dans le menu le lien pour la modif occtax, le lien vers la fiche taxon, vers le JDD, vers la validation si activée et peut-être d'autres accès..

@camillemonchicourt
Copy link
Member

Oui ça c'est un sujet plus large.
Je n'étais pas favorable à ajouter un lien de cette manière vers les fiches taxon, car il est plus visible que le lien principal vers la fiche de l'observation et je craignais les confusions ou clics malheureux, comme discuté ici : #2718 (comment)

Long débat et je me suis rangé à la majorité. :-)

De là à passer sur un "MENU ICON" c'est une piste intéressante, ça se discute et se réfléchit car c'est pas hyper simple non plus.
Mais ce que je trouve intéressant, c'est qu'on se propose de base qu'une seule action qui est la principale = ouvrir la fiche observation. Sans proposer au même niveau 10 trucs différents où on se perd rapidement.
Et on regroupe toutes les actions de second niveau dans un MENU ICON. A creuser, discuter.

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.

2 participants