Guy Ezra

Latest comments

• 

  Jose Juarez Rodrigues Dos Santos•20

  @juniorjuarezSubmitted about 3 years ago

  

  NFT-preview-card-component

  1

  40
  

  Guy Ezra•615

  @ezraguyPosted about 3 years ago

  Hey @juniorjuarez, Great Work! 1 suggestion that I have is adding a little bit of transition to the hover state of the image by adding this CSS:

  .image div{
     transition: .3s ease;
  }
  

  Marked as helpful

  0
• 

  Abhik•4,840

  @abhik-bSubmitted about 4 years ago

  

  Ping coming soon with staggering animations

  3

  40
  

  Guy Ezra•615

  @ezraguyPosted about 4 years ago

  hey @abhik-b, Great work! about the setTimeout on the animations, maybe animation-delay in your CSS is the solution you are looking for.

  Marked as helpful

  1
• 

  env-y•20

  @env-ySubmitted over 4 years ago

  

  Gatsby js, styled-components

  4

  11
  

  Guy Ezra•615

  @ezraguyPosted over 4 years ago

  Hey, @Env-y your project looks very good! I have only a suggestion. in the 960px width break point maybe make the pricing cards half the width of the screen I think it will make them look better and more readable. other than that it looks great! Happ coding! 😄

  1
• 

  Erik Tangvik•255

  @Etang131Submitted over 4 years ago

  

  Intro component with sign-up form

  3

  10
  

  Guy Ezra•615

  @ezraguyPosted over 4 years ago

  Hey @Etang131, I have 1 suggestion to offer, I think the image component should not be in the container. because the container has a max-width:1440px and when you look at the website with a desktop monitor there are white spaces on the sides because the screen is wider than 1440px. what I would do is would take the image out of the container CSS as a background image and add it as an img tag and give it the following CSS:

  .background-img{
  position:'absolute';
  width:100%;
  height:100%;
  z-index:-1;
  }
  

  other than that it looks great! Happy coding! 😄

  1
• 

  Abhik•4,840

  @abhik-bSubmitted over 4 years ago

  

  Article preview component with HTML , CSS , JS

  3

  20
  

  Guy Ezra•615

  @ezraguyPosted over 4 years ago

  Hey @abhik-b, great job!

  I have 2 suggestions:

  1. like @axevldk said around 300px width the text in the card starts to overflow.

  2. In the 1023px breakpoint I would make the card's width around 55% percent because, in my opinion, 80% percent looks a bit stretched and I would add min-width to the card.

  @media screen and (max-width: 1023px)
  article {
      grid-template-areas:
          "image image"
          "details details";
      grid-template-columns: 1fr;
      grid-template-rows: 200px auto;
      height: 70%;
      width: 55%;
      min-width: 300px;
  

  other than that it's great!

  1
• 

  ApplePieGiraffe•30,545

  @ApplePieGiraffeSubmitted over 4 years ago

  

  Room Homepage with CSS Transitions and JS (GSAP) Animations

  9

  2414
  

  Guy Ezra•615

  @ezraguyPosted over 4 years ago

  Hey @ApplePieGiraffe, your solution is absolutely beautiful, WOW. very professional! great job!

  1