remoção do link no header ao entrar nos detalhes/edição do abrigo#282
remoção do link no header ao entrar nos detalhes/edição do abrigo#282MayconPadilha wants to merge 2 commits intoSOS-RS:developfrom
Conversation
src/components/Header/Header.tsx
Outdated
| <Link className="font-medium text-white" to="/"> | ||
| {title} | ||
| </Link> | ||
| ) : ( | ||
| <span className="font-medium text-white"> | ||
| {title} | ||
| </span> |
There was a problem hiding this comment.
Será que conseguimos manter o comportamento de voltar, já presente na setinha/chevron, também no título?
There was a problem hiding this comment.
eu tentei essa possibilidade, mas quando o EditShelterSupply enviar os parâmetros para o header
<Header title="Editar Itens" className="bg-white [&_*]:text-zinc-800 border-b-[1px] border-b-border" startAdornment={ <Button variant="ghost" className="[&_svg]:stroke-blue-500" onClick={() => navigate(/abrigo/${shelterId})} > <ChevronLeft size={20} /> </Button> } />
neste momento, eu não achei como poderia enviar/receber o navigate como é feito com o button para o header e mudar a rota no link
então para não arrumar uma coisa e estragar a outra achei melhor desta forma, ate porque tenho pouca experiência em projeto tão extenso
There was a problem hiding this comment.
Será que conseguimos manter o comportamento de voltar, já presente na setinha/chevron, também no título?
Atualize a interface IHeader para incluir a propriedade to:
export interface IHeader extends React.ComponentPropsWithoutRef<'div'> {
title: string;
startAdornment?: React.ReactNode;
endAdornment?: React.ReactNode;
to?: string;
}
Modifique o componente Header para renderizar condicionalmente o título como um link se a propriedade to for fornecida:
const Header = React.forwardRef<HTMLDivElement, IHeader>((props, ref) => {
const {
endAdornment,
startAdornment,
title,
className = '',
to,
...rest
} = props;
{to ? (
<Link className="font-medium text-white" to={to}>
{title}
</Link>
) : (
<span className="font-medium text-white">{title}</span>
)}
Certifique-se de refatorar outros arquivos que usam o componente Header para passar a propriedade to conforme necessário.
There was a problem hiding this comment.
sua explicação me ajudou muito a melhorar o código, obrigado!
|
A issue foi fechada por não se tratar de um problema de fato. Foi adicionado comentários com vídeos lá explicando. |
|
Reaberta, motivo comentado na issue. |
|
Atualizei a navegação de retornor no header, agora é possível clicar para voltar diretamente no nome do abrigo para o home, e na edição e atualização do abrigo voltando para o abrigo referente. vídeo do teste: https://www.loom.com/share/cd773acc212e422c868e1a11ba213da9?sid=13b204dc-3bed-485d-a75e-23bc2ed6691d vou aguardar o figma com o fluxo @HbLuca |
| {to ? ( | ||
| <Link className="font-medium text-white" to={to}> | ||
| {title} | ||
| </Link> | ||
| ) : ( | ||
| <Link className="font-medium text-white" to="/">{title}</Link> | ||
| )} |
There was a problem hiding this comment.
Aqui poderia ser uma condicional mais simples para melhorar a legibilidade:
| {to ? ( | |
| <Link className="font-medium text-white" to={to}> | |
| {title} | |
| </Link> | |
| ) : ( | |
| <Link className="font-medium text-white" to="/">{title}</Link> | |
| )} | |
| <Link className="font-medium text-white" to={to || '/'}> | |
| {title} | |
| </Link> |
Adicionei um IF para que quando ele entrar nas informações/edições do abrigo ele remova o link que leva ate a tela inicial, porem o link da tela inicial volta ao home normalmente.
Link do teste: https://www.loom.com/share/d4d2022cfb1e4f6aa5631bc44b4e180d?sid=c0547d55-e3ec-4c7e-824c-9d5314d40709
Link da Issues: #245