Lifestyle app with advances in javascript.#22
Conversation
Html skeleton and graphic resources
Screen to upload images to the store with functionality
…contain login with facebook
Download database changes and local storage for new publications and …
The user who logs in can already see his profile and other users' posts.
Profile window shows modal window that allows editing and deleting
Modal window works correctly in profile
HectorBlisS
left a comment
HectorBlisS
left a comment
There was a problem hiding this comment.
Edith tu trabajo es muy consistente, es una lastima no poder ver el trabajo de ninguna de tus compañeras pero al mismo tiempo demuestras la solides con la que puedes crear una aplicación web, felicidades por el trabajo duro y por no rendirte. La modularización puede mejorar y también la semántica pero se ve que vas por buen camino.
| <button id="logo" href="#/" class="btnNav"><img src="images/logoWhite.png"> </button> | ||
| <button id="browser" href="#/browser" class="btnNav"><img src="images/browser.png"></button> | ||
| <button id="favorites" href="#/favorites" class="btnNav"><img src="images/whiteHeart.png"></button> | ||
| <button id="oupladContent" href="#/newpost" class="btnNav"><img src="images/camera.png"></button> | ||
| <button id="profile" href="#/profile" class="btnNav"></button> |
There was a problem hiding this comment.
Todas las imagenes deben llevar un alt="" con un caption para ayudar con el SEO, siempre!
| </footer> | ||
| <!-- The core Firebase JS SDK is always required and must be listed first --> | ||
| <script src="https://www.gstatic.com/firebasejs/7.14.0/firebase-app.js"></script> | ||
| <script src="https://www.gstatic.com/firebasejs/7.14.0/firebase.js"></script> |
There was a problem hiding this comment.
Con este script no necesitamos el de analitycs ya lo incluye
|
|
||
|
|
||
| // Your web app's Firebase configuration | ||
| var firebaseConfig = { |
| @@ -0,0 +1,16 @@ | |||
| import showOupladWindow from './uopladContent.js' | |||
| let content= document.querySelector(".root"); | |||
There was a problem hiding this comment.
Agreguemos poquita más semantica a los nodos contentSection o homeContent
| /*Flexbox*/ | ||
| } | ||
|
|
||
| .btnNav{ |
There was a problem hiding this comment.
En css es importante no usar camelCase, se prefiere con guiones 'preview-image'
| import singIn from './login.js' | ||
| import profile from './profile.js' |
There was a problem hiding this comment.
Bien por las importaciones, pero podríamos meter mas funciones en el mismo archivo e importar deconstruyendo {signIn, profile} además son relacionados.
| let header= document.querySelector(".header"); | ||
| let staticMenu= document.querySelector("#staticMenu"); | ||
|
|
||
| export function showOupladWindow (){ |
There was a problem hiding this comment.
Muy buen nombre de función
| <section> | ||
| <div id="divFile"> | ||
| <img id="labelImg" src="images/plusImage.png"> | ||
| <input type="file" accept="image/*" class="addImage" id="addImage"> | ||
| </div> | ||
| <div id="previewImage"></div> | ||
| <label> | ||
| Título | ||
| <input type="text" placeholder="Máximo 70 carácteres"/> | ||
| </label> | ||
| <label> | ||
| Descripción | ||
| <input type="text" id="title" placeholder="Máximo 200 carácteres"/> | ||
| </label> | ||
| <button id="toPost">PUBLICAR</button> | ||
| </section>` |
There was a problem hiding this comment.
La identación es importante al usar template strings, ya que nos permite usar espacios para la legibilidad.
| firebase.firestore(); | ||
| window.addEventListener('load', init); No newline at end of file | ||
| // Your web app's Firebase configuration | ||
| const firebaseConfig = { |
There was a problem hiding this comment.
Bien el cambio de var a const
| const content = document.querySelector(".root"); | ||
| let header= document.querySelector(".header"); | ||
| let staticMenu= document.querySelector("#staticMenu"); |
There was a problem hiding this comment.
Es indiferente (para mi y la mayoria de devs) el uso de let o const pero es mejor usar solo 1 para ser consistentes
EdithOrt
force-pushed
the
master
branch
2 times, most recently
from
bddf96a to
d53704f
Compare
2 participants
No description provided.