If you see something that I could improve, please, tell me, I appreciate it, thank you!
Jorge
@f-avalosAll comments
- @jennawlin3Submitted 6 months agoWhat specific areas of your project would you like help with?
- @Shanahan914Submitted 5 months agoWhat are you most proud of, and what would you do differently next time?
First Junior challenge completed. Learned a few things but no major issues. Not happy with the JS so I'll refactor that over the next week. I'm pretty sure it's not well-written but does work.
@f-avalosPosted 20 days agoEverything is very good, maybe change the color of the border and the padding
0 - @pierrehany1010Submitted 3 months ago
- @hyperavtrSubmitted 3 months ago
- @arsh-deSubmitted 29 days agoWhat are you most proud of, and what would you do differently next time?
I am proud able to create the website and make it work.
What challenges did you encounter, and how did you overcome them?Getting the right design was a little challenging.
What specific areas of your project would you like help with?I am tried using tailwind on this project, but it got to complicated to read and I had to change it in different places, can someone help me with that.
@f-avalosPosted 28 days agonice solution, but the site is not the exactly same at the preview, i guess because you try to use Tailwind.
0 - @jevcenkokozlovskaSubmitted about 1 month ago@f-avalosPosted about 1 month ago
maybe you could improve the responsive design, but overall it's very good.
0 - @m-maher97Submitted about 1 month ago
- @CoolNight99Submitted about 1 month agoWhat are you most proud of, and what would you do differently next time?
I think I got the design pretty close to what was given.
What challenges did you encounter, and how did you overcome them?It was tricky to get the sizes of the grid items correct but then I remembered the grid-row and grid-column properties.
- @eros77scSubmitted about 1 month agoWhat specific areas of your project would you like help with?
I had trouble centering the text after the main title. I was using Flexbox, and when I applied
max-width
to the text to create the line-break effect without using ``, the text was aligned to the left, as if it wasn’t following Flexbox behavior. Neithertext-align
noralign-self
set tocenter
worked. I usedmargin-inline-start
, but I would like another solution.Any other tips about something I might be missing would be greatly appreciated!
@f-avalosPosted about 1 month agoThe width of the cards is static, you could adjust it to fit the screen before it drops below 768px resolution, the rest is very good.
0 - @Fikerte-TSubmitted about 1 month ago
- @gabrilovSubmitted about 1 month ago
- @haloha1410Submitted 2 months ago@f-avalosPosted about 2 months ago
Very good, maybe you could add some type of transition when selecting a button
0