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

fix: proposed input file #151

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

fix: proposed input file #151

wants to merge 26 commits into from

Conversation

davisLjr
Copy link
Contributor

@davisLjr davisLjr commented Apr 24, 2023

Se genera una propuesta base para componente de adjuntar archivo, estilos exactos se definiran luego con equipo de diseño.

ejemplo tomado de :

https://xd.adobe.com/view/5d234c49-eddd-4a68-b37c-8a01f5341675-0870/screen/3cc41c85-8afc-4c68-ab52-647de763d36e/specs/

Quedan pendientes:

  1. iconos distintos.
  2. espaciados.
  3. boton de eliminar
  4. font-size

image


image


image

alan199912 and others added 16 commits March 6, 2023 15:27
* fix: se corrige espaciado de icono en boton

* fix: se agrega el icono de material icons al boton

---------

Co-authored-by: Victoria Raquel Nastasi <vnastasi@buenosaires.gob.ar>
* feat: tooltip

* fix: ajustes al scss de tooltip

* fix: ajustes en los valores del scss de tooltip

* fix: mejoras en los estilos de tooltip

* fix: mejoras al scss de tooltip

* fix: correcciones al scss de tooltip

* fix: refactor del scss de tooltip

* fix: cambios en el scss de tooltip

* fix: se corrige el scss de tooltip en mobile

* fix: ajustes en los valores del scss de tooltip

* fix: ajustes en los valores de right en mobile

* fix: mejoras al scss de tooltip en mobile

* fix: correccion a tooltip

* fix: se corrige el espaciado de tooltip container

* fix: se cambian lo valores de left y right

* fix: correcciones de los comentarios del pr

* fix: ajustes y comentarios del pr

* fix: se ajuestes a las propiedades y atributos
#140)

* fix: se modifica visualizacion de breadcrumb en storybook, se reemplaza divider por icono de material-icons-round, se corrigen espaciados

* fix: correcion de nombre en stories
* feat: alert dismissible

* fix: ordenamiento y renombre de las stories

* fix: se removio subrayado y se cambio padding a margin en button close

* fix: textos de la ficha

* fix: error lint & creacion de componentes react

* fix: se removio efecto fade de alert link
* fix: pagination

* fix: correccion de estilos y nombres

* fix: modificacion de estilos en mobile

* fix: correccion en espaciados, nombre de variable en scss
* feat: navegacion por pestañas

* fix: correccion comentarios en PR y se agrega variante tamaños

* fix: se actualizan espaciados en navegacion vertical

* fix: se agrega opcion draggable al slider de tabs

* fix: correccion de draggable se agrega onMouseLeave

* fix: se agregan props y correcion segun PR

* fix: correccion de error en interface

* fix: correccion comentarios en PR

* fix: correccion interface de pestañas, y comentarios en PR

* fix: se ajusta el gap de icono segun diseño en figma

* fix: correccion font-weight en pestañas estado active

* fix: correccion nombre de pestañas y padding de nav-link

* fix: se agregan pestañas de distintos tamaños en tres filas

* fix: se eliminan clases de bootstrap y se agregan los estilos correspondientes

* fix: correccion nombre de pestañas - navegacion vertical y horizontal

* fix: se agrega 0.2s adicional de transition en font-weight cuando esta active y el ease-in-out

* fix: se actualiza el nombre de la variable nav-link-padding-x

---------

Co-authored-by: Victoria Nastasi <victorianastasi@gmail.com>
* fix: spinner y boton con spinner

* fix: modificaciones en estilos de spinner y boton con spinner

* fix: comentarios pr

* fix: ajuste en imports en componentes

* fix: flex-wrap de spinner-wrapper
* fix: se pasa el componente de html a react

