Skip to content

Conversation

@Stef-Rousset
Copy link

@Stef-Rousset Stef-Rousset commented Dec 9, 2025

🎩 What? Why?

This PR adds semantic p tag in flash messages.

  • it updates the announcement_cell.rb file methods to add p tags when it is relevant.
    The issue has been spotted in the audit of Angers city (page 82)
  • it also adds p tag in the flash_helper_extension

📌 Related Issues

Link your PR to an issue

Testing

  • log in to the application and see that the text of the flash message is displayed in a p tag (screenshot one)
  • go to the show page of an evaluating proposal and see that the flash messages are displayed in a p tag, and that there is not empty p tags before and after the div class="rich-text-display" (screenshot two)

📷 Screenshots

ONE
Capture d’écran 2025-12-09 à 10 39 58

TWO
Capture d’écran 2025-12-09 à 16 25 52

♥️ Thank you!

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