Skip to content

Conversation

@stephanegigandet
Copy link
Contributor

This PR:

  • changes the icon of the "Smoking kills" panel to the "dangerous" material icon (instead of the "limit" downward icon we had before)
  • add a new panel specific for France with links to the national health site on stopping tobacco (this one is hardcoded, it's probably not a good idea to use the taxonomy properties for local panels in only one language)
  • move recommendation panel properties from cigarettes to tobacco products

https://fr.openproductsfacts.org/produit/87248364/cigarettes-marlboro

@stephanegigandet stephanegigandet requested a review from a team as a code owner October 20, 2025 14:11
@github-actions github-actions bot added 🧬 Taxonomies https://wiki.openfoodfacts.org/Global_taxonomies 📸 Open Products Facts Our project to increase the lifespan of objects. https://world.openproductsfacts.org 📖 Knowledge Panels https://wiki.openfoodfacts.org/Knowledge_panels Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. icons Display labels Oct 20, 2025
@github-project-automation github-project-automation bot moved this to To discuss and validate in 🍊 Open Food Facts Server issues Oct 20, 2025
@stephanegigandet
Copy link
Contributor Author

/update_tests_results

@stephanegigandet
Copy link
Contributor Author

/update_tests_results

@github-actions github-actions bot added the 💥 Merge Conflicts 💥 Merge Conflicts label Oct 28, 2025
}
},
],
} No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
}
}

@stephanegigandet
Copy link
Contributor Author

@Freso regarding the HTML, yes it's great to improve it, but we want it to remain very simple HTML (just basic tags like

etc.) as it's also displayed in the app and other things. So no CSS styles etc.

@sonarqubecloud
Copy link

Comment on lines +17 to +39
Au programme du site <a href="https://www.tabac-info-service.fr/">Tabac info service</a> édité par Santé publique France :
<br>
<br> <b><a href="https://www.tabac-info-service.fr/je-trouve-ma-motivation">Je trouve ma motivation</a></b>
<br>
<br> Faites le plein de motivation !! Rien de tel pour trouver le courage de vous lancer, et de tenir bon dans votre arrêt du tabac
<br>
<br> <b><a href="https://www.tabac-info-service.fr/j-arrete-de-fumer">J'arrête de fumer</a></b>
<br>
<br> C’est parti ! Découvrez tous nos conseils et stratégies pour vous débarrasser de la cigarette. Et composez votre propre méthode pour arrêter de fumer !
<br>
<br> <b><a href="https://www.tabac-info-service.fr/questions-reponses">Questions / Réponses</a></b>
<br>
<br> Vous vous posez des questions sur l'arrêt, sur les traitements existants, les méthodes pour y arriver, ... ? Dans cette rubrique, Tabac info service répond aux questions les plus fréquemment posées et vous donne les clés pour mieux appréhender votre démarche d'arrêt du tabac.
<br>
<br> <b>Je me fais accompagner</b>
<br>
<br> <a href="https://www.tabac-info-service.fr/je-me-fais-accompagner/le-39-89">Le 39 89</a>
<br> Contacter un professionnel de l'arrêt du tabac
<br>
<br> <a href="https://www.tabac-info-service.fr/je-trouve-un-tabacologue">Je trouve un tabacologue</a>
<br> Consulter un professionnel près de chez vous
<br>
<br> <a href="https://www.tabac-info-service.fr/je-me-fais-accompagner/je-telecharge-l-application-pour-arreter-de-fumer">Je télécharge l’application Tabac info service pour arrêter de fumer</a>
Copy link
Collaborator

@Freso Freso Oct 29, 2025

Choose a reason for hiding this comment

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

This makes paragraphs semantically separated by use of <p>…</p> blocks instead of solely visually (by use of forced line breaks) and also makes the headers actual headers.

