@mostafa183Submitted about 1 month ago
What are you most proud of, and what would you do differently next time?
yes
P
Bojan Lukic
@bojanognjenAll comments
- @bojanognjenPosted about 1 month ago
Maybe you have to pay more attention to styling. As far as icons concerned, you can find them at ionicons website. Your main tag need larger padding.
0 - @FixitoSubmitted 4 months agoWhat are you most proud of, and what would you do differently next time?
.
What challenges did you encounter, and how did you overcome them?I don't know how to do the popup mobile view
What specific areas of your project would you like help with?The share popup mobile view
@bojanognjenPosted 3 months agoIt's excellent. I paid more attention to details but not succeded to make like this. It's great.
0 - @grmbyrnSubmitted almost 3 years agoWhat are you most proud of, and what would you do differently next time?
I did this a long time ago so I don't remember.
What challenges did you encounter, and how did you overcome them?I did this a long time ago so I don't remember.
What specific areas of your project would you like help with?I did this a long time ago so I don't remember.
@bojanognjenPosted 6 months agoMaybe at the beginning of main section should be used align-self for images width values of start and end. But I made same mistake. I don't think it is something important. It looks great.
0 - @auri222Submitted 6 months agoWhat are you most proud of, and what would you do differently next time?
I can finally understand grid now!
What challenges did you encounter, and how did you overcome them?Fonts, color, and background color are a mess in my CSS code.
What specific areas of your project would you like help with?Nothing much.
@bojanognjenPosted 6 months agoI think you should use smaller padding values.
Marked as helpful1 - @Emoji123-sSubmitted 7 months agoWhat are you most proud of, and what would you do differently next time?
Apart from having issues with css grid, none.
What challenges did you encounter, and how did you overcome them?I tried using with css grid, took me a whole lot of time, so I ended up using CSS flexbox instead, which was a lot faster and easier
What specific areas of your project would you like help with?CSS Grid
@bojanognjenPosted 6 months agoYou should add max-height: at your .card, due to differences among cards heights.
0 - @AbecarneSubmitted 7 months agoWhat are you most proud of, and what would you do differently next time?
This is the first responsive challenge that I submit and I tried to complete it using mobile first approach. Also, it was the first time for me using Sass for style and i spent some time lo learn how to set it up.
What challenges did you encounter, and how did you overcome them?I was stuck for a while trying to change images between the desktop preview and the mobile one. I also had difficulties to set the correct width for image and text in the desktop view.
What specific areas of your project would you like help with?How do you usually start a responsive challenge ? Are you doing it mobile first ? Also, feel free to share any advice or feedback :)
@bojanognjenPosted 7 months agoI would set larger padding in content area, and that would be excellent.
1 - @Kurt3zSubmitted 8 months agoWhat are you most proud of, and what would you do differently next time?
In my opinion, I think I was able to create a nice solution pretty close to the design that was provided. So overall, I am proud and satisfied with this challenge.
What challenges did you encounter, and how did you overcome them?I didn't find any part of this project too difficult. Maybe the responsiveness part could be a bitter, but in general I think this is a successful project.
What specific areas of your project would you like help with?I would be grateful if someone could provide some general critiques about the code organization, semantic HTML and some opinions on how the solution looks.
@bojanognjenPosted 8 months agoI think you should pay attention to padding and margin dimensions only. It looks ok.
0 - @lindajensenSubmitted 8 months agoWhat are you most proud of, and what would you do differently next time?
I'm most proud of the fact that I figured out the whole link-button-hove situation. I thought the links where suppose to be buttons (maybe they were?) but then I couldn't get the hover thing to work. So then I tried wrapping the buttons in anchor-tags which solved the problem but when I tried to validate my code I got a bunch of errors. But eventually I figured out that they were suppose to be links (duh it's called social links profile lol).
What would I do differently? I don't know. Maybe try and finish the challenge in one sitting, because it's difficult working on it and then not doing anythin on it for like a day and then picking it up again. But I have a full-time job and work has been crazy for a few days and sleep is a thing so...
What challenges did you encounter, and how did you overcome them?The button-problem. Which wasn't really a problem because they were suppose to be links. I'm guessing. I also had a problem with the bottom part of my buttons (before I turned them into links) disappering which I have no idea what that was about or how I solved it.
What specific areas of your project would you like help with?When are you suppose to use buttons and when are you suppose to use links? I mean links are links and buttons are buttons, but I mean what do you use buttons for? Is it just like submit, send, and stuff like that?
@bojanognjenPosted 8 months agoIt looks good. I think only you should add padding on main holder to 40 pixels from right and left side.
0 - @selmakoksalSubmitted 9 months agoWhat specific areas of your project would you like help with?
I would be grateful if you could tell me what I can improve.
@bojanognjenPosted 9 months agoI think it would be better to include this in you css file. body { display: flex; justify-content: center; align-items: center; }
0 - @imgaurav01Submitted 9 months ago
made qr code componet challange by frontend mentor. feedbacks are welcome
@bojanognjenPosted 9 months agoI think it would be better to remove text below and to shorten the width. Other things look pretty good.
0
