Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remoção do link no header ao entrar nos detalhes/edição do abrigo #282

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

MayconPadilha
Copy 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

Comment on lines 28 to 34
<Link className="font-medium text-white" to="/">
{title}
</Link>
) : (
<span className="font-medium text-white">
{title}
</span>
Copy link

Choose a reason for hiding this comment

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

Será que conseguimos manter o comportamento de voltar, já presente na setinha/chevron, também no título?

Copy link
Author

Choose a reason for hiding this comment

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

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

Choose a reason for hiding this comment

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

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.

Copy link
Author

Choose a reason for hiding this comment

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

sua explicação me ajudou muito a melhorar o código, obrigado!

@fagundesjg fagundesjg added the invalid This doesn't seem right label May 22, 2024
@fagundesjg
Copy link
Collaborator

A issue foi fechada por não se tratar de um problema de fato. Foi adicionado comentários com vídeos lá explicando.

@fagundesjg fagundesjg closed this May 22, 2024
@HbLuca
Copy link

HbLuca commented May 22, 2024

Reaberta, motivo comentado na issue.
@MayconPadilha favor aguardar... foi realizado no figma um fluxo além de sugestão para adição de breadcrumbs (sugestão da @barbiebrega).

@HbLuca HbLuca reopened this May 22, 2024
@HbLuca HbLuca added bug Something isn't working and removed invalid This doesn't seem right bug Something isn't working labels May 23, 2024
@MayconPadilha
Copy link
Author

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

Comment on lines +28 to +34
{to ? (
<Link className="font-medium text-white" to={to}>
{title}
</Link>
) : (
<Link className="font-medium text-white" to="/">{title}</Link>
)}
Copy link
Member

Choose a reason for hiding this comment

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

Aqui poderia ser uma condicional mais simples para melhorar a legibilidade:

Suggested change
{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>

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.

[BUG] Editar Itens do Abrigo não retorna ao abrigo
6 participants