Demetrio Julian• 410
@dejuliansr
Submitted
Obare
@simplyObare
All comments
- Obare• 290
@simplyObare
Posted
it's amaizing
0 - KapteynUniverse• 420
@KapteynUniverse
Submitted
What specific areas of your project would you like help with?
I would like to see feedback about accessibility. There are a lot to digest in this learning path.
Obare• 290@simplyObare
Posted
nice work, though I couldn't see the modal after submission
0 - KapteynUniverse• 420
@KapteynUniverse
Submitted
What are you most proud of, and what would you do differently next time?
Reposting to progress learning path. Actually made without Js. Also learned details element and srcset attribute. Looks like details can't get a transition value tho, i would like to know if there is a way.
Obare• 290@simplyObare
Posted
Love it, but you should probably add a shadow to the card to make it more visible.
1 - EL harabiy• 230
@Mubarak-Adeyemi
Submitted
What are you most proud of, and what would you do differently next time?
I’m most proud of successfully implementing the popovertarget attribute and creating a fully accessible design using ARIA attributes. I also achieved a smooth user experience with form reset functionality after modal interaction. Next time, I’d focus on adding real-time form validation and improving keyboard accessibility for even better usability.
What challenges did you encounter, and how did you overcome them?
The biggest challenge was integrating the popovertarget within the form. The form's default behavior was interfering with the modal popover. I solved this by using e.preventDefault() to manually handle form submission and modal display. I also encountered difficulties with error messaging, which I fixed by switching from the hidden attribute to using classList in JavaScript along with aria-live to make error messages accessible.
Obare• 290@simplyObare
Posted
looks good
Marked as helpful
0 - yahappylemon• 170
@yahappylemon
Submitted
Obare• 290@simplyObare
Posted
looks great not far off from the design
0 - PMohd Sadaf• 690
@MsadafK
Submitted
Obare• 290@simplyObare
Posted
great
1 - PSam Hooker• 550
@35degrees
Submitted
What are you most proud of, and what would you do differently next time?
I wish I had done a mid-sized width with two columns, it's a huge switch from 4 columns to one. Proud that I tested my skills and text wrap edits.
Obare• 290@simplyObare
Posted
love the way you handled the height of the cards, I personally did found that challenging.
0 - PSam Hooker• 550
@35degrees
Submitted
What are you most proud of, and what would you do differently next time?
I'm proud that I used both flex and grid, got the border color and height just right, and used absolute positioning for the image icons.
What challenges did you encounter, and how did you overcome them?
in the smaller width media query, it took me a minute to realize I have to 'reset' the grid-template-rows or the grid tries to force itself into the bigger pattern even at one column.
Obare• 290@simplyObare
Posted
I love it, just a pointer; a fixed height does limit the spacing and the ability for the content inside the card to wrap
Marked as helpful
0 - Solomon Johnson• 60
@Roxxy28
Submitted
What are you most proud of, and what would you do differently next time?
Add more time and be patient with the learning process
What challenges did you encounter, and how did you overcome them?
The images were not loading on with the live url
What specific areas of your project would you like help with?
More knowledge with CSS
Obare• 290@simplyObare
Posted
The reason your images aren't loading is because you are siting them incorrectly. You should either put them in an images folder or remove the "images/…." Example:
- What you have:
<img class="supervisor-img" src="images/icon-supervisor.svg" alt="supervisor-icon"> - how it should be:
<img class="supervisor-img" src="/icon-supervisor.svg" alt="supervisor-icon">
Marked as helpful
1 - What you have:
- Sedar Yildirim• 250
@sedaryildirim
Submitted
What challenges did you encounter, and how did you overcome them?
First time with a mobile first work flow.
Obare• 290@simplyObare
Posted
Thumb's up
1 - Sedar Yildirim• 250
@sedaryildirim
Submitted
What challenges did you encounter, and how did you overcome them?
Had my first issues with this
-
Border radius failed to work multiple times on my img, discovered this was due to img having padding, created workaround with margin and another container to wrap everything in.
-
Border radius failed to work on my main card/article - !important tag after border radius fixed this for me.
Obare• 290@simplyObare
Posted
Love it, You did forget to use the brown color on some headers and the nutrition units.
Marked as helpful
0 -
- meredithincode• 90
@meredithincode
Submitted
What specific areas of your project would you like help with?
I'm not sure why my screenshot on frontendmentor isn't displaying my avatar image properly. As you can see from my live site and github, my code is correct and all is working.
Obare• 290@simplyObare
Posted
Am not sure if this will be helpful, but try setting the max-width of the image to 100%
0
