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

add icons and refactor navbar to support mobile view #400

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

beetlejuice96
Copy link
Collaborator

Issues relacionados.

  • Resuelve issue #

Que implementa o corrige este Pull Request.

Lista los links a documentación, recursos, etc que estén relacionados:

Tipo de Pull Request:

  • Refactor (Modificación de código existente).
  • New feature (Introducción de nueva funcionalidad).
  • Bug fix (Corrección de error/es)
  • Performance (Mejoras de optimización).
  • Docs (Actualización/extensión de documentación).

Pasos necesarios para probar/visualizar los cambios implementados:

  1. Ejecutar yarn dev

Capturas de pantalla/videos:

  • Nuevo componente.
  • Modificación a elemento en la interfaz de usuario.
  • No aplica.

Código de conducta y contribuir.

⚠️ Antes de realizar el Pull Request, por favor asegúrate de haber leído el código de conducta y cómo contribuir.

Note

⛑️ En caso de que necesites ayuda o tengas alguna duda recordá que también podés preguntar en el canal #juguetear dentro de la comunidad FrontendCafé en Discord. El código de conducta de este proyecto es extensible también a tu participación en el server de FrontendCafé.

@beetlejuice96 beetlejuice96 requested a review from TonyMckes April 1, 2024 23:24
@beetlejuice96 beetlejuice96 self-assigned this Apr 1, 2024
@beetlejuice96 beetlejuice96 linked an issue Apr 1, 2024 that may be closed by this pull request
3 tasks
Copy link

vercel bot commented Apr 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
juguetear-web ✅ Ready (Inspect) Visit Preview Apr 1, 2024 11:25pm
storybook-juguetear-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 1, 2024 11:25pm

@beetlejuice96
Copy link
Collaborator Author

  • Estado previo a la modificacion:
    image

  • Estado actual:
    image

Copy link
Member

@TonyMckes TonyMckes left a comment

Choose a reason for hiding this comment

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

Hey there @beetlejuice96! Perdona la tardanza!
Dejo un poco de feedback... Se esta viendo muy bien, solo note un par de cosas:

  1. El padding en el boton de cerrar se le podria reducir un poco

    image image
  2. Los links deberian indicar la pagina actual, asi como se ve en Figma

    image

De resto todo se ve perfecto! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Maquetar barra de navegación (mobile)
2 participants