Skip to content
OverviewSolutions
19
Comments
32
Mohamed Ragheb
@mohamed-fm

All comments

  • khaduj || خدوج•670
    @khaduj03
    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
    Mohamed Ragheb•560
    @mohamed-fm

    What a fantastic code! It's really impressive!

    Marked as helpful
  • saramdkour•10
    @saramdkour
    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
    Mohamed Ragheb•560
    @mohamed-fm

    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
  • Mohamed Ragheb•560
    @mohamed-fm

    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
  • Mohamed Ragheb•560
    @mohamed-fm

    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
  • Mohamed Ragheb•560
    @mohamed-fm

    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
  • BozJR•260
    @BozJR
    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
    Mohamed Ragheb•560
    @mohamed-fm

    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
  • zh4r•150
    @zh4r
    What are you most proud of, and what would you do differently next time?

    N/A

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

    N/A

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

    N/A

    NFT Card with added popup preview

    #accessibility#sass/scss
    1
    Mohamed Ragheb•560
    @mohamed-fm

    Great work! Consider adding some padding on the left and right in the body. Overall, the code is clean and impressive.

    Marked as helpful
  • Mohamed Ragheb•560
    @mohamed-fm

    Great design, but the overlay should also have the class rounded-lg.

  • P
    Lo-Deck•3,700
    @Lo-Deck
    What are you most proud of, and what would you do differently next time?

    Hi here is my solution for Scoot-multi-page-website. I used HTML, CSS, JS.

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

    It was to do a bigger website than usual and keep a connection between all the css, background, and different size, and to finally have a good responsiveness between mobile, tablet, desktop.

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

    Feel free to leave any comments or suggestion.

    Thanks.

    Scoot-multi-page-website

    1
    Mohamed Ragheb•560
    @mohamed-fm

    Great work! However, you might want to use Tailwind for a project like this. Tailwind will help you tremendously and will reduce the time needed to code a project like this. Overall, great work!

  • Mohamed Ragheb•560
    @mohamed-fm

    Great work! However, you might want to use containers. Since you are using Tailwind, I recommend checking the Tailwind documentation for the container class name: container docs. Overall, good job!

    Marked as helpful
  • leodev•480
    @hangtime319
    What are you most proud of, and what would you do differently next time?

    It was a big challenge, as I applied CSS techniques that I'm not used to using, but I learned a lot.

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

    The biggest challenges were in the JS code to make the image carousel and the product cart.

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

    Any tips on how to improve my JS code, I'd appreciate it.

    Ecommerce product page using HTML, CSS and JavaScript

    #accessibility
    2
    Mohamed Ragheb•560
    @mohamed-fm

    You did an amazing job! To enhance your JavaScript code further, consider using classes. For reference, you can take a look at my code. Overall, your JavaScript is great, and your CSS is very clean. However, you might want to make your project responsive for screen sizes between 1200px and 500px.

    Marked as helpful
  • Mohamed Ragheb•560
    @mohamed-fm

    great work keep it going <3

  • Mohamed Ragheb•560
    @mohamed-fm

    Great job! It is well-designed and organized. However, wrapping everything in a container would be helpful to ensure your design is responsive and fully functional on larger screen sizes. It would look something like this:

    .container {
    width: 100%;
    padding: 0px 15px;
    margin: auto;
}

@media  (min-width:768px) {
    .container {
        width: 750px;
    }
}
@media  (min-width:992px) {
    .container {
        width: 970px;
    }
}
@media  (min-width:1200px) {
    .container {
        width: 1170px;
    }
}

    You can adjust the display property of the container class to suit your needs, using options like flex, grid, or others.

    Marked as helpful
  • Weverson Godinho•490
    @Godinhoweverson
    What challenges did you encounter, and how did you overcome them?

    I encountered a challenge in checking if the email is valid or not, so I overcame it by creating a function. This function takes a parameter 'e', which represents the email value, and uses a regular expression to match the email against a specific format, returning either true or false.

    Contact Form

    #accessibility
    1
    Mohamed Ragheb•560
    @mohamed-fm

    great work keep it going <3

    Marked as helpful
  • Mohamed Ragheb•560
    @mohamed-fm

    Amazing js code keep it going <3

  • Mohamed Ragheb•560
    @mohamed-fm

    That is amazing! I love how you brought a lot of ideas to this project. Definitely, I will check your code as a reference. <3

  • Mohamed Ragheb•560
    @mohamed-fm

    Amazing work, keep it going <3

  • Brandon Jr Cortez•190
    @BrandonJr20
    What are you most proud of, and what would you do differently next time?

    He mejorado en cuanto a la responsividad de mis proyectos y considero que en cuanto a javascript y las funcionalidades voy aprendiendo.

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

    Manipular de manera correcta las cuentas.

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

    Me gustaria mejorar en cuanto a la redundancia de los estilos, creo que defini muchas veces algunas reglas y considero que eso puede afectar el rendimiento del sitio en proyectos más grandes.

    Calculadora de propinas con css puro y javascript

    1
    Mohamed Ragheb•560
    @mohamed-fm

    amazing <3