Marko Nikolajević

Latest comments

• 

  Vishal•330

  @vshal-chSubmitted almost 4 years ago

  

  Mobile first build and Responsive website built using HTML, CSS and JS

  1

  10
  

  P

  Marko Nikolajević•1,095

  @MarkoNikolajevicPosted almost 4 years ago

  Hi Vishal

  you did a good job on this challenge, I noticed a few things you could check and maybe fix.

  • the background image is missing
  • check font sizes to make them look more as the design
  • check input size
  • you could add hover effect on the button
  • check the report to see accessibility and html issues

  Keep on coding :)

  1
• 

  Marco•185

  @devMarco02Submitted almost 4 years ago

  

  Four Card Feature using Sass

  2

  10
  

  P

  Marko Nikolajević•1,095

  @MarkoNikolajevicPosted almost 4 years ago

  Hi Marco

  You did a really good job on this challenge. Your solution is responsive and everything looks well.

  To make your solution looks more like the design, I would suggest to add a white background color on your cards

  Your code is clean and well documented with comments :)

  Keep on coding :)

  1
• 

  idkbit•350

  @idkbitSubmitted almost 4 years ago

  

  css variables, flexbox, grid

  2

  10
  

  P

  Marko Nikolajević•1,095

  @MarkoNikolajevicPosted almost 4 years ago

  Hi idkbit

  you did a good job for this challenge, I 've a few feedbacks for you.

  • I would wrap a elements inside a <button> in this case
  • You should add a hover effect on buttons, you can just add :hover pseudo class on link class and apply hover styles. To make hover effect smoother you could add transition on it

  Keep on coding :)

  1
• 

  Orkhai Dickson•320

  @orkhaiSubmitted almost 4 years ago

  

  3 column preview card made with HTML and CSS

  1

  10
  

  P

  Marko Nikolajević•1,095

  @MarkoNikolajevicPosted almost 4 years ago

  Hi Orkhai

  You did a great job as a first challenge! Your solution is responsive and everything works well.

  I've a few suggestions for you

  • You don't need to set the width on the body
  • You could add some transition on buttons to have a smoother hover effect. If you decide to di that you should add border: 1px solid transparent on your buttons because you have a border on hover and this will make a better effect

  Anyway you good job and keep on coding with fun :)

  0
• 

  tediko•6,700

  @tedikoSubmitted almost 4 years ago

  

  Countdown timer - Mobile first, Flexbox, Sass, JS, backface-visibility

  5

  207
  

  P

  Marko Nikolajević•1,095

  @MarkoNikolajevicPosted almost 4 years ago

  Hi tediko

  You did really a great job on this project! I like animations especially ones on social icons :). Your code is clean and clear. Good job adding sr-only for accessibility, this is a good plus.

  Your work is almost pixel perfect from the design :)

  Good job and keep on coding :)

  1
• 

  Antoine•10

  @ChampyAntoineSubmitted almost 4 years ago

  

  desktop version without fonts

  1

  10
  

  P

  Marko Nikolajević•1,095

  @MarkoNikolajevicPosted almost 4 years ago

  Hi Antoine,

  your solution look good. Nice job on finishing it!

  To include fonts you can use the link tag in your html file

  <link rel="preconnect" href="https://fonts.gstatic.com">
  <link href="https://fonts.googleapis.com/css2?family=Inter:wght@300;400;700&family=Lexend+Deca&display=swap" rel="stylesheet">
  // here you import both fonts needed for this project
  

  and the set the font-family in your css file

  font-family: 'Inter', sans-serif;
  font-family: 'Lexend Deca', sans-serif;
  

  you have to set specific font where it is needed for example

  h1 {
    font-family: 'Inter', sans-serif;
  }
  

  Keep on coding and have fun :)

  1