Michael Leopold• 20
@michaelleopold
Submitted
Still so much to learn and fix. Any advice would be great.
Jonathan Martínez
@Jommartinez
All comments
- Jonathan Martínez• 3,275
@Jommartinez
Posted
There is nothing wrong, the only thing, remember that the IDs must be unique and you are repeating them
3 - Nikkita Leftwich• 140
@nlefty
Submitted
I could use some feedback on the sizing of the desktop image. I'd like the image height to be that of the viewport. Thanks
Jonathan Martínez• 3,275@Jommartinez
Posted
You can use one container 50% width, height 100%. Also image width and heigth 100% with object-fit: cover
2 - Gagan Arora• 110
@Scoro6
Submitted
Hey It is my first challenge It is not perfect but I'll love to hear how can I make this perfect. Thanks.
Jonathan Martínez• 3,275@Jommartinez
Posted
To be your first challenge it is perfect :) keep it up and you will see that great projects you can develop very soon
6 - Farah Aly• 240
@f-m-a-ly
Submitted
Feedback appreciated! Thanks!
Jonathan Martínez• 3,275@Jommartinez
Posted
it's not bad at all. maybe I would correct the accessibility notices to go perfectly :)
6 - Keside Ezeala• 570
@kcde
Submitted
Are there better ways to do it?
Jonathan Martínez• 3,275@Jommartinez
Posted
Hello :) It is very well developed. In the end there are no better or worse solutions. Each developer has a different point of view.
maybe the hamburger on the menu would put it as a <span> so that you don't get the accessibility error, otherwise everything is ok
5 - Eklavya Sharma• 255
@sharmaeklavya
Submitted
I would really appreciate if you could spare few mins of yours and provide a feedback.
Thank you so much.
Jonathan Martínez• 3,275@Jommartinez
Posted
Its fine, but for the white box you can use the pseudoelement ::after for example: ::after { content: ""; position: absolute; top: 69px; right: 0; border-top: 25px solid #ffffff; border-left: 25px solid transparent; } (or similar for your solution)
Marked as helpful
5 - maujor• 230
@Maujor
Submitted
Feedback are wellcome
Jonathan Martínez• 3,275@Jommartinez
Posted
It is fine, but I have seen a problem ... If you want to do a search from the input search, it does not work with the keyboard enter. you need to click on the icon. I think it is highly recommended to enable keyboard use :)
5 - Georgios Trialonis• 130
@GTrialonis
Submitted
It took me a very long time to complete to the stage it is now. Some CSS code may not be necessary. Mobile active state not as recommended but working anyway. This is my second web site.
Jonathan Martínez• 3,275@Jommartinez
Posted
Oh! To be your second challenge it is very well developed .. Keep it up and with practice it will be very easy to do advanced challenges :)
6 - Arun• 285
@arun-roberts
Submitted
First time using BEM. How did I go applying the structured naming? It was a little getting used to but in the end I really enjoyed the order it brought to my thought process. Not to mention the potential for reusability.
I can see that it might be a little overboard for a project of this size and smaller. Is this how others feel or is it applicable to pages of any size?
Jonathan Martínez• 3,275@Jommartinez
Posted
From my point of view, BEM is a very good methodology for large projects or small projects that are going to scale over time ... But for small projects, I don't really see it as necessary.
5 - Rohail Taha• 355
@rohailtaha
Submitted
How can I make the bottom right corner of the white pop up (one that says "185 GB left") appear triangular?
Jonathan Martínez• 3,275@Jommartinez
Posted
You can use the pseudoelement ::after for example: ::after { content: ""; position: absolute; top: 69px; right: 0; border-top: 25px solid #ffffff; border-left: 25px solid transparent; } (or similar for your solution)
6
