Update - Melhoria na listagem de suplementos #249
Conversation
|
@lucianomlima Fiz um PR pro back também removendo essa limitação de 10 items, visto que o objetivo é quando exceder mais de 10 items, deverá existir um décimo primeiro 'Chip' com a quantidade restante que não foi exibida. Link do PR: |
Deixa isso explícito na descrição do PR, assim quem está revisando sabe que houve alteração lá e quem vai fazer merge sabe que precisa antes que o PR do backend seja aprovado. |
Boa! brigadão mano, vou add lá na descrição do PR pra ficar mais fácil |
src/pages/Home/components/ShelterSupplyCategoryRow/ShelterSupplyCategoryRow.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
O nome correto da class é text-black-600, conforme a documentação oficial disponível aqui com os tokens de cores. Outro ponto, o bg não está sendo aplicado corretamente:
Sugiro a criação de uma "variante" do chip, com o nome de "moreInfo", adicionando as classes que você precisa, algo assim:
moreInfo: 'bg-gray-200 text-black-600'E o chip seria chamado dessa forma:
{tags.length > 10 && (
<Chip
label={`+${tags.length - 10} items`}
variant='moreInfo'
/>
)}Aí então teríamos o resultado esperado:
|
Subi um commit com um update com a correção do texto "itens" e a sugestão do @vinny-silveira |
There was a problem hiding this comment.
Avaliando a codificação e o resultado está de acordo, mas antes, é necessário finalizar o PR 136 aberto no repositório de backend para que entregue o valor e a funcionalidade completa solicitada na issue, até que o outro PR esteja apto ao merge, este ainda não deve ser mergeado.
src/pages/Home/components/ShelterSupplyCategoryRow/ShelterSupplyCategoryRow.tsx
Outdated
Show resolved
Hide resolved
|
@SOS-RS/frontenders , preciso de mais um review aqui. |
* fix: 🐛 Ajuste do tamanho do botao reload Fixado tamanho do botão para corrigir erro no Header * Criado botao * Adicionado funcoes para o botao. COMENTADO OVERFLOW-X:HIDDEN no global.css * removendo necessidade de remover overflow-x:hidden no global.css * resolvendo funcoes * Finalizando. durante meus testes esqueci de inverter o estado inicial do botão. * aplicado o design recomendado pela barbiebrega * Removido erro de tipagem. * #287 - [FIX] Itens Cadastrados sem Categoria estão indo para Medicamentos (#296) * #287 * Delete src/components/Icon directory * Update SupplyRowInfo.tsx * RollBack SupplyRowInfo.tsx * Update SupplyRow.tsx * Update EditShelterSupply.tsx * Update CreateSupply.tsx - De forma a evitar termos genéricos demais, é solicitado ao usuário que registre um recurso com no mínimo 3 caracteres. Validação via Yup. * Update CreateSupply.tsx - Bloqueia cadastro de items com números e caracteres especiais. Validação via Yup. * Update CreateSupply.tsx * Update CreateSupply.tsx - Limite de 30 itens retornados enquanto o usuário está digitando o termo desejado. * Update CreateSupply.tsx - Bloqueia caracteres especiais; - Requer no mínimo 3 letras (bloqueia apenas números). * Update - Melhoria na listagem de suplementos (#249) * feat: add multi option for priority queryParam * fix(filtro-shelters): add missing typing definition * fix: name of array of priority to priorities, priority field changed from string to array and removed unused import --------- Co-authored-by: kevindsousa <kevindiegodasilvasousa@gmail.com> Co-authored-by: leonardo <lmedeiros045@gmail.com> Co-authored-by: Leonardo <57924586+Lmedeiros-leiman@users.noreply.github.com> Co-authored-by: Eric Ricielle <suporte@tucanoweb.com.br> Co-authored-by: Lucas <ls4388387@gmail.com> Co-authored-by: Rhuam Sena <rhuam.estevam@doc9.com.br> Co-authored-by: Diego Dario <diego.dario@gazin.com.br> Co-authored-by: Rodrigo Oler <roodrigoprogrammer@gmail.com> Co-authored-by: Diego Dario <25825145+diegodario88@users.noreply.github.com> Co-authored-by: José Fagundes <fagundesjg@outlook.com>
* fix: 🐛 Ajuste do tamanho do botao reload Fixado tamanho do botão para corrigir erro no Header * Criado botao * Adicionado funcoes para o botao. COMENTADO OVERFLOW-X:HIDDEN no global.css * removendo necessidade de remover overflow-x:hidden no global.css * resolvendo funcoes * Finalizando. durante meus testes esqueci de inverter o estado inicial do botão. * aplicado o design recomendado pela barbiebrega * Removido erro de tipagem. * #287 - [FIX] Itens Cadastrados sem Categoria estão indo para Medicamentos (#296) * #287 * Delete src/components/Icon directory * Update SupplyRowInfo.tsx * RollBack SupplyRowInfo.tsx * Update SupplyRow.tsx * Update EditShelterSupply.tsx * Update CreateSupply.tsx - De forma a evitar termos genéricos demais, é solicitado ao usuário que registre um recurso com no mínimo 3 caracteres. Validação via Yup. * Update CreateSupply.tsx - Bloqueia cadastro de items com números e caracteres especiais. Validação via Yup. * Update CreateSupply.tsx * Update CreateSupply.tsx - Limite de 30 itens retornados enquanto o usuário está digitando o termo desejado. * Update CreateSupply.tsx - Bloqueia caracteres especiais; - Requer no mínimo 3 letras (bloqueia apenas números). * Update - Melhoria na listagem de suplementos (#249) * feat: add multi option for priority queryParam * fix(filtro-shelters): add missing typing definition * fix: name of array of priority to priorities, priority field changed from string to array and removed unused import ---------
### Melhorar a descoberta de que existem mais itens depois que o limite de 10 é excedido no card Conforme solicitado na issue [#257](SOS-RS/frontend#257) foi removido o limite no retorno dos supplies de abrigo. PR do Frontend: SOS-RS/frontend#249
### Melhorar a descoberta de que existem mais itens depois que o limite de 10 é excedido no card Conforme solicitado na issue [#257](SOS-RS/frontend#257) foi removido o limite no retorno dos supplies de abrigo. PR do Frontend: SOS-RS/frontend#249
Melhorar a descoberta de que existem mais itens depois que o limite de 10 é excedido no card
Issue #257
Exibindo 10 primeiros items do abrigo e décimo primeiro 'Chip' com a quantidade de items não exibidos.
PR para remoção de limite no backend: SOS-RS/backend#136
Screenshot da melhoria:
