Hello, Charissa Ramirez! 👋
Congratulations on finishing another challenge! In addition to Agata exhaustive feedback I would suggest you to:
- Try not to repeat your HTML code with image for each arrow. Easier way around is to use pseudo element
::before
on your.accordion__header
element. Set it toposition: relative
and your pseudo element asposition: absolute
. Put your image usingcontent: url('image.jpg')
. - Read again about BEM methodology. Overall you're doing it correctly but I found that you're using modifiers for elements and it is wrong. Modifier is an entity that defines the appearance, state, or behavior of a block or element.You used modifiers for
accordion__header--icon
andaccordion__header--text
which is wrong use of it. They're yours elements, not modifiers. For example you haveaccordion__header-text
element and then you want to change color just of that element you use MODIFIER `accordion__header-text--red".
Good luck with that, have fun coding! 💪