Latest solutions
Latest comments
- @SebastianLaka@rafaeltakano
Hello there 👋
Great job on getting started, please keep going! There are a few things I think it would help.
As pointed out, we can't see your exemple on the live site, the index.html file should be at the root of your project folder.
After you replace the index.html file to the correct place do not forget to change url locations and stylesheet links like
<link rel="stylesheet" href="/css/style.css">
to the correct path.Your BEM looks good and the structure is correct.
Hope this helps 🤞
Marked as helpful - @APdev88@rafaeltakano
Hello there 👋
Great job on your solution, it looks great!
About the accessibility issues, your
html
should have at least one main landmark and the page should be contained byhtml
landmarks.About sass, good job using sass, you used sass nestings really well. To make it shine even more, take a look at the BEM pattern design for css.
Hope this helps ✌
Marked as helpful - @briansegura15@rafaeltakano
Hi there! ✌
Great job on your solution! It looks great and it's responsive!
About your question, the
html { margin: 0 }
will make the<html>
tag have margin of value zero.using the
*
selector, it will set all tags the value ofmargin: 0
, not only one tagHope it helps!
Marked as helpful - @DaniDanis@rafaeltakano
Hello There ✌
Great job on your solution!
The accessibility issues at your report are about not having landmarks elements on your page, if you include it it will be fine!
Hope it helps, happy coding 😁
- P@LucianoDLima@rafaeltakano
Hi, Luciano! 👋
Nice work on this challenge! ✌ Your solution is responsive and looks great! 👍
About BEM, it is a methodology that has Blocks, Elements and Modifiers (B.E.M).
Blocks that have Elements related to it should be named
[Block]__[Element]
And only modifiers should have the double dash
[Block]__[Element]--[Modifier]
An exemple:
//css .button {} .button__text {} .button--green {} .button--blue {} //html <button class="button button--green"> <p class="button__text"> Green Button </p> </button> <button class="button button--blue"> <p class="button__text"> Blue Button </p> </button>
I hope this helps!
Marked as helpful - @gcapistrano@rafaeltakano
Hello there ✌
To fix the part of the container gonne missing at moblie view, you'll need to refactor your code to work without the position absolute on the class
.container
And the accessibility issue at the report card, its only about the
.attribution
class that it is not wrapped with a landmark tag, wrap it on a <footer> and it will be fixed!Hope It helps!
Marked as helpful