Design comparison
SolutionDesign
Solution retrospective
What are you most proud of, and what would you do differently next time?
Each link opens in a new tab.
What challenges did you encounter, and how did you overcome them?I use flexbox to center the content.
Community feedback
- @grace-snowPosted 8 months ago
Hi, I hope this feedback is helpful 😁
- The image alt must be properly written. See the excellent post in the resources channel on discord about how to write good alt text.
- If opening links in a new tab (unexpected behaviour) you must notify screen reader users that will happen. Include some visually hidden text in a span on each link that says (opens in a new tab) for example.
- Get into the habit of including a full modern css reset at the start of the styles in every project. Andy Bell or Josh Comeau both have good ones you can look up and use.
- Never limit the height of elements that contain text, including the body. This is making your solution overflow the page at the top and bottom on some screen sizes and is especially bad when users increase their font size (I'll add some Screenshots to demonstrate in discord for you). Min-height is fine to use on elements that contain text in descedents, if you need to, but height is not. The only elements that sometimes require a height are icons/images.
- Add text align center to the component.
- ⚠️ Very important: Font size must never be in px
- Unless using gap there is no real benefit to making the component into a flex column. It's not wrong but there is no benefit either so these kind of things will bloat the css.
- The link and hover styles should be on the anchors not on the list items. The whole anchor is what is clickable so that is where the hover must go.
- This challenge does not need a media query at all. The container should have a max width in rem (and optionally a width 100%).
- If you do need media queries in future challenges they must be defined in rem or em not px.
0@francloboPosted 8 months agoHi, @grace-snow
Thank you for your feedback. I have completed some required changes, but I prefer to keep others like that.
- Done! It is good to know there are excellent posts in the resources channel on Discord about how to write good
alt
text, but it would be useful if you shared the link to the specific resource. - When you click on the
PREVIEW SITE
button it opens the site without showing an alert to the user. Is the alert mandatory as you said? Is there an ISO standard? - I don´t know who Andy Bell or Josh Comeau are. I noticed you used this comment several times in your feedback, I found 138 results on Discord, but it would be better if you shared a link.
- Please use a constructive tone. "Never..." sounds rude, doesn't it? It sounds better "Limiting the height of elements that contain text, including the body... Min-height is fine to use on elements that contain text in descedents, if you need to, but height is not. The only elements that sometimes require a height are icons/images."
- Yes, madame. I did it. Oh, sorry. It sounds like an order. "Please add...", "kindly add...", "It is better..." sound polite.
- Done! Thank you for the link to the article that clarifies the reason for the change requested.
- I used a flex column because the position of the elements is vertical otherwise it will appear in a row. Does it make sense?
- I got it, thank you for the feedback.
- I recommend using "In my opinion..." when the feedback isn't based on the challenge's requirements.
Your challenge is to build out this social links profile and get it looking as close to the design as possible. You can use any tools you like to help you complete the challenge. So if you've got something you'd like to practice, feel free to give it a go.
- Done! Thank you for the link. That's a good feedback.
0@grace-snowPosted 8 months ago@franclobo
- Screen reader users see no alert. They just get taken to a new page unexpectedly and without warning which is why the label must be amended (or better yet don't open in a new tab at all because users can choose to do that themselves if they wish).
- Looking up resources is a key skill for any developer. I give hours and hours of free consultancy every week. Do not ask me to do even more. I will signpost to where you need to look but it is yo to you to go and learn.
- Never means never, that's all. It's essential, absolute. I will use direct communication on all essential items that must not be missed.
- Block elements stack vertically by default. So there is no need for flex at all unless using gap to create inner vertical spacing (it would be margins by default).
- This is based on challenge requirements.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord