Desktop Version Done.

Community feedback

• 

  noga•15

  @noga-aviatorPosted over 4 years ago

  First you forgot that grey square in the background (hint: use position: absolute and z-index), secondly your padding is a bit messy. And about the device image, you need to change it depending on width (two way, javascript or a div with background-image property)

  0

  ---

  

  Shubham Verma•225

  @ShubhamVerma1811Posted over 4 years ago

  @nogayoutub Yeah,It is difficult to assume paddings and font-sizes without having access to XD files. Padding of the nav links right? There is grey background provided with files, So I assume I need to create it. I need to remove img tag of device mockup and instead add it as bg image ?

  1

  

  noga•15

  @noga-aviatorPosted over 4 years ago

  @ShubhamVerma1811 The grey background isn't provided. You need to make it with a div using height, width and position absolute. z-index will help you too. And i was also talking about the padding of the text, they are too close.

  0