Pull request squad 4 (Gris, Abi, Nat)#32
Conversation
reloadercf
left a comment
reloadercf
left a comment
There was a problem hiding this comment.
Me gusto mucho que su proyecto esta modularizado, se que las 3 trabajaron en el proyecto pero la mayoría de los commits fueron subidos por una persona, me hubiera gustado ver algunas funciones en testing, recomiendo es tener cuidado de no repetir funcionalidades, por lo demás su código esta bien estructurado y me gusta =)
| @@ -0,0 +1,5 @@ | |||
| /* | |||
There was a problem hiding this comment.
Literalmente subieron todo el Framework bootsrap xD hay otras alternativas que les recomiendo probar y consiste en instalar el framework pero utilizando los cdn y con solo las 3 ó 4 lineas de codigo en el archivo html funciona todo el framework de estilos.
| <meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1"> | ||
| <meta http-equiv="x-ua-compatible" content="ie-edge"> | ||
| <link rel="stylesheet" type="text/css" href="screens/style.css"> | ||
| <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css" integrity="sha384-Gn5384xqQ1aoWXA+058RXPxPg6fy4IWvTNh0E263XmFcJlSAwiGgFAW/dAiS6JXm" crossorigin="anonymous"> |
There was a problem hiding this comment.
tienen instalado 2 veces bootstrap, si revisas la carpeta CDMX009-Social-Network/src/Bootsrap/ encontraras la descarga de bootstrap y en particular esta invocando el framework, recomiendo se queden con esta linea de codigo
| <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css" integrity="sha384-Gn5384xqQ1aoWXA+058RXPxPg6fy4IWvTNh0E263XmFcJlSAwiGgFAW/dAiS6JXm" crossorigin="anonymous"> | |
| <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css" integrity="sha384-Gn5384xqQ1aoWXA+058RXPxPg6fy4IWvTNh0E263XmFcJlSAwiGgFAW/dAiS6JXm" crossorigin="anonymous"> |
| <link rel="stylesheet" href="screens/style.css"> | ||
| <script type="module" src="https://unpkg.com/ionicons@5.0.0/dist/ionicons/ionicons.esm.js"></script> | ||
| <script nomodule="" src="https://unpkg.com/ionicons@5.0.0/dist/ionicons/ionicons.js"></script> | ||
| <link href="https://fonts.googleapis.com/css?family=Varela|Vollkorn&display=swap" rel="stylessheet"> |
There was a problem hiding this comment.
puedes instalar las fuentes en tu archivo css y con eso empezar a utilizarla de inmediato, intenta esa opción.
| <script type="module" src="https://unpkg.com/ionicons@5.0.0/dist/ionicons/ionicons.esm.js"></script> | ||
| <script nomodule="" src="https://unpkg.com/ionicons@5.0.0/dist/ionicons/ionicons.js"></script> | ||
| <link href="https://fonts.googleapis.com/css?family=Varela|Vollkorn&display=swap" rel="stylessheet"> | ||
| <title>Document</title> |
There was a problem hiding this comment.
Aquí va el nombre de la red social
| <title>Document</title> | |
| <title>Red social de las morraz</title> |
|
|
||
| <header> | ||
| <div class="container-fluid borde"> | ||
| <div class="row"></div> |
There was a problem hiding this comment.
si no se usa no se pone
| <div class="row"></div> | |
| max-width: 80px; | ||
| } | ||
|
|
||
| #borde1 { |
There was a problem hiding this comment.
| #borde1 { | |
| #borde-1 { |
| text-justify: auto; | ||
| } | ||
|
|
||
| .navBar { |
There was a problem hiding this comment.
| .navBar { | |
| .nav-bar { |
| box-shadow: 0 2px 4px rgb(127, 255, 146); | ||
| } | ||
|
|
||
| #border2 { |
There was a problem hiding this comment.
| #border2 { | |
| #border-2 { |
| text-align: center; | ||
| } | ||
|
|
||
| .contenedor { |
There was a problem hiding this comment.
| .contenedor { | |
| .container { |
| border: solid 2px; | ||
| } | ||
|
|
||
| .logo-brand { |
4 participants
No description provided.