When trying to style and , I learned that it sets the display to list-style. I was using Flexbox on it, and the bullets of the list would disappear, so it took me a bit to figure out why.
Johann Maldonado
@Johann-AlphaAll comments
- @jcgoodwin86Submitted about 2 months agoWhat challenges did you encounter, and how did you overcome them?@Johann-AlphaPosted about 2 months ago
I liked the solution you gave to the bullet points in unordered lists. I did it in a different way, but your solution is definitely better than mine, so I'm stealing it from you.
I would only suggest that you use the Figma file so that your dimensions closely match those in the design.
0 - @Aboelsayed78Submitted about 2 months ago@Johann-AlphaPosted about 2 months ago
Hi Mohamed, I hope you're doing well. Here, I will leave some suggestions in the hope you find them helpful.
- Using semantic HTML. In your links, instead of using a
div class="links"
you could have used a<nav></nav>
tag, which would make your code more accessible to people with disabilities. You could have also added a<main></main>
tag or a<section></section>
tag so that your page gets better SEO. - Describing images with the alt property. In your images, you can always write a descriptive text in your alt property, so that visually impaired people can hear the descriptions provided by their reader devices.
- Tailoring your design to different screen sizes. Although what you did was very accurate for the desktop size, the design had subtle changes for different sizes. For example, the width of the inner container changes depending on if it's desktop, tablet, or mobile.
- External CSS. You could have used a different file for the styles. That way you can further separate the structure of the page from the appearance.
I hope you continue doing great work!
0 - Using semantic HTML. In your links, instead of using a
- @nathan-perkinsSubmitted 2 months agoWhat are you most proud of, and what would you do differently next time?
I am proud that I was able to pick an area to focus on improvement and make progress in that area. In this project, I focused on being more organized with my CSS, and I think I accomplished this by using Kevin Powell's convention for organizing CSS properties, as well as using BEM to name elements' classes.
What specific areas of your project would you like help with?I would like to know if there are other things I should consider when it comes to organizing my CSS or just improving my CSS in general. I know that this smaller project doesn't necessarily require as much organization, but I wanted to aim for organization as if this were a much larger project.
@Johann-AlphaPosted 2 months agoHi Nathan, I hope you are doing well. I will briefly describe some things that you can consider for future projects:
- Using semantic tags. Your design very closely reflects the mandatory design. I would just only comment that in addition to giving the footer a class called
card__footer
you could have actually used a footer tag there, so that you improve the accessibility of your page. - Using unitless values for line heights. This one was advised by someone that commented my code. He said that I should stick to 1.5 (with no units) to ensure smooth adjusting of the letter sizes and better catering for the user's browser's preferences.
- Responsive design. I noticed that you didn't account for the mobile view. I realise that you thought that properties had the same value for the desktop view and the mobile view. The thing is that font sizes, some borders, etc., had different values for the mobile view. For that reason, it would have been useful to use
@media
so that you could add those different values.
I think your coding was excellent, and you definitely succeeded at implementing the BEM model. I'm clearly taking that away from you.
Marked as helpful0 - Using semantic tags. Your design very closely reflects the mandatory design. I would just only comment that in addition to giving the footer a class called
- @becakdaruratSubmitted 2 months agoWhat are you most proud of, and what would you do differently next time?
dalam pembuatan website, saya membuat dulu sebuah container yang saya buat pembungkusnya body, dengan lebar dan tinggi vw,vh agara relatif dengan lebar layar kita, didalamnya ada 2 kolom , kiri dan kanan , saya berikan juga padding , di bagian kanan / frame saya berikan margin auto , begitu juga dengan rectangel , karena pembungkusnya flex, didalamnya bisa / itemnya bisa kita kasih margin: auto; biar pas ditengah
@Johann-AlphaPosted 2 months agoHi, @Sofyan, I hope you are doing fine. Your design is full of creativity, but I think the goal was to reproduce the provided design as it appears, with no explanatory text.
Marked as helpful2 - @rtoddmSubmitted 8 months agoWhat are you most proud of, and what would you do differently next time?
This is a simple project but it my first non-tutorial project since learning to code so I am proud of simply being able to complete it on my own.
What challenges did you encounter, and how did you overcome them?The main difficulties that I encountered were learning to get the appropriate information from the Figma file and getting the page to display properly on GitHub Pages (it was a file path issue).
What specific areas of your project would you like help with?Have I completed this challenge as intended? The design file in figma does not show any additional background but the "preview" file shows an additional background applied behind the grey box background.
@Johann-AlphaPosted 8 months agoClick on the link I sent and it'll give you an idea of what you have to do. Do not worry about them saying there are mobile or desktop versions. When you do the desktop version, mobiles will accommodate the content automatically.
Marked as helpful0 - @rtoddmSubmitted 8 months agoWhat are you most proud of, and what would you do differently next time?
This is a simple project but it my first non-tutorial project since learning to code so I am proud of simply being able to complete it on my own.
What challenges did you encounter, and how did you overcome them?The main difficulties that I encountered were learning to get the appropriate information from the Figma file and getting the page to display properly on GitHub Pages (it was a file path issue).
What specific areas of your project would you like help with?Have I completed this challenge as intended? The design file in figma does not show any additional background but the "preview" file shows an additional background applied behind the grey box background.
@Johann-AlphaPosted 8 months agoHi @rtoddm. I think that you misunderstood the aim of the challenge. You shouldn't have replicated the screenshot. On the other hand, you should have created the page for desktop, and it would automatically adjust for mobiles.
My challenge isn't perfect and I have a lot of suggestions from our colleagues that I have to implement. I will share my URL so that you can see what it was all about. Click here for the QR code challenge
Cheers,
Johann.
Marked as helpful0