-
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
02 components #4
Conversation
GomezMorinFrontEnd/src/App.jsx
Outdated
import IconTitle from "./components/IconTitle" | ||
import DropdownText from "./components/DropdownText" | ||
|
||
|
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.
Usar Prettier
* @returns {JSX.Element} - The CardImage component JSX element. | ||
*/ | ||
const CardImage = ({icon}) => { | ||
return ( |
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.
Usar Prettier
* @param {string} icon3 - The url of the third image. | ||
* @returns {JSX.Element} - The GalleryCard component JSX element. | ||
*/ | ||
const GalleryCard = ({icon1, icon2, icon3}) => { |
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.
Usar prettier
* @param {string} headerText - The text content for the header. | ||
* @returns {JSX.Element} - The IconTitle component JSX element. | ||
*/ | ||
function IconTitle({ image, headerText }) { |
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.
Usar arrow function: const IconTitle = ({ image, headerText }) => {}
* @param {string} icon3 - The url of the third image. | ||
* @returns {JSX.Element} - The GalleryCard component JSX element. | ||
*/ | ||
const GalleryCard = ({icon1, icon2, icon3}) => { |
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.
Mejor usen un arreglo de icons y un map
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.
Apruebo estos cambios
Descripción
<-- Agregar una descripción concisa de lo que hace el pull request !-->
6 componentes han sido terminados:
CardImage
GalleryCard
TextContainer
TextButton
IconTitle
DropdownText
Resumen de cambios
<-- Enlistar los commits hechos del pull request !-->
4 commits=
Commit inicial = Card Image, GalleryCard y TextButton
2do commit = Text Container
3ro commit = IconTitle
4to commit= DropdownText
Notas (opcional)
<-- Agregar notas para explicar aspectos adicionales del pull request !-->
N/A