* feat: Alerta - Tooltip - Paginador - Breadcrumb - Pestañas (#143)

* ci: se agrega la rama dev al workflow

* fix: se corrige espaciado de icono en boton (#136)

* fix: se corrige espaciado de icono en boton

* fix: se agrega el icono de material icons al boton

---------

Co-authored-by: Victoria Raquel Nastasi <vnastasi@buenosaires.gob.ar>

* feat: tooltip (#137)

* feat: tooltip

* fix: ajustes al scss de tooltip

* fix: ajustes en los valores del scss de tooltip

* fix: mejoras en los estilos de tooltip

* fix: mejoras al scss de tooltip

* fix: correcciones al scss de tooltip

* fix: refactor del scss de tooltip

* fix: cambios en el scss de tooltip

* fix: se corrige el scss de tooltip en mobile

* fix: ajustes en los valores del scss de tooltip

* fix: ajustes en los valores de right en mobile

* fix: mejoras al scss de tooltip en mobile

* fix: correccion a tooltip

* fix: se corrige el espaciado de tooltip container

* fix: se cambian lo valores de left y right

* fix: correcciones de los comentarios del pr

* fix: ajustes y comentarios del pr

* fix: se ajuestes a las propiedades y atributos

* fix: se modifica visualizacion de breadcrumb en storybook, se reempla… (#140)

* fix: se modifica visualizacion de breadcrumb en storybook, se reemplaza divider por icono de material-icons-round, se corrigen espaciados

* fix: correcion de nombre en stories

* feat: alert dismissible (#135)

* feat: alert dismissible

* fix: ordenamiento y renombre de las stories

* fix: se removio subrayado y se cambio padding a margin en button close

* fix: textos de la ficha

* fix: error lint & creacion de componentes react

* fix: se removio efecto fade de alert link

* fix: pagination (#141)

* fix: pagination

* fix: correccion de estilos y nombres

* fix: modificacion de estilos en mobile

* fix: correccion en espaciados, nombre de variable en scss

* feat: Alerta - Tooltip - Paginador - Breadcrumb

* fix: version actual de manera dinamica

* feat: navegacion por pestañas (#138)

* feat: navegacion por pestañas

* fix: correccion comentarios en PR y se agrega variante tamaños

* fix: se actualizan espaciados en navegacion vertical

* fix: se agrega opcion draggable al slider de tabs

* fix: correccion de draggable se agrega onMouseLeave

* fix: se agregan props y correcion segun PR

* fix: correccion de error en interface

* fix: correccion comentarios en PR

* fix: correccion interface de pestañas, y comentarios en PR

* fix: se ajusta el gap de icono segun diseño en figma

* fix: correccion font-weight en pestañas estado active

* fix: correccion nombre de pestañas y padding de nav-link

* fix: se agregan pestañas de distintos tamaños en tres filas

* fix: se eliminan clases de bootstrap y se agregan los estilos correspondientes

* fix: correccion nombre de pestañas - navegacion vertical y horizontal

* fix: se agrega 0.2s adicional de transition en font-weight cuando esta active y el ease-in-out

* fix: se actualiza el nombre de la variable nav-link-padding-x

---------

Co-authored-by: Victoria Nastasi <victorianastasi@gmail.com>

* fix: se modifican nombres, se reemplaza textos secundarios por epigrafe y se ajusta su font-size (#144)

---------

Co-authored-by: Alan199912 <alan199912@gmail.com>
Co-authored-by: Victoria Nastasi <victorianastasi@gmail.com>
Co-authored-by: Victoria Raquel Nastasi <vnastasi@buenosaires.gob.ar>
Co-authored-by: Jesus Avendaño <53105473+Jesus1397@users.noreply.github.com>
Co-authored-by: Alan Marcelo Di Masi <40763553+alan199912@users.noreply.github.com>
Co-authored-by: Davis Jr Lapenta <98839538+davisLjr@users.noreply.github.com>

* fix: ajustes al componente

* fix: se corrige el nombre del componente

* fix: se elimina la clase progression-container del componente

* fix: se agrega borderradius y se cambia la forma de visualizarse

* fix: se corrige espaciado en el scss

---------

Co-authored-by: lautarorodriguez96 <70086268+lautarorodriguez96@users.noreply.github.com>
Co-authored-by: Alan199912 <alan199912@gmail.com>
Co-authored-by: Victoria Nastasi <victorianastasi@gmail.com>
Co-authored-by: Victoria Raquel Nastasi <vnastasi@buenosaires.gob.ar>
Co-authored-by: Alan Marcelo Di Masi <40763553+alan199912@users.noreply.github.com>
Co-authored-by: Davis Jr Lapenta <98839538+davisLjr@users.noreply.github.com>
* fix: se agrego el estado activo, cambios paddings y margenes, lista de acceso, completo

* fix: cambios comentarios pr, mas arreglos extras

* fix: se cambio en el scss span por .material-icons-round

* fix: en organismos va un solo icono junto al titulo y parrafo

* fix: cambios con vicky, paddings, reverse

* fix: padding, format document

* fix: se agregaron ejemplos de los componentes anteriores, paddings, margins

* fix: arreglos

* fix: se pasaron a componentes

* fix: se cambiaron los nombres de la interfaz y el type a ingles

* fix: se sacaron los ejemplos de versiones anteriores en accesos

* fix: comentarios pr, estados en los accesos de organismo

* fix: alan

* fix: acceso con lista y con categoria sin shadow

* fix: tamaño texto descriptive, alineacion icon cuando se usa etiqueta i

* fix: comentarios pr, margenes

* fix: se me colo un archivo

* fix: modificaciones en estilos, paddings, se eliminan importants, se elimina clase mt-0 en hr

* refactor: se hizo un solo componente para reutilizarlo en organismo haciendo uso de props

* fix: pase el h6 a h5 y extendi la clase, se arreglaron los accesos de anteriores versiones

* fix: se saco una variable, estilos de la clase p-list, variacion de icono en simple

* fix: margin en iconos

* fix: ajustes en css

* fix: ajuste de padding de hr en mobile

* fix: ajuste de css de hr en mobile en accesos con categorias, y de lineas en accesos luego de hover

* fix: ajustes en css, modificacion de color de fondo de accesos

---------

Co-authored-by: Victoria Nastasi <victorianastasi@gmail.com>
@davisLjr davisLjr self-assigned this Apr 24, 2023
@davisLjr davisLjr changed the title Fix proposed input file fix: proposed input file Apr 24, 2023
@davisLjr davisLjr changed the base branch from master to dev April 24, 2023 20:58
Copy link
Contributor

Choose a reason for hiding this comment

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

Consultaria sobre el tipo de archivo permitido y el tamaño maximo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Falta detallar

src/form/File.stories.tsx Outdated Show resolved Hide resolved
src/form/File.stories.tsx Outdated Show resolved Hide resolved
src/form/File.stories.tsx Outdated Show resolved Hide resolved
src/form/File.stories.tsx Outdated Show resolved Hide resolved
src/form/File.stories.tsx Outdated Show resolved Hide resolved
src/form/File.stories.tsx Outdated Show resolved Hide resolved
src/form/File.stories.tsx Outdated Show resolved Hide resolved
src/form/File.stories.tsx Outdated Show resolved Hide resolved
@davisLjr
Copy link
Contributor Author

En el Proximo commits, trataré de evitar el position absolute, y tener los espaciados correctos de un diseño de figma, se agregaran los estados del componente y se consultaran los iconos exactos o si pueden variar.

Base automatically changed from dev to master May 3, 2023 16:55
<div className="form-wrapper-lg">
<div className="file-group bg-light">
<i className="bx bx-cloud-upload"></i>
<label htmlFor="file-input" className="input-file-title">
Copy link
Collaborator

Choose a reason for hiding this comment

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

Le agregaria el cursor: pointer

src/scss/components/_form.scss Outdated Show resolved Hide resolved
};

LoadFileMulti.story = {
name: 'Subir un archivo multiple'
Copy link
Contributor

Choose a reason for hiding this comment

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

tal vez lo modificaria por Subir múltiples archivos , para que sea más claro

};

Example.story = {
name: 'Archivo cargado'
Copy link
Contributor

Choose a reason for hiding this comment

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

tal vez se podria mencionar como Demo (como en modal)

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.

None yet

6 participants