Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • juan-mentor• 120

    @juan-mentor

    Posted

    @curiosodilip

    Buenas noches Bilip kumar:

    Se nota que eres bueno en esto, por que me he quedado alucinado con tu solución de las cuatro tarjetas.

    Sobre todo con todas las carpetas que tienes, tu organización y cribado para principiantes , intermedios y expertos.

    Algunas de las cosas que haces las entiendo, otras no.

    No sabría como hacer un comentario útil para tu solución, así que felicidades y espero algún día poder tener tu nivel de organización y comprensión del código.

    Un saludo.

    0
  • abhi123-rgb• 50

    @abhi123-rgb

    Submitted

    What challenges did you encounter, and how did you overcome them?

    Nothing big changes I faced but I little bit struggle in doing the flex for large screens.

    What specific areas of your project would you like help with?

    what are mistakes I have done and how to enhance the code little more than this . It might be more responsive , flexible code and how can I improve more better than now.

    product preview cad

    1

    juan-mentor• 120

    @juan-mentor

    Posted

    Hola Abhi, tu código se ve fenomenal.

    • No se me ocurre como mejorarlo. A lo mejor se podría hacer algo más semántico, como por ejemplo:
               <main>
                 <article class= "previewCard">
                     <section class= "imgSection">...</section>
                      <section class= "details">...</section>
                  </article>
             </main>
    • Con lo de la imagen, yo también he tenido dificultades. Pero en mi caso porque puse un div vacío y le puse un imagen de fondo.
               <div class= "hero__image></div>

              .hero__image {
               background-image: url(" ruta de la imagen");
         }

    -La dificultad, radicó en que le puse una altura fija, que a la hora de aplicar flex me dió problemas.

    -Sinceramente, comparando códigos, creo que el tuyo es más completo. Siento no poder ayudarte más.

    Un saludo y sigue así.

    Marked as helpful

    0
  • Denner Silva• 40

    @Denner-94

    Submitted

    What are you most proud of, and what would you do differently next time?

    Estou bem satisfeito com resultado, ficou incrível no dispositivos móvel. Não mudaria exatamente nada do projeto. Eu gostei muito da solução.

    What challenges did you encounter, and how did you overcome them?

    Em relação ao tamanho da viewport, estava com dificuldade de fazer as adaptações para dispositivos moveis, nos testes sempre aparecia uma barra de rolagem por mais que iniciasse no first mobile. Ao testar em telas maiores os textos se estendiam demais, então limitei a largura da minha hero para 1000px, mas quando voltava para mobile aparecia a rolagem, confesso que ficava bem triste!..rsrs. Mas comecei a estudar e considerei um comentário feito no meu projeto anterior. Usar max-width, pronto! Tudo resolvido, todos os elementos se adaptaram.

    What specific areas of your project would you like help with?

    Gostaria muito de uma melhor solução para definir o padding de tabelas, para ficar próximo do projeto inicial que tem um espaço a esquerda das linhas da primeira coluna, tive dúvida se fiz de forma correta.

    Flexbox e max-width nas caixas

    1

    juan-mentor• 120

    @juan-mentor

    Posted

    Hola Denner Silva.

    Tu código se ve genial, salvo quizás porque observo un uso excesivo de estilos en línea y que la imagen es algo más alta que la del diseño.

    Pero no te preocupes, por que todo tiene solución. Hay un fandom donde te explica porque no hay que abusar de los estilos en linea. fandom estilos en línea.

    En cuanto a la imagen una forma de que las imágenes no se estiren aunque su altura no coincida con su anchura.

             object-fit: cover;

    La verdad es que he aprendido con tu @face, no sabía que se podían incrustar fuentes así.

    Vas por buen camino, felicidades por tu diseño.

    0
  • juan-mentor• 120

    @juan-mentor

    Posted

    • Hola Lucielub tengo algunas reflexiones que ofrecerte, si me lo permites:

      • La estructura de tu código es prácticamente perfecta. La única duda es si deberías haber utilizado la etiqueta <nav class ="LinksContainer">...</nav> en vez del <div class= "LinksContainer">...</div>.

    • Gracias a tu código he aprendido a utilizar el atributo role y aria-label de forma correcta.

    • Además es escalable y muy legible, con clases muy descriptivas que no hace falta ningún comentario o explicación de que hace cada elemento.

    • Hay algo que siempre me ha parecido algo contradictorio y es el hecho de resetear estilos, para luego poner margins y paddings a los elementos. No es una crítica, más bien es una reflexión que me hago cuando lo veo plasmado en un código.

              * {
               margin: 0;
               padding: 0;
         }
         .
         .
         .
         element1 {
                 margin-button: 1rem;
         }
        
           element2 {
                 padding: 2rem;
         }
        

         - La verdad es que no puede estar mejor hecho, es sencillo, legible y reutilizable.

    ###Enhorabuena, buen trabajo.

    Marked as helpful

    0
  • Mik2nd• 350

    @mik2nd

    Submitted

    What are you most proud of, and what would you do differently next time?

    i finish it

    What challenges did you encounter, and how did you overcome them?

    i cant center text author before, then i using flex for the container author and again margin-y: auto ,to fix it

    What specific areas of your project would you like help with?

    i try using 10% margin in my image container but still cant make it centered

    flex & auto margin-y for author

    1

    juan-mentor• 120

    @juan-mentor

    Posted

    • Hola Mik2do, tu código se ve estupendo, pero si me lo permites tengo una sugerencia para que centres el contenedor, con el siguiente código.
    body {
     display: flex;
    flex-direction: column;
   justify-content: center;
   align-items: center;
}
    • Recuerda que añadir alturas de forma estática, tiene el inconveniente de desbordamiento del contenido. un saludo y sigue así.
    0
  • juan-mentor• 120

    @juan-mentor

    Posted

    Desde mi punto de vista, quizás le falte algo de HTML semántico, por ejemplo:

     <article class="container">...</article>

    Por lo demás está muy bien estructurado, es perfectamente legible y me gusta como queda.

    En cuanto al Css, a mí en particular solo me gusta quitarle márgenes al body y suelo añadir

     * {
       box-sizzing: border-box;
}

Cuando añadimos una altura estática, podemos encontrarnos con que el contenido se nos desborde, por lo que si puedo evitarlo no establezco ninguna altura.
Pero en este caso te queda genial.
Es un buen trabajo por tu parte, sigue así.
    0
  • juan-mentor• 120

    @juan-mentor

    Posted

    • Gracias Daniel

    .Toda ayuda es bienvenida, sobre todo si estás empezando.

    *Aún no se utilizar este tipo de editor, pero poco a poco aprenderé.

    *Un saludo.

    1