Victoria
@vcalvo25All comments
- @dREY73Submitted about 1 month ago@vcalvo25Posted 14 days ago
Awesome job completing the project! Your solution differs from the design due to the width of your container being set to 1440px. The card itself is smaller, so it should be around 700px instead. If you want to account for different screen sizes, use the media query and edit the width for screen sizes.
0 - @mohamedballaSubmitted about 1 month agoWhat are you most proud of, and what would you do differently next time?
I was able to use Grid to produce a good layout. I need to focus more on CSS to style and layout using Grid and Flexbox.
What challenges did you encounter, and how did you overcome them?Still have problem with size , how to maintain size during screen size transition. centring is also another problem. the solution I have while did the job still aren't satisfactory.
What specific areas of your project would you like help with?Centring Element , and Element size control.
@vcalvo25Posted about 1 month agoFor different screen sizes, use @media queries. This can assist with sizing your project on different screens. After using the media queries, right click on your project website and click "Inspect". You can change the screen size at the top to various media screens such as mobile phones and computer screens.
Project looks great! But most of the links, name, and location need to be placed in h2 or h3 so that it gets the bolded effect. Also, lessen the border radius a tad by, about to 7px, to give it a more square-ish look. Keep up the great work! :)
Marked as helpful1 - @devsidersSubmitted 2 months agoWhat are you most proud of, and what would you do differently next time?
Container measurements.
What challenges did you encounter, and how did you overcome them?Padding and height of containers.
What specific areas of your project would you like help with?In the text containers.
@vcalvo25Posted 2 months agoDesign looks awesome so far! I would recommend to adjust the height and width measurements in your body, and also add in margins for the card class. This will help the card look shorter and larger for the width. Increasing the line height will also help the text to take up more space on the card.
Card looks great on smaller screen sizes, no adjustments needed. Great job!!
0 - @EfthymiosKSubmitted 2 months agoWhat are you most proud of, and what would you do differently next time?
Next time i will use tailwind-css.
@vcalvo25Posted 2 months agoThe code is well-structured and is readable. The solution is extremely close to the original design. In the original, I believe there may be a slight drop-shadow towards the bottom of the card to make it pop from the page.
Looks awesome on mobile devices; however, the footer shows a lot more white space when used on devices with a larger height of over 500 pixels (iPads and desktop monitors). Overall well done!
Marked as helpful1