Add contribution (issue: #57 Implementar interacciones principales) #59
Conversation
annalbirena
commented
May 11, 2022
annalbirena
commented
- Scroll to contact section
- header links redirect to a page
- Update link of OurDreamTeam button
- Add link to OutGitHub button
package.json
Outdated
| @@ -11,7 +11,7 @@ | |||
| }, | |||
| "dependencies": { | |||
| "@directus/sdk": "9.7.1", | |||
| "@mantine/core": "4.1.0", | |||
| "@mantine/core": "^4.1.0", | |||
There was a problem hiding this comment.
Al parecer es mejor práctica usar dependencias fijas (sin expresiones generales (^)). Buscaré algo de referencias para pasarlas por aquí, ¿tú qué piensas?
|
Anna, estamos probando deepsource para asegurar/mejorar la calidad del código. Es una pieza de software bastante interesante, remarca los posibles problemas en el código fuente, es bastante más "inteligente" que un analizador estático clásico. Dejo abajo una captura de los issues detectados en este PR, si puedes, revísalo, a ver si te hace sentido, algunos son issues relativamente simples de solventar, te dejo el link con detalles (nombre del archivo, número de línea, etc.) aquí.
|
|
Claro Bregy, lo estare revisando! |
src/components/ui/Header/index.tsx
Outdated
| @@ -177,7 +182,7 @@ export function MinskyLandingHeader({ links }: MinskyLandingHeaderProps) { | |||
| > | |||
| {colorScheme === "dark" ? <Sun size={18} /> : <MoonStars size={18} />} | |||
| </ActionIcon> | |||
| <Button radius="md">Contact</Button> | |||
| <Button radius="md" onClick={() => scrollTo({ y: 3229 })}>Contact</Button> | |||
There was a problem hiding this comment.
Quizás esto sirva de algo: https://www.delftstack.com/howto/javascript/scroll-to-element-in-javascript
