I like feedbacks, in this way i can improve my front-end abilities! feel free to give me good constructive feedback.
Muhammad Ahmad Hassan
@hassanahmadpAll comments
- @nofear1985Submitted over 3 years ago@hassanahmadpPosted over 3 years ago
Hello Fadi,
Hope you are alright... You did a great job, however, You lacked at some points. Which near to me is due to a lack of theoretical knowledge... I recommend you to have a strong search on the following concepts because these are very crucial:
-
Z-index
-
Z-index scoping ( This is hard to find but solves a lot of problems )
-
Flex-box
-
Grid
Moreover, you have used accessive media queries which is not an efficient way and will take a lot of your time. I recommend following along with some youtube channels and see how they write code... What I can see is you are trying to run before trying to walk, and in this way, you are getting a hard time running...
I recommend slowing down a bit and try to understand the crucial concepts of HTML and CSS...
I hope this helps, Happy Coding!
0 -
- @Fawkes11Submitted over 3 years ago
This is the last challenge for newcomers (free) you learn a lot with these practices.
@hassanahmadpPosted over 3 years agoYou did a great work!
just add
transition: all .2s ease;
to the button so that it nicely animates instead of just jumping between colors...other than that... your work was great.
0 - @sijandahalSubmitted over 3 years ago
Tried to create a pixel-perfect Design, Any Feedback will be appreciated
Thank you
@hassanahmadpPosted over 3 years agoYou did it well!
Although it is not pixel perfect, but with respect to UX principles, Your work is more user friendly... For a second I got confused, that which one is the design and which one is the solution.
There was one place where I felt that this could be a little bit better, where on the menu slide (mobile view) there is an overlay behind the menu, whereas in the design there is a box-shadow with a border-radius.
I hope this makes sense to you.
1 - @ShaFreak95Submitted almost 4 years ago
Any feedback on my scripts will be appreciated!
Just wondering, why my profile-card suddenly has a bit of transparency? I tried to figure out why but I seemed to not find the reason why.
@hassanahmadpPosted almost 4 years agoWell done Shafiq!
You have done a great job. Here are a few suggestions for you:
- You should use only those font-weights which are recommended in the
style-guide.md
. - For adding a background color you don't need to make a separate div and give it a background-color (correct me if I am wrong).
- Give the body a
height: 100vh;
andwidth: 100vw;
and there is no need to have a div classedbackground
and if it is necessary then you can give itheight: 100%;
andwidth: 100%;
.
Other than that everything is done perfectly... I have also done this project... Here is the link to that have a look and give me suggestions too.
1 - You should use only those font-weights which are recommended in the
- @agatablazejewskaSubmitted almost 4 years ago
I'd like to know:
- is it possible to structure the HTML and CSS in a better way?
- are there any CSS tricks I could have used for this assignment?
- your opinion overall :)
@hassanahmadpPosted almost 4 years agoWeldone Agata!
You have done a great job! Let me try to answer your Questions:
- Firstly, the is no need to have multiple HTML files for such a small project. One HTML file is more than enough. As far as CSS is concerned, I have you writing
margin: 0;
andpadding: 0;
and there are some other repetitions as well. What I do for that is I write the following piece of code at the start of my CSS* { margin: 0; padding: 0; box-sizing: border-box; }
This makes you skip many errors in the long run. - As covered in the previous answer, you should use universal selector (
*
), also SASS is a great tool to structure and organize code. I will recommend you watching this short introductory video about SASS. - My personal opinion for you is to learn SASS.
Tell me which code editor you use so that I may suggest more things...
3 - @gouppiSubmitted almost 4 years ago
Some SVGs were not aligned correctly (intended?), had to use relative/absolute positioning here. Is it ok to do so?
@hassanahmadpPosted almost 4 years agoGreat work is done Janne!
As you asked about using positioning, it is perfectly fine to use it. However, I found some issues with the mobile view as the main SVG part does not show properly, for that I suggest using fixed height in this case (although I discourage using fixed heights)... also there is one minor issue too that is there is no
alt
attribute for the image used... you need to work on the drop-down transition too...Other than that you have done a great job...
2 - @hernandoagfSubmitted almost 4 years ago
Hi guys, this is my first challenge here on Frontend Mentor, please, any feedback is more than welcome :)
@hassanahmadpPosted almost 4 years agoGreat work done!
I will suggest you to focus more on design and try to make it as close as possible to the design provided in the source file. Otherwise, every thing is great. Please let me know whether you have used BOOTSTRAP or not?
Regards,
1 - @MexitacoSubmitted almost 4 years ago
Hello, I would like to know if I can achieve the same with less code
@hassanahmadpPosted almost 4 years agoHi Mexitaco!
You have done great job. However, as you asked, if you want to write less code you can use SASS or SCSS instead of CSS.
Thank You.
0