All comments

• 

  Bartosz•360

  @bkozeckiSubmitted over 2 years ago

  Any feedback would be appreciated :) Thanks

  

  Slider Solution

  2

  30
  

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  Looks great, but would look better if you hide overflow-x "html{overflow-x. hidden;}".

  1
• 

  Ævie•150

  @General-EvieSubmitted over 2 years ago

  Advice appreciated :)

  

  FAQ Accordion

  1

  20
  

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  Make it a little responsive. Inside the container create another div like card and place all the content inside that div, then set the height of the container to auto.

  Marked as helpful

  0
• 

  nikoProg•120

  @nikoProgSubmitted over 2 years ago

  Any advice about the layout in general?

  The button starts glowing upon hovering, but if you click it, the glow gets stuck, no matter where you move the mouse, until you click on something else. How can I make the button glow and stop glowing when I stop hovering? My solution is a bit of a hack, using ::after pseudoelement, so I am not sure how to fix it.

  This was also my intro to typescript. I installed it, because I like that strongly-typed languages make it easier to fix some mistakes, so I hoped this could help. I didn't have much use of it yet, since the only script needed here is to fetch advice from the API.

  

  Advice Generator App using typescript

  3

  60
  

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  Make your .advice-container centered. *{margin: 0; padding: 0; box-sizing: border-box}. Remove body{padding: 10%;} and set body height to body{height: 100vh}

  0
• 

  Sugam Sahu•100

  @iSugamSubmitted over 2 years ago

  This is my 2nd solution to the frontend mentor challenge.

  Any advice would be really helpful. Thank you :)

  

  Completed with HTML5, CSS3, Grid, FlexBox, JavaScript, and API

  2

  100
  

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  Hello! looks great, just add a little JavaScript and it will be much better

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Javier Morales•110

  @JavierA-MorHerSubmitted over 2 years ago

  I learned a lot about how I can connect with an API and how it works :)

  

  Advice Generator

  1

  60
  

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  Looks great, but I suggest not reloading the page every time, but making API calls inside the Event Listener

  Marked as helpful

  1
• 

  Daniel•80

  @duokobiaSubmitted over 2 years ago

  It was interesting coding. Feedbacks are highly appreciated.

  

  Product preview card component using CSS grid.

  2

  40
  

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  Add a little responsive design and change the size of the card

  Marked as helpful

  0
• 

  Gavin Lang•30

  @GavinLang9Submitted over 2 years ago

  If there is anything I can improve on, please let me know!

  

  Used flexbox and grid

  2

  40
  

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  try to improve your responsive design

  1

  This comment has received 1 replyShow
• 

  Yokke•620

  @JexinteSubmitted over 2 years ago

  Hey ,

  The challenge is not seenable have some trouble to link it to github page with vue but if you try it on your computer it's full responsive.

  

  Vuejs , Flexbox

  1

  40
  

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  Try uploading your Vue site to another service like Vercel to open it in a browser

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Nima•20

  @nima-caSubmitted over 2 years ago

  fasdfadsfd

  

  Tip calculator

  1

  10
  

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  slightly different from the design =)

  0