Luka Glonti

Latest comments

• 

  acabrie•170

  @acabrieSubmitted over 1 year ago

  

  Responsive Stats preview card using Grid and Flex

  1

  51
  

  Luka Glonti•3,440

  @lack21Posted over 1 year ago

  Hi there, that's a good work right there, but I have some recommendations!

  • Instead of width: 310px in .container add width: min(500px, 90vw), it will adjust width to 500px and if screen's width will be less than 500px .container will adjust to it!

  • Replace margin: auto in .container to margin: 1.5rem auto to add some additional space to the top and bottom!

  Good Luck! 👍

  Marked as helpful

  0
• 

  Bibiwei Pere•490

  @Bibiwei-PereSubmitted over 1 year ago

  

  Sunnyside agency landing page built with CSS and JavaScript

  1

  30
  

  Luka Glonti•3,440

  @lack21Posted over 1 year ago

  Great work 👍, but I have a suggestion for you!

  • Remove background-size: 100% 100% from class .hero and add these styles instead

  background-size: cover;
  background-position: center;
  

  like this background image will fit well on all devices!

  0
• 

  YoungZV•260

  @YoungZVSubmitted almost 2 years ago

  

  Order Summary Component

  1

  40
  

  Luka Glonti•3,440

  @lack21Posted almost 2 years ago

  Good job 👍, but I have a suggestion!

  • Add background-size: contain to the body, so background image would fit well on larger devices!

  Marked as helpful

  1
• 

  supakorn klampod•220

  @BKvipSubmitted almost 2 years ago

  

  Interactive rating component

  #react

  2

  50
  

  Luka Glonti•3,440

  @lack21Posted almost 2 years ago

  Good job 👍, but I have a suggestion!

  • Instead of padding: 1rem 1.2rem in the class .point use these styles

  width: 3rem;
  height: 3rem;
  

  like this circles will be round!

  Marked as helpful

  0
• 

  Jacqui Michel•60

  @Jacqui72Submitted almost 2 years ago

  

  3-column-preview-card-component using Flexbox, CSS

  1

  40
  

  Luka Glonti•3,440

  @lack21Posted almost 2 years ago

  Excellent work 👍, but I have a suggestion!

  • Remove these styles from .container

  margin-top: 100px;
  height: auto;
  

  and add min-height: 100vh instead, like this layout will be centered!

  2
• 

  levy_m•60

  @LevyMatiasSubmitted almost 2 years ago

  

  Order summary card

  #bem

  1

  110
  

  Luka Glonti•3,440

  @lack21Posted almost 2 years ago

  Great work 👍, but I have some recommendations!

  • Replace height: 100vh to min-height: 100vh in the body, the difference is that, when you set height: 100vh to something, that means it won't be bigger than that, it might cause some problems in the future, so better approach is to set min-height: 100vh, like this in case content is overflowing container will adjust to it!
  • Remove width: 100% from the body, body is a block level element which by default takes full space available, so width: 100% does nothing here!

  Marked as helpful

  1