Suggested change
Au programme du site <a href="https://www.tabac-info-service.fr/">Tabac info service</a> édité par Santé publique France :
<br>
<br> <b><a href="https://www.tabac-info-service.fr/je-trouve-ma-motivation">Je trouve ma motivation</a></b>
<br>
<br> Faites le plein de motivation !! Rien de tel pour trouver le courage de vous lancer, et de tenir bon dans votre arrêt du tabac
<br>
<br> <b><a href="https://www.tabac-info-service.fr/j-arrete-de-fumer">J'arrête de fumer</a></b>
<br>
<br> C’est parti ! Découvrez tous nos conseils et stratégies pour vous débarrasser de la cigarette. Et composez votre propre méthode pour arrêter de fumer !
<br>
<br> <b><a href="https://www.tabac-info-service.fr/questions-reponses">Questions / Réponses</a></b>
<br>
<br> Vous vous posez des questions sur l'arrêt, sur les traitements existants, les méthodes pour y arriver, ... ? Dans cette rubrique, Tabac info service répond aux questions les plus fréquemment posées et vous donne les clés pour mieux appréhender votre démarche d'arrêt du tabac.
<br>
<br> <b>Je me fais accompagner</b>
<br>
<br> <a href="https://www.tabac-info-service.fr/je-me-fais-accompagner/le-39-89">Le 39 89</a>
<br> Contacter un professionnel de l'arrêt du tabac
<br>
<br> <a href="https://www.tabac-info-service.fr/je-trouve-un-tabacologue">Je trouve un tabacologue</a>
<br> Consulter un professionnel près de chez vous
<br>
<br> <a href="https://www.tabac-info-service.fr/je-me-fais-accompagner/je-telecharge-l-application-pour-arreter-de-fumer">Je télécharge l’application Tabac info service pour arrêter de fumer</a>
<p>Au programme du site <a href="https://www.tabac-info-service.fr/">Tabac info service</a> édité par Santé publique France :</p>
<h6><b><a href="https://www.tabac-info-service.fr/je-trouve-ma-motivation">Je trouve ma motivation</a></b></h6>
<p>Faites le plein de motivation !! Rien de tel pour trouver le courage de vous lancer, et de tenir bon dans votre arrêt du tabac</p>
<h6><b><a href="https://www.tabac-info-service.fr/j-arrete-de-fumer">J'arrête de fumer</a></b></h6>
<p>C’est parti ! Découvrez tous nos conseils et stratégies pour vous débarrasser de la cigarette. Et composez votre propre méthode pour arrêter de fumer !</p>
<h6><b><a href="https://www.tabac-info-service.fr/questions-reponses">Questions / Réponses</a></b></h6>
<p>Vous vous posez des questions sur l'arrêt, sur les traitements existants, les méthodes pour y arriver, ... ? Dans cette rubrique, Tabac info service répond aux questions les plus fréquemment posées et vous donne les clés pour mieux appréhender votre démarche d'arrêt du tabac.</p>
<h6><b>Je me fais accompagner</b></h6>
<p><a href="https://www.tabac-info-service.fr/je-me-fais-accompagner/le-39-89">Le 39 89</a>
<br> Contacter un professionnel de l'arrêt du tabac</p>
<p><a href="https://www.tabac-info-service.fr/je-trouve-un-tabacologue">Je trouve un tabacologue</a>
<br> Consulter un professionnel près de chez vous</p>
<p><a href="https://www.tabac-info-service.fr/je-me-fais-accompagner/je-telecharge-l-application-pour-arreter-de-fumer">Je télécharge l’application Tabac info service pour arrêter de fumer</a></p>

Copy link
Member

@alexgarel alexgarel left a comment

Choose a reason for hiding this comment

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

LGTM, but I think it's also already in #12550

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Display icons 📖 Knowledge Panels https://wiki.openfoodfacts.org/Knowledge_panels 💥 Merge Conflicts 💥 Merge Conflicts 📸 Open Products Facts Our project to increase the lifespan of objects. https://world.openproductsfacts.org 🧬 Taxonomies https://wiki.openfoodfacts.org/Global_taxonomies Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. 🧪 tests

Projects

Status: To discuss and validate
Status: In progress
Status: In progress

Development

Successfully merging this pull request may close these issues.

Ensure we display Legal Warnings on cigarettes for Open Products Facts

4 participants