All comments

• 

  Moussa•540

  @Mahmoud974Submitted 12 months ago

  Bonjour, voici mon projet reactjs et typescript avec l'API pour modifier son lien

  

  URL shortening API landing page

  1

  20
  

  Eya Gargah•230

  @eyagargahPosted 12 months ago

  Bravo ! juste un petit conseil lorsque vous cliquez sur le bouton "Copy" changez le texte en "Copied" pour que l'utilisateur sache que le lien a bien été copié

  1

  This comment has received 1 replyShow
• 

  Gracious•20

  @KuunibeGraciousSubmitted about 2 years ago

  

  JS Fetch API

  1

  50
  

  Eya Gargah•230

  @eyagargahPosted about 2 years ago

  Nice job ! just a few remarks 1- the solution is not optimised for mobile screen. In future you might want to add a media query to solve it. 2- to solve the warning issue always add alt to buttons

  0
• 

  amal roshan•70

  @amalazimroshanSubmitted over 2 years ago

  I had some trouble with returning the value from the function. It showed pending promise. so I wrote the code for displaying the data in the same function that fetch the data

  

  Advice generator using plane CSS and JavaScript

  1

  40
  

  Eya Gargah•230

  @eyagargahPosted over 2 years ago

  Good job! regarding your question the promise will always log pending as long as its results are not resolved yet so to solve it you should call .then() on the promise to capture the results regardless of the promise state (resolved or still pending):

  Marked as helpful

  1
• 

  Paula Gayle•70

  @P-GayleSubmitted over 2 years ago

  Hello! This is my first project on FrontEnd Mentor. I mainly used CSS Flexbox and a little Grid. couldn't get the card centre aligned correctly using Flexbox so any feedback on my alignment (or anything else that you noticed) would be very welcome. I completed the project without the Figma file so the design might not be exactly the same. Thanks for looking!

  

  Order Summary Component Using HTML and CSS flexbox and Grid

  1

  50
  

  Eya Gargah•230

  @eyagargahPosted over 2 years ago

  Great work! I have some advice about the code 1 - Instead of <div class="card"> , replace that with <main> element.it will give your code more semantic meaning and Fix the issues that appear in Accessibility Report. 2- when you hover over the change button and cancel order they should change color so add a hover state

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Abdul•150

  @Abdul-SJSubmitted about 3 years ago

  Submitting again as the first attempt's link isn't working. Kindly check this out and help me getting better.

  

  First Attempt ! Order Summary using HTML & CSS Flexbox

  2

  10
  

  Eya Gargah•230

  @eyagargahPosted about 3 years ago

  Nice work on the design but when i hover over the proceed button it dosent change color , also for the cancel link and change link

  1