Mohamed Ragheb

Latest comments

• 

  khaduj || خدوج•670

  @khaduj03Submitted 3 months ago

  What challenges did you encounter, and how did you overcome them?

  One challenge I faced was making the modal automatically scroll to the specific reward section when the "Select Reward" button was clicked This was a bit tricky for me to figure out :/

  What specific areas of your project would you like help with?

  I would really appreciate it if someone could review my code and guide me on how to write cleaner and more efficient code. I’m eager to improve my coding skills and learn best practices :)

  

  crowdfunding-product-page-main

  #react#typescript#framer-motion

  1

  80
  

  Mohamed Ragheb•560

  @mohamed-fmPosted 3 months ago

  What a fantastic code! It's really impressive!

  Marked as helpful

  0
• 

  saramdkour•10

  @saramdkourSubmitted 3 months ago

  What specific areas of your project would you like help with?

  Evaluate the method of writing codes, whether it is clean or not, and if there is anything I can do to improve the method of writing code

  

  social links profile website

  1

  30
  

  Mohamed Ragheb•560

  @mohamed-fmPosted 3 months ago

  The code you have is great, but I recommend adding the following CSS to normalize the properties:

  * {
      margin: 0;
      padding: 0;
      box-sizing: border-box;
  }
  

  This will enhance and standardize the default CSS values. I suggest reading this article for further reference.

  Additionally, consider using max-width and min-width instead of fixed widths to improve the design's responsiveness. Utilizing rem and em units will also contribute to a more flexible layout.

  While the solution is well-designed, there is a discrepancy in the color scheme. Please refer to the style guide file for color references.

  I have also completed this challenge, so feel free to check it out if you're interested.

  Marked as helpful

  0
• 

  olaf•235

  @Ziorko87Submitted 3 months ago

  

  HTML TAILWINDCSS JAVASCRIPT -> CLICK ON CHANGE :)

  1

  60
  

  Mohamed Ragheb•560

  @mohamed-fmPosted 3 months ago

  The code is great, but you should remove the overflow hidden property from the card to ensure the design works properly on tablets and medium screen sizes. Additionally, using a container to wrap everything could improve the layout since you’re using Tailwind CSS. You can find more information about the container class in the documentation.

  Marked as helpful

  0
• 

  Rowland•320

  @rowleksSubmitted 3 months ago

  

  time-tracker

  #accessibility#fetch#pure-css#solid-js

  1

  30
  

  Mohamed Ragheb•560

  @mohamed-fmPosted 3 months ago

  This is a well-structured JavaScript code you have, but incorporating error handling can enhance best practices. Here’s an improved version:

  try {
    const response = await fetch('APIURL');
  
    if (response.ok) {
      console.log('Promise resolved, and the HTTP status is successful.');
      // ...handle the response data here
    } else {
      // Custom messages for specific HTTP error codes
      if (response.status === 404) throw new Error('404: Not Found');
      if (response.status === 500) throw new Error('500: Internal Server Error');
      // For any other server error, throw a general error
      throw new Error(`Error: ${response.status}`);
    }
  } catch (error) {
    console.error('Fetch error:', error);
    // Output example: "Fetch error: 404: Not Found"
  }
  

  This version maintains clarity and effectively handles potential errors when making a fetch request.

  Marked as helpful

  0
• 

  P

  Teano Goss•180

  @tcaturani-gossSubmitted 3 months ago

  

  Newsletter Sign Up With Success Message Using Flexbox and JavaScript

  1

  30
  

  Mohamed Ragheb•560

  @mohamed-fmPosted 3 months ago

  This code is excellent, but it's essential to remove the width property from the body. Failing to do so will lead to overflow issues that could impact the design.

  Marked as helpful

  1
• 

  BozJR•260

  @BozJRSubmitted 3 months ago

  What are you most proud of, and what would you do differently next time?

  Im most of proud of being able to complete this but it was difficult in terms of figuring out the JS and then putting my thoughts onto the code. got there in the end with some help.

  What challenges did you encounter, and how did you overcome them?

  using the JS and getting it to do what i wanted.

  What specific areas of your project would you like help with?

  Just need more practical PRACTISE with JS.

  

  Article Preview Component using basic HTML, CSS & JS

  2

  30
  

  Mohamed Ragheb•560

  @mohamed-fmPosted 3 months ago

  Great code! However, you might want to avoid giving the article__content-container a height property. This will allow it to adjust automatically for smaller screen sizes and will help resolve the issue with the mobile-social-sharing-bar, making it stick to the bottom.

  Additionally, you can use overflow-x: hidden; on the body and correctly position the share-shape-container to prevent any overflow. Overall, it's excellent code!

  Marked as helpful

  0