- I found the media query part difficult and a bit challenging just getting acquainted with it.
- What better can the project be achieved and clean without writing long code
suhaib52
@suhaib52All comments
- @DammyDSubmitted almost 2 years ago@suhaib52Posted almost 2 years ago
You did it in a very good manner. Your code is well structured and readable.
It would be nice if you start using semantic html tags, these tags would reduce the accessibility issues.
You can also use :root to define the colors and then call them by using var() in your stylesheet. When you use the colors as a variable than your css code will look more precise.
Marked as helpful1 - @almeida883Submitted almost 2 years ago
One more exercise done, thank you for giving me constructive criticism.
@suhaib52Posted almost 2 years agoYou did it in a very good manner. Your code is well structured and readable.
It would be nice if you start using semantic html tags, these tags would reduce the accessibility issues.
You can also use :root to define the colors and then call them by using var() in your stylesheet.
I do not understand why you create separate style sheet for mobile and desktop.
Marked as helpful1 - @sawsan-codeSubmitted almost 2 years ago
all feedbacks are welcome thank you .
@suhaib52Posted almost 2 years agoGood job! Your code is well structured and readable. For html, try to use semantic html tags. Semantic html tag would help to improve the accessiblity.
Marked as helpful0 - @stefan123283Submitted almost 2 years ago@suhaib52Posted almost 2 years ago
It would be better if you use the semantic html tags because these tags would reduce the accessibility issues.
You can also define the body in more details to make it more better such as; body { width: 100%; height: 100vh; display: flex; align-items: center; justify-content: center; font-family: 'Karla', sans-serif; background-color: var(--grayishBlue); font-size: 16px; }
Also use the margin 0 auto on the card id so it should be in the center of the page.
It would be nice if you use classes more as compared to the ids.
Marked as helpful0 - @kathrryn1Submitted almost 2 years ago
All Feedbacks will be greatly appreciated
@suhaib52Posted almost 2 years agoYour code is well structured and easily readable. Good job! You can define the colors by using :root at the start of the style sheet and then call them by using var().
Marked as helpful0 - @myrojoyleeSubmitted almost 2 years ago
Done a week of coding!
I had the hardest time with lining up the background images. I ended up just doing what Mr. Coder on YT (https://www.youtube.com/watch?v=NZpG9EBKYWc) did after trying for hours and not fully understanding how to get it to where I needed it to be...
I also do not understand how the profile picture got to be on top of the header background, because I tried all sorts of z-index lines getting nowhere, and then it magically happened in the midst of deleting unnecessary lines, so I'm not even sure what I did.
Thanks so much for all the comments so far in the other challenges I've done...I'm still trying to get better at not ending up with a million lines of code, so if you see me repeat myself excessively, please let me know!
@suhaib52Posted almost 2 years agoI think you need to use the position relative and then use bottom property to move the profile picture.
2 - @Shraddha-V22Submitted almost 2 years ago
1- Assuming the actual width and properties of elements was a bit challenging. 2- I am unsure whether I have written an efficient code or not. 3- There are couple of doubts about best practices and whether I am practicing them or not.
@suhaib52Posted almost 2 years agoYour code is looking clean. It would be great if you use classes as compared to the ids in the html, because use of classes considered best practice as compared to the ids.
Marked as helpful0 - @ajchinasaSubmitted almost 2 years ago
It is a great start as a first project. Please let me know if there is a line of code you would have written differently.
@suhaib52Posted almost 2 years agoIt would be great to use semantic tags than the div, because semantic tags improve the accessiblity.
You can also try :root in the css to define the colors at the start of the file then you can use var() to call the color.
Marked as helpful0 - @ViniciusMassariSubmitted almost 2 years ago@suhaib52Posted almost 2 years ago
Your code is looking clean and easy to read. Try to use :root to define the colors once at the start of the css sheet and then just call them by using color: var(). It is also nice if you define the font in the style sheet rather than in the html file. For image it is also good practice to add the width and height attribute in the image tag.
Marked as helpful1 - @lovishduggalSubmitted over 2 years ago
Share your all suggestion.. Thanks
@suhaib52Posted over 2 years agoHello Lovish,
Your solution is very clean. I like your approach. It would be better if you add external style sheet. That would make your code more readable. Best of luck for next time!
Marked as helpful0