Tentativa de endereçar issues 15 e 32#34
Conversation
There was a problem hiding this comment.
Pull request overview
Atualiza informações e UI da tabela de corretoras no exterior para endereçar os issues 15 e 32.
Changes:
- Adiciona um campo
altcom explicação detalhada de corretagem (BTG EUA). - Torna o cabeçalho da tabela “sticky” para melhorar a navegação em listas longas.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| pages/corretoras_exterior/btg.yml | Inclui texto alternativo/descritivo para esclarecer regras de corretagem. |
| pages/corretoras-exterior.html.erb | Ajusta classes Tailwind para fixar o <thead> no topo durante o scroll. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # source: https://www.btgpactual.us/pt/faq | ||
| corretagem_eua: | ||
| text: $1 a $7,50 | ||
| alt: "Conforme o escritório parceiro os custos de corretagem mudam. Exemplo com a AUVP: de US$20,01 até US$100 = US$1,00; até US$1.000 = US$1,25; até US$2.000 = US$2,50; acima de US$2.000 = US$3,75" |
There was a problem hiding this comment.
Os valores do alt (ex.: até US$1.000 = US$1,25; até US$2.000 = US$2,50; acima = US$3,75) conflitam com os valores do subtext (até $1.000 = $2,50; até $2.000 = $5; acima = $7,50). Isso pode exibir informações contraditórias ao usuário. Sugestão: alinhar subtext/text ao mesmo exemplo (AUVP) ou ajustar o alt para explicar explicitamente que há tabelas diferentes e qual delas está sendo apresentada no subtext.
| alt: "Conforme o escritório parceiro os custos de corretagem mudam. Exemplo com a AUVP: de US$20,01 até US$100 = US$1,00; até US$1.000 = US$1,25; até US$2.000 = US$2,50; acima de US$2.000 = US$3,75" | |
| alt: "Conforme o escritório parceiro os custos de corretagem mudam. Exemplo com a AUVP: de US$20,01 até US$100 = US$1,00; até US$1.000 = US$2,50; até US$2.000 = US$5,00; acima de US$2.000 = US$7,50" |
| <div class="w-full"> | ||
| <table class="min-w-full bg-white shadow-md rounded-lg overflow-hidden"> | ||
| <thead class="bg-gray-200 text-gray-600 uppercase text-sm leading-normal"> | ||
| <thead class="sticky top-0 z-10 bg-gray-200 text-gray-600 uppercase text-sm leading-normal"> |
There was a problem hiding this comment.
Aplicar position: sticky diretamente no <thead> não é consistentemente suportado em todos os navegadores (frequentemente funciona melhor ao aplicar sticky top-0 nos <th>). Para evitar regressões de UI, considere mover sticky top-0 para as células de header (th) e manter o z-index/bg-* nelas, garantindo que o cabeçalho permaneça visível e com fundo sólido ao sobrepor o conteúdo.
No description provided.