@Vidottizzz
Latest solutions
Latest comments
- @MariaValentinova
Hi, there! I personally took advantage of all the default functionalities of Bootstrap to create the accordion and control its behavior. You can look at my code in case it is helpful to you. You can achieve closing the previously opened item when opening a new one by keeping track of the currently opened item and closing it before opening the new one. Here are some suggestions to implement this in your code: 1.Create a global variable to track the currently opened item. You can initialize it with null initially since no item is open at the beginning. Like: let currentItem = null; 2.Modify the showhide and hideshow functions: When opening an item, check if there is currently an open item (currentItem is not null). If there is one, close it before opening the new item. When closing an item, update the currentItem variable to null.
- @EugeniaAntonova@MariaValentinova
Hi there! I don´t see the assets/images/icon-star.svg that should be before the FAQ´s title.
Marked as helpful - @mhykah-webdevWhat are you most proud of, and what would you do differently next time?
Wasn't too hard!
What challenges did you encounter, and how did you overcome them?I had to review how to change images based on media queries
What specific areas of your project would you like help with?Semantic html, best practices that I might be missing.
@MariaValentinovaGreat job! Did you do this with access to figma? It's just excellent 👌 I think you could merge this part: /*
- Use a more-intuitive box-sizing model. */ *, *::before, *::after { box-sizing: border-box; }
/* 2. Remove default margin */
- { margin: 0; } or you didn't want to get margin: 0 to after and before? If you adjust the margin you have in the card and the button "Add to card" it will be exactly the same as the design.
- @andreaskdk@MariaValentinova
Hi there! The image is not displaying, nor is it centered. I experienced this too and had to adjust the paths.
Marked as helpful - @Abbassher55@MariaValentinova
Hi there! I find this to be very well executed. Perhaps a few minor details could be fine-tuned, such as the yellow background being cropped when scaled below 320px, and in my opinion, the font-weight needs adjustment. For someone starting, I think this is excellent work! It's been challenging for me to pinpoint areas for improvement. This showcases the work of someone with a promising future, in my opinion.
Marked as helpful - @vcgmuse@MariaValentinova
I think you could optimize your code. I don't understand why you're accommodating so many screen size options. In the documentation, it says it's intended for mobile screens of 375px and desktops of 1440px, I believe. This way, there would be much less code. You should pay more attention to proportions: border-radius, and margins. Good use of the CSS flex property!
Marked as helpful
