diff --git a/evidencia-validacion.png b/evidencia-validacion.png new file mode 100644 index 0000000..cbde736 Binary files /dev/null and b/evidencia-validacion.png differ diff --git a/src/App.css b/src/App.css index 77f05b6..8ba07bd 100644 --- a/src/App.css +++ b/src/App.css @@ -1,8 +1,11 @@ +/*En general los estilos están bien pero agregaría otros colores solo por dinamismo y en algunas partes como en el about me*/ body { background-color: black; color: #d1d1d1; font-family: "Tajawal", sans-serif; font-size: 1.2rem; + /*Agregaría centrado de texto*/ + text-align: center; } h1, diff --git a/src/App.js b/src/App.js index 19b8848..103eb0e 100644 --- a/src/App.js +++ b/src/App.js @@ -10,6 +10,8 @@ import Projects from "./components/projects"; import Hobbies from "./components/hobbies"; import Contact from "./components/contact"; +//Buen uso de React para dividir propiamente las secciones de la pagina + function App() { return (
diff --git a/src/components/contact.js b/src/components/contact.js index 1d05685..81e66e9 100644 --- a/src/components/contact.js +++ b/src/components/contact.js @@ -3,6 +3,8 @@ import { FontAwesomeIcon } from "@fortawesome/react-fontawesome"; import { faWhatsapp, faGithub } from "@fortawesome/free-brands-svg-icons"; import { faEnvelope } from "@fortawesome/free-solid-svg-icons"; +//Tambien acá me areció bien que haya dejado el footer adherido a la parte inferior de la pantalla + class Contact extends React.Component { render() { return ( diff --git a/src/components/navigation.js b/src/components/navigation.js index 6e9e4cb..907a207 100644 --- a/src/components/navigation.js +++ b/src/components/navigation.js @@ -2,6 +2,8 @@ import React from "react"; import Navbar from "react-bootstrap/Navbar"; import Nav from "react-bootstrap/Nav"; +//Me parece bien que haya dejado este componente fijo en la pantlla + class Navigation extends React.Component { render() { return ( diff --git a/src/index.css b/src/index.css index ec2585e..4cb3a90 100644 --- a/src/index.css +++ b/src/index.css @@ -5,6 +5,8 @@ body { sans-serif; -webkit-font-smoothing: antialiased; -moz-osx-font-smoothing: grayscale; + /*Agregaría centrado de texto*/ + text-align: center; } code {