-
Notifications
You must be signed in to change notification settings - Fork 2
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
fix: se implementa nuevamente reproductor de video con contenedor #133
base: master
Are you sure you want to change the base?
Conversation
…us modificaciones
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agrega el enlace a figma
@@ -4,13 +4,37 @@ import './Audio.stories.scss'; | |||
|
|||
// Addons | |||
import { withA11y } from '@storybook/addon-a11y'; | |||
import { Video, VideoYt } from '.'; | |||
|
|||
// Configuración general del componente | |||
export default { | |||
title: 'Componentes|Audio', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
La idea era que la carpeta principal sea Reproductor
y despues subcarpetas con audio y video
/> | ||
</div> | ||
<div className="card-body"> | ||
<div className="mb-2"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
El margen debería ser de margin-bottom: 0.75rem; en desktop y 0.5rem en mobile.
En mi opinión por la diferencia vale la pena poner el margen correcto.
<span className="badge badge-danger">Etiqueta</span> | ||
</div> | ||
<h4 className="card-title">Titular de la tarjeta con dos líneas de ejemplo</h4> | ||
<p className="card-text"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
El margen debería ser de margin-bottom: 0.75rem; en desktop y 0.5rem en mobile.
z-index: 2; | ||
} | ||
|
||
.main-iframe{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mandale un format document con prettier.
|
||
// Configuración general del componente | ||
export default { | ||
title: 'Componentes|Audio', | ||
decorators: [withA11y] | ||
}; | ||
|
||
export const VideoLocal = (): JSX.Element => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Si se quiere que se ordene alfabéticamente las historias, habría que poner las constantes en orden alfabético, no se ordenan con la implementación del Storybook 1.46.0
esta es una propuesta para el componente, esta abierto a mejoras de rendimiento, estilos, codigo y accesibilidad.
Enlace a figma:
https://www.figma.com/file/ToGlPBzfp1gKDmaCwKWtXS/UI-KIT---Obelisco?node-id=1141%3A14083&t=M0RtjuyCsM6RxM43-0