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

docs(punto-inicio): maquetado para puntos de inicio #297

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

JSantarelli
Copy link
Contributor

@JSantarelli JSantarelli commented Aug 25, 2021

Tasks (roadmap del componente):

Concepción: https://proyectos.andes.gob.ar/browse/PLEX-7
Maquetado: https://proyectos.andes.gob.ar/browse/PLEX-276
Iteraciones: https://proyectos.andes.gob.ar/browse/PLEX-278 | https://proyectos.andes.gob.ar/browse/PLEX-296

Recurso de navegación con doble función:

  • Agrupar/secuenciar información extensa en espacios reducidos a partir de la navegación entre anclas (anchors).
  • Navegación de las diferentes secciones de un módulo (submódulos) a partir de ruteos.

Plus
Se agrega demo en templates/punto-inicio

Para testear
Navegar a http://localhost:4200/templates/punto-inicio

@JSantarelli JSantarelli requested a review from a team as a code owner August 25, 2021 23:15
@JSantarelli JSantarelli requested review from MCele and palita1991 and removed request for a team August 25, 2021 23:15
height: calc(100vh - var(--vHeight));
position: fixed;
overflow-x: hidden;

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
margin-left: -10px;

Va como comment, porque lo hablamos en una videollamada: Con eso quedaría centrado horizontalmente respecto al contenedor ppal. Además despega el scroll del contenido en el sidebar.

Copy link
Contributor

@andrrrl andrrrl left a comment

Choose a reason for hiding this comment

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

En general está súper @JSantarelli!
Dejo como comment esto sobre lo que charlamos: que podría ser que cuando el mini-nav está cerrado y no se ven los labels, tengan tooltip por defecto.
De nuevo, va como comentario pensando en terminar de definir si hace falta el cambio.

@JSantarelli JSantarelli marked this pull request as draft October 14, 2021 12:55
@JSantarelli JSantarelli changed the title feat(plex-mininav): componente de navegación demo(punto-inicio): maquetado para puntos de inicio Oct 14, 2021
@JSantarelli JSantarelli changed the title demo(punto-inicio): maquetado para puntos de inicio docs(punto-inicio): maquetado para puntos de inicio Oct 14, 2021
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.

2 participants