Ediongsenyene Joseph I.
@IEdiongAll comments
- @Rahul-JS-DevSubmitted 4 months ago
- @elisilkSubmitted 4 months agoWhat are you most proud of, and what would you do differently next time?
I'm most proud of taking on the challenge of the "Ideas to test yourself" about noticing that the font sizes in this project are slightly smaller in the mobile layout and finding a way to reduce font size for smaller screens without using media queries. I was able to learn about how to use
What specific areas of your project would you like help with?calc
andclamp
andvw
units to implement what I think is a nice solution for this.I feel like my text sizes, letter spacing, and line heights never quite match the solution. Not sure what the issue is there. I had access to the Figma file for this challenge, and so tried to design based around the given units in that design file rather than the jpg screenshots. So not really sure the best way to think about approaching that. Or maybe I made a mistake in reading the Figma file or in implementing the typography in some way.
- @Yashi-Singh-1Submitted 4 months ago@IEdiongPosted 4 months ago
Hello Yashi π, great job in completing this challenge. I noticed your card's (".frame" class)
box-shadow
doesn't match that of the given Figma design along with some minor spacing values.The' box-shadow' value is
0px 25px 25px rgba(0, 0, 0, 0.05)
. Here is a link to a picture showing how to get CSS values and more from a Figma design (assuming you don't have the Figma dev mode).Also, Frontend mentor have a great article on how to work with a design file, I recommend you check it out to learn more about this topic.
I hope this was helpful to you, Shalom!
Marked as helpful0 - @tloxiuSubmitted 10 months ago
Hi! As recommended I used Pixel Perfect Extension, and it was really easy to use. Feedback welcome guys and girls!
@IEdiongPosted 10 months agoHello Roksana π, great job in completing this challenge. Some suggestions:
- the
div
with the.social-media
class can be converted into an unordered list (ul
). Theul
tag conveys semantic meaning. - the
button
tag used for the social links can be converted to anchor tags (a
) as they are meant to be "links to social sites" after allπ€·ββοΈ
Just picture your site without the styles, when someone visits it they should see a picture of the lady, her name, location and what she does, then a list of links to her social media profile. This is just to help you with the HTML semantic tag.π
Lastly, I love the way you brought out the design with pixel precision, how did you achieve this? You mentioned that you used a "Pixel Perfect Extension", can you share the link to the extension download page?
I hope this was helpful, Shalom!
Marked as helpful1 - the
- @DrKelvinSubmitted almost 3 years ago
I didn't understand where the responsiveness fits
@IEdiongPosted almost 3 years agoHello Kelvinho π, great job in completing this challenge. Here are some suggestions:
-
the
.nft-title
should contain a heading tag enclosed by the<a>
. eg<a class='nft-title'><h1></h1></a>
. This should help fix one of the accessibility issues raised here. -
you could fix the responsiveness issue by using media queries. If you're not familiar with media queries check out this post on responsive design.
-
Also, look into the accessibility and HTML issues raised here.
I hope this was helpful to you, Shalom!
Marked as helpful1 -
- @emmcgillSubmitted almost 3 years ago
My second submission. Potential improvements include putting a transition on the hover effects and centering the icon a little better.
@IEdiongPosted almost 3 years agoHello Erik π, great job in completing this challenge. Some suggestions:
-
the image of the eye doesn't show when I hover over the image. You should fix this.
-
the image should be wrapped inside an
<a>
tag, since it's a clickable element. -
Look into the accessibility issues raised here. Click on the
read more
to gain more insight into the problem.
I hope this was helpful, Shalom!
Marked as helpful1 -
- @asifsalimSubmitted almost 3 years ago
i try my best to complete the project with minimal project.suggest me more if you have any suggestion :)
@IEdiongPosted almost 3 years agoHi Asif,
The stats could be an unordered list rather than a div. This is for semantics. Picture someone going through this your page without their CSS enabled, how will they consume the content. Think about this.
I hope this was helpful to you, Shalom!
Marked as helpful1 - @halamhSubmitted almost 3 years ago@IEdiongPosted almost 3 years ago
Hi @halamh π, some suggestions here:
-
Set a
max-width
inpx
on the<main>
tag in order to make the content look consistent even as the screen sizes increase beyond1440px
. -
Look into the accessibility issue raised here. It's quite easy to fix all you have to do is just click on the learn more and read through. You would be glad that you did.
I hope this was helpful to you, Shalom!
Marked as helpful1 -
- @VidottizzzSubmitted almost 3 years ago
I found this challenge a little bit hard, but i completed it, the code is kinda messed up, can somebody tell me how can i improve this code?
@IEdiongPosted almost 3 years agoHello Vidottizz π, nice solution. Some suggestion:
- Your blend mode doesn't seem to match that of the design, you can refer to my solution for this challenge to see how I fixed this.
I hope this was helpful, Shalom!
Marked as helpful0 - @KhaledAhmedKassemSubmitted almost 3 years ago
Hope you could give me feedback. Thanks!
@IEdiongPosted almost 3 years agoHello Khaled π, this is a well built component. Some suggestions:
-
The images are meant to have a colored border around it, refer to the design.
-
This is a personal suggestion: I feel you could play around with the layout at different screen sizes even though it wasn't given in the design. You could refer to my solution for this challenge to see how the layout changes at different screen sizes.
-
Also look into the accessibility issues raised here.
I hope this was helpful, Shalom!
Marked as helpful0 -
- @KhaledAhmedKassemSubmitted almost 3 years ago
Hope you could give me feedback. Thanks!
@IEdiongPosted almost 3 years agoHello Khaled π,
-
The content of the
.box-two p
should be an unordered list of items and not a paragraph that is formatted to look like a list of items -
The sign up button should have a
box-shadow
as shown in the design -
The
border-radius
should be set on the parent component of the whole component rather than being set on each section of the component. -
Also, look into the accessibility issues raised here.
I hope this was helpful, Shalom!
Marked as helpful0 -
- @vikrantmallaSubmitted almost 3 years ago
Drops some tips..
@IEdiongPosted almost 3 years agoHello Vikrant π, this is a well built component. Some suggestions:
-
The image in the
.cardImg
should be wrapped in an<a>
tag, along with the content of the<h1>
and the content of the.cardCreator span
. This is because those items seem to be clickable (links). -
Set the hover state of the
<h1>
and.cardCreator span
. The color should change on hover according to the design spec.
I hope this was helpful, Shalom!
0 -