Latest comments
- @TathianiLimaSubmitted 12 days ago@toxa-devPosted 12 days ago
Hi,
Your work looks great! I think there might be a small mistake—"Ingredients" is wrapped in an <h1> instead of an <h2>.
All the best!
Marked as helpful0 - @EniloladeSubmitted 13 days agoWhat are you most proud of, and what would you do differently next time?
The hover effect. Subtle but nice.
What challenges did you encounter, and how did you overcome them?Resizing
What specific areas of your project would you like help with?Not sure
@toxa-devPosted 12 days agoHi, Nice Job! I have a couple of small suggestions that might improve readability and overall presentation:
- Increasing the font size would make the text easier to read, especially on larger screens.
- Vertically centering the container could help create a more balanced and visually appealing layout.
Let me know what you think!
0 - P@ecarlsteSubmitted 13 days agoWhat are you most proud of, and what would you do differently next time?
I got a bit stuck on this one for a while due to the fact that the website that I created to host these little components and website designs a few years back when Node 18.x was LTS. I couldn't for the life of me get the tailwind class names to work once I had deployed the code to Vercel. Locally they worked perfectly, but in Vercel nothing worked. I didn't realize that Vercel has a project setting that's auto set when you create the project which had set Node to 18.x. After a few hours of trying different things I found that setting and once it matched what I was using with pnpm locally, 22.x, everything worked perfectly.
I think next time I'll be sure to check that setting very soon now that I know it exists! :-)
What challenges did you encounter, and how did you overcome them?The biggest challenge was also what I mentioned previously in the "What are you most proud of?" section.
I overcame it by lots of experimentation and reading online. I tried LLMs to help as well but they didn't come close to figuring it out.
After I figured out that setting exists I wrote a little TypeScript
What specific areas of your project would you like help with?prebuild
script to check the version of both pnpm and nodejs to see if they match the versions that I have specified in both.tool-versions
andpackage.json
for the repo. If they don't match the versions a dev would be running locally then the build in Vercel will fail and give a helpful error message so that this never happens again!I'd like some help looking at the use of any semantic HTML elements within the card like
<h1>
tags, etc. I'm not 100% sure how to deal with that since there is no guarantee how this component would be used within another site's semantic structure.Maybe potentially passing the heading level via a prop would work? I'd love to get some feedback on how to do this effectively or hear about any ideas at all related to the semantic structure of the component.
- @AdonMazumder26Submitted 21 days ago@toxa-devPosted 20 days ago
Hello! Nice Job! Maybe a bigger font size would be better for my eyes?
0 - @toxa-devSubmitted about 1 month agoWhat specific areas of your project would you like help with?
HTML semantics: Is it a good choice of tags?
@toxa-devPosted about 1 month agoHello guys, thank you for your help. The transition was actually there, but it was written like this:
transition: background-color 0.3s ease; transition: color 0.1s ease;
The background-color transition didn't work because it was overridden. I didn't know this at the time. Also, it seems like 0.3s and 0.1s are too short; maybe 0.5s is indeed a better choice.
Appreciate your help and kind comments!
2