Skip to content
OverviewSolutions
11
Comments
10
ahmed safwat
@ahhmedsafwat

All comments

  • Thanos Kalaitzis•100
    @Thanos-M11Submitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    I used the context API on this project combined with a react-router to manage UI state and I think that on such an app I would use the context API again.

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

    It took me some time to find a way to change the svg icons according to light-dark mode. Initially i was importing the svg icons as img files and I was not able to change their colors. The solution i came up with was to include the svg script directly to the html.

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

    Any comment on any approach would be much appreciated trying to improve myself and apply best practices.

    Rest Countries API with React, React Router, Responsive, Modular CSS

    1
    ahmed safwat•230
    @ahhmedsafwatPosted 5 months ago

    That is a pretty good website man keep it up, my advice to you is that you didn't create any pagination which I think is essential for this kind of website otherwise than that it's so good keep it up <3

    0
  • Kalin Angelov•360
    @kalin-angelovSubmitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    Building it.

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

    The REST API that was provided was going down all the time and i couldn't make the challenge with Fetch Api so i had to use the DB Json file.

    Countries With Color Theme Switcher - Vite, React

    1
    ahmed safwat•230
    @ahhmedsafwatPosted 5 months ago

    best one i have ever seen so far

    0
  • Mihai Niculae•250
    @mihainrsSubmitted 6 months ago
    What are you most proud of, and what would you do differently next time?

    Worked some more on the overall responsiveness, and got to code the burger menu again

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

    Not very familiar with grid, so I struggled a bit to wrap my head around how it works but I figured it out in the end!

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

    Any feedback's good!

    News homepage

    1
    ahmed safwat•230
    @ahhmedsafwatPosted 6 months ago

    it's perfect, the only issue I came across was your navbar you don't have to give the <nav> a fixed width because in mid to small screens it expands out of the screen but when you remove the fixed width the issue is no longer there and also the three cards in the bottom they grow out of the screen to make them wrap in mid to small screens so they don't compress on themselves I know this might seem not perfect to you but its really good just 2 modifications 😂😂

    Marked as helpful
    1
  • Raptor0x1•210
    @Raptor0x1Submitted 6 months ago
    What are you most proud of, and what would you do differently next time?

    I use the local fonts next time and use the flexbox for the card description.

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

    I had difficult in the centering the card and changing the box-shadow size when hovered over the card

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

    I guess the link in the card description is not clickable as the whole card is covered by the anchor tag of the h2 tag using ::after. So need help with this.

    Responsive Blog Preview Card Using Flexbox and pseudo-content trick

    1
    ahmed safwat•230
    @ahhmedsafwatPosted 6 months ago

    the issue is that the :: after of your title covers the whole card, to solve this remove the::after and It will work just fine

    0
  • Aaron Smith•280
    @medic-codeSubmitted 6 months ago
    What are you most proud of, and what would you do differently next time?

    Getting the design mostly accurate. i think i would've probably done less on getting the inbetween sizes working correctly and sticked to the desktop and mobile view only.

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

    Whilst i was able to get the desktop and mobile versions to the design, inbetween sizes of height were incredibly difficult to figure out how to fix the overflowing top part of the card. It looked like the flexbox was actually causing this problem and was unable to find a solution to center the card without, despite thinking about floats or using auto margin.

    Additionally getting the padding between the bullet point and the text in the list was really challenging, solutions made to seem this was relatively easy but nothing seemed to work, using ::marker, using ul or li padding/margin. Not quite sure why!

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

    Better responsiveness on intermediate desktop sizes, the card overflows on my own laptop for example.

    A better way to add padding on the bullet point lists.

    Responsive recipe card

    3
    ahmed safwat•230
    @ahhmedsafwatPosted 6 months ago

    hello there, the photo isn't visible in mid to x-large screens

    0
  • ahmed safwat•230
    @ahhmedsafwatPosted 11 months ago

    great work my dude <3

    1
  • Nakul•140
    @Official-NakulSubmitted about 1 year ago

    I should have followed mobile first approach

    URL Shorter

    1
    ahmed safwat•230
    @ahhmedsafwatPosted about 1 year ago

    what did you use to make the login page ?

    0
  • ahmed safwat•230
    @ahhmedsafwatPosted about 3 years ago

    there are about 3 issues in this solution because I forget to add the font awesome file but its okay

    1
  • Jon Erik Ullvang•160
    @UllvangSubmitted about 3 years ago

    How did you solve adding color filter over image? I used CSS filter but it did not work so well.

    React JS

    1
    ahmed safwat•230
    @ahhmedsafwatPosted about 3 years ago

    there are very different ways to do it, I do it with adding overlay using after or before as you love and added some opacity to the color until it is like the design and this is the color (i have used #501674 and this is the opacity I used too 0.72 ) I am sorry if make some mistakes in the lang don't forget the z-index to make the overlay over the photo

    Marked as helpful
    1
  • ahmed safwat•230
    @ahhmedsafwatPosted about 3 years ago

    man how could I upload my solution

    0