P
@Jason-Donmoyer
Abdellah el aajjouri
@abdellahelaajjouriAll comments
- @abdellahelaajjouri
Hey There , Youre Solution is Perfect well done , But I corrected the active satate of your elements in the mobile layout and also the max with of your container ; so I have added it for you . check out your code . Other than that everything is Very good . Good Jobe !
- @solvedbiscuit71@abdellahelaajjouri
Hey There , Youre Solution is Perfect well done , But in the mobil layout you did forget to add the code the active state of the bar (hover) so added it for you . check out your code . Other than that everything is Very good . Good Jobe !
Marked as helpful - @ColdLikeMcFlurry@abdellahelaajjouri
You doing a great job body , your solution is amazing 👌 . Keep up
Marked as helpful - @MohamedWagdy7@abdellahelaajjouri
Goog job Mohamed , nice looking solution . You just want to change the position of the image in the right and the content in left . Nice work . Keep up 👌
Marked as helpful - @osmannurierdogan@abdellahelaajjouri
Wow amazing job osman
- @catherineisonline@abdellahelaajjouri
Hey catherin , where did you learn best practice to use css units ?
- @artemis-tiwa@abdellahelaajjouri
Amazing job artemis , your solution looks perfect 👌👍
- @moheb2000@abdellahelaajjouri
Your card looks good , just decrease the height using max height and aligne it the content , Keep up 👌
Marked as helpful - @NatanielCosta@abdellahelaajjouri
Good job Nataniel , your solution looks amazing 👌 . just reduce a little of the filter contrast to see more of the img , Keep Up 👍
Marked as helpful - @hoomi88@abdellahelaajjouri
Good it's look amazing 👌👍
- @YanVictorSN@abdellahelaajjouri
Good job Victor your solution is amazing. You just want to reduce the opacity of the paragraph . Other than that all perfect. Keep Up with the good work 😜
- @HibaBadran122@abdellahelaajjouri
Hey @HibaBadran122 We can't see your solution to fix that problem Try vercel, or netlify. They are user friendly: just push changes to github and they take care of hosting and relaunching of the pages.
Marked as helpful - @fborlido@abdellahelaajjouri
Hey @fborlido We can't see your solution to fix that problem Try vercel, or netlify. They are user friendly: just push changes to github and they take care of hosting and relaunching of the pages.
- @wlscafe@abdellahelaajjouri
Hey @wlscafe We can't see your solution to fix that problem Try vercel, or netlify. They are user friendly: just push changes to github and they take care of hosting and relaunching of the pages.
- @AchmadRiyadi@abdellahelaajjouri
Hey men Good job working this challenge. You solutions looking very good to place the footer you can just use the margine property or give him positions absolut then change the placement where you want. Keep up with the good work
- @mochaaa18@abdellahelaajjouri
It's all good ,Your solution look the same as the challenge Just a little detail you can decrease the opacity and the with of the ligne in top of the stats Good job 👌 Keep up
- @clamonty@abdellahelaajjouri
To fix accessibility issues you need to wrap everything in main tag instead of using div, as div doesn't have any semantic meaning. Also, Frontend Mentor requires at least one h1 tag which isn't always possible in some components.
- @melwynt@abdellahelaajjouri
To fix accessibility issues you need to wrap everything in main tag instead of using div, as div doesn't have any semantic meaning. Also, Frontend Mentor requires at least one h1 tag which isn't always possible in some components.
Marked as helpful
