Skip to content

refactor(backend/newsletter/templates/footer.html): Inclusao da logo …#1172

Open
josiane-silwa wants to merge 1 commit into
mainfrom
1169-incluir-logo-inct-nos-emails-de-subscricao
Open

refactor(backend/newsletter/templates/footer.html): Inclusao da logo …#1172
josiane-silwa wants to merge 1 commit into
mainfrom
1169-incluir-logo-inct-nos-emails-de-subscricao

Conversation

@josiane-silwa

Copy link
Copy Markdown
Contributor

Inclusao da logo e-universo inct no template de emails da subscrição

…e-universo inct no template de emails da subscrição
@josiane-silwa josiane-silwa linked an issue Apr 15, 2025 that may be closed by this pull request
@rcboufleur
rcboufleur requested a review from Copilot June 3, 2025 21:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the newsletter email footer by including the e-universo INCT logo and reorganizes inline styles for better readability.

  • Adds the e-universo logo in the sponsorship section
  • Converts long inline style attributes into a multiline format
Comments suppressed due to low confidence (2)

backend/newsletter/templates/footer.html:18

  • [nitpick] The alt text "INCT" is ambiguous. Consider using a more descriptive alt attribute like "e-Universo INCT" to improve accessibility and clarity.
alt="INCT"

backend/newsletter/templates/footer.html:5

  • This block has repeated inline style definitions. Consider extracting these into a shared CSS class or partial to reduce duplication and improve maintainability.
<div style="...">

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.

Incluir logo INCT nos emails de subscricao

2 participants