Skip to content

Danfe: fixing fields exceeding the limit in a large font#138

Merged
antoniospneto merged 1 commit intoEngenere:mainfrom
PedroJoMa:fix-danfe-fields-exceeding-the-limit-in-a-large-font
Mar 17, 2026
Merged

Danfe: fixing fields exceeding the limit in a large font#138
antoniospneto merged 1 commit intoEngenere:mainfrom
PedroJoMa:fix-danfe-fields-exceeding-the-limit-in-a-large-font

Conversation

@PedroJoMa
Copy link
Copy Markdown
Contributor

O que foi feito

  • Implementada a delimitação dos campos de endereço na DANFE para evitar que o texto extrapole o espaço reservado;
  • Ajuste na fonte 'Courier':
    • Devido à diferença de proporção em relação à fonte 'Times', o uso do tamanho BIG causava problemas de layout. A fonte 'Courier' foi fixada como SMALL, independentemente da configuração de tamanho definida pelo usuário;
  • Ajustada a largura dos campos Valor Total e %IPI para fontes grandes, pois os campos apresentavam quebra de linha indesejada mesmo com poucos dígitos;
  • Realizada a regeração de todos os DANFEs de teste/exemplo, refletindo as alterações de delimitação nos campos de:
    • Emitente
    • Destinatário
    • Transportadora

Imagens

DANFE-fixed

@PedroJoMa
Copy link
Copy Markdown
Contributor Author

PedroJoMa commented Mar 11, 2026

@antoniospneto Me sinaliza o que pode ser corrigido para que os testes possam aprovar, por gentileza

@antoniospneto
Copy link
Copy Markdown
Contributor

@PedroJoMa

Qual a versão do fpdf que está instalada no seu ambiente onde foi gerados os pdfs?
se tiver diferente do CI pode acabar gerando com pequenas diferenças no diff

@PedroJoMa PedroJoMa force-pushed the fix-danfe-fields-exceeding-the-limit-in-a-large-font branch from 6528b42 to 9070354 Compare March 16, 2026 12:30
@PedroJoMa
Copy link
Copy Markdown
Contributor Author

@antoniospneto atualizei as versoes das bibliotecas para validar os testes corretamente, obrigado pela dica!

@antoniospneto antoniospneto merged commit cae040f into Engenere:main Mar 17, 2026
7 checks passed
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.

3 participants