Se modifican los componentest Modal, SimpleInput y SelectDropdown

[iTzGooDLife]

Componentes modificados

• SimpleInput
• SelectDropdown
• Modal

Cambios Realizados

• SimpleInput:
  • Se quita el "relative" para que no se sobreponga a otros elementos
  • Se agrega prop (opcional) disabled; con default en false.
• Modal:
  • Se agrega un nuevo prop para el hr superior e inferior a los children. Si se quiere habilitar, entonces se debe agregar el prop disablehr con valor true, ya que por defecto está habilitado.
  • Se agrega un icono en la parte superior derecha el cual es otra manera de cerrar el modal.

• Dropdown:
  • Cambio de prop onChange
  • Arreglo sobreposición
  • Nuevo prop opcional para ListItem
  • Se arregla el tamaño si no hay un texto default
  • Se agrega un prop opcional para placeholder
  • El prop selected puede ser null para no tener seleccionada una opción por defecto

Implementación Dropdown

Los cambios están hechos de tal manera que las implementaciones actuales no deberían tener problemas. Si se quiere hacer uso de lo implementado, a continuación se muestra un ejemplo:

interface ListItem {
  id: number;
  value: string;
  label?: string;
}

const complexityOptions = [
  { id: 1, value: t('easy'), label:"easy"},
  { id: 2, value: t('medium'), label:"medium" },
  { id: 3, value: t('complex'), label:"complex"},
]

const [complexity, setComplexity] = useState<ListItem|null>(null)

<SelectDropdown 
  title={t('remediationComplexity')} 
  items={complexityOptions}
  selected={complexity}
  placeholder="placeholder"
  onChange={setComplexity}
/>

Resultados de ejemplo:

[jllanosg]

OJO con los commits @iTzGooDLife , modificaste 3 archivos con un mensaje genérico, tratar de ser más "atómico" con los commits, de manera en que se entiendan los cambios mirando el blame.

[iTzGooDLife]

OJO con los commits @iTzGooDLife , modificaste 3 archivos con un mensaje genérico, tratar de ser más "atómico" con los commits, de manera en que se entiendan los cambios mirando el blame.

No sé si corresponde, no se puede mencionar todo lo que se modificó de cada archivo en el mensaje de commit, por lo mismo se especifica completamente en el PR.

[jllanosg]

OJO con los commits @iTzGooDLife , modificaste 3 archivos con un mensaje genérico, tratar de ser más "atómico" con los commits, de manera en que se entiendan los cambios mirando el blame.

No sé si corresponde, no se puede mencionar todo lo que se modificó de cada archivo en el mensaje de commit, por lo mismo se especifica completamente en el PR.

Si corresponde, los commits son independientes del PR; sirven para tener un historial de los cambios, pero si pusheas todo en un solo commit se pierde la gracia de esto. En el peor de los casos puedes hacer un commit por archivo

https://dev.to/samuelfaure/how-atomic-git-commits-dramatically-increased-my-productivity-and-will-increase-yours-too-4a84

[iTzGooDLife]

OJO con los commits @iTzGooDLife , modificaste 3 archivos con un mensaje genérico, tratar de ser más "atómico" con los commits, de manera en que se entiendan los cambios mirando el blame.

No sé si corresponde, no se puede mencionar todo lo que se modificó de cada archivo en el mensaje de commit, por lo mismo se especifica completamente en el PR.

Si corresponde, los commits son independientes del PR; sirven para tener un historial de los cambios, pero si pusheas todo en un solo commit se pierde la gracia de esto. En el peor de los casos puedes hacer un commit por archivo

https://dev.to/samuelfaure/how-atomic-git-commits-dramatically-increased-my-productivity-and-will-increase-yours-too-4a84

